Skip to content

Commit

Permalink
fix: hide route calls from tracing (#3032)
Browse files Browse the repository at this point in the history
  • Loading branch information
mxschmitt authored Oct 24, 2024
1 parent e6e6da4 commit 70d67ad
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
1 change: 0 additions & 1 deletion src/Playwright.Tests/TracingTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,6 @@ await Context.Tracing.StartAsync(new()
"Page.EvaluateAsync",
"Page.RouteAsync",
"Page.GotoAsync",
"Route.ContinueAsync",
"Page.GotoAsync"
};
Assert.AreEqual(expectedActionApiNames, actualActionApiNames);
Expand Down
4 changes: 2 additions & 2 deletions src/Playwright/Transport/Connection.cs
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ internal Task<T> SendMessageToServerAsync<T>(
ChannelOwner @object,
string method,
Dictionary<string, object> args = null,
bool keepNulls = false) => WrapApiCallAsync(() => InnerSendMessageToServerAsync<T>(@object, method, args, keepNulls));
bool keepNulls = false) => WrapApiCallAsync(() => InnerSendMessageToServerAsync<T>(@object, method, args, keepNulls), @object?._isInternalType ?? false);

private async Task<T> InnerSendMessageToServerAsync<T>(
ChannelOwner @object,
Expand Down Expand Up @@ -184,7 +184,7 @@ private async Task<T> InnerSendMessageToServerAsync<T>(
};
}

if (_tracingCount > 0 && frames.Count > 0 && !@object._isInternalType)
if (_tracingCount > 0 && frames.Count > 0 && @object.Guid != "localUtils")
{
LocalUtils.AddStackToTracingNoReply(frames, id);
}
Expand Down

0 comments on commit 70d67ad

Please sign in to comment.