Skip to content

feat(trace-viewer): show message source in the console panel#41342

Open
dgozman wants to merge 1 commit into
microsoft:mainfrom
dgozman:fix-41320
Open

feat(trace-viewer): show message source in the console panel#41342
dgozman wants to merge 1 commit into
microsoft:mainfrom
dgozman:fix-41320

Conversation

@dgozman

@dgozman dgozman commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add a per-entry "source" badge to the Console panel, like the Network panel's Source column: page#N when multiple pages, page for a single page, and test for runner (stdout/stderr) messages.
  • This badge replaces the old browser/file icon + separate page id badge, which were redundant.
  • Move page/api id assignment onto TraceModel (pagerefToTitle, contextToTitle) so the Network and Console panels label the same source identically.
Screenshot 2026-06-17 at 1 20 27 PM

References #41320

Adds a per-entry source badge to the Console panel, mirroring the
Network panel's "Source" column: page#N for browser messages from
multiple pages, "page" for a single page, and "test" for runner
(stdout/stderr) messages. The page/api id assignment is unified onto
TraceModel so both panels label the same source identically.

Fixes: microsoft#41320
@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

7354 passed, 1122 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

4 flaky ⚠️ [chromium-library] › library/chromium/chromium.spec.ts:177 › serviceWorker(), and fromServiceWorker() work `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/popup.spec.ts:260 › should not throw when click closes popup `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:717 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:717 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node20`

39605 passed, 743 skipped


Merge workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant