[RPP] Force tab targetType in trace_app [devtools/devtools-frontend : main]

0 views
Skip to first unread message

Connor Clark (Gerrit)

unread,
Oct 13, 2025, 5:03:04 PM (9 hours ago) Oct 13
to Paul Irish, devtools-rev...@chromium.org
Attention needed from Paul Irish

Connor Clark voted Auto-Submit+1

Auto-Submit+1
Open in Gerrit

Related details

Attention is currently required from:
  • Paul Irish
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: devtools/devtools-frontend
Gerrit-Branch: main
Gerrit-Change-Id: I62e656d9f92892d8b051e0c7b98c663b432499d1
Gerrit-Change-Number: 7037208
Gerrit-PatchSet: 1
Gerrit-Owner: Connor Clark <cja...@chromium.org>
Gerrit-Reviewer: Connor Clark <cja...@chromium.org>
Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
Gerrit-Attention: Paul Irish <paul...@chromium.org>
Gerrit-Comment-Date: Mon, 13 Oct 2025 21:02:58 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Paul Irish (Gerrit)

unread,
Oct 13, 2025, 5:28:22 PM (9 hours ago) Oct 13
to Connor Clark, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org
Attention needed from Connor Clark

Paul Irish voted

Code-Review+1
Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Connor Clark
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: devtools/devtools-frontend
Gerrit-Branch: main
Gerrit-Change-Id: I62e656d9f92892d8b051e0c7b98c663b432499d1
Gerrit-Change-Number: 7037208
Gerrit-PatchSet: 1
Gerrit-Owner: Connor Clark <cja...@chromium.org>
Gerrit-Reviewer: Connor Clark <cja...@chromium.org>
Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Attention: Connor Clark <cja...@chromium.org>
Gerrit-Comment-Date: Mon, 13 Oct 2025 21:28:14 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Connor Clark (Gerrit)

unread,
Oct 13, 2025, 7:01:17 PM (7 hours ago) Oct 13
to Paul Irish, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org
Attention needed from Paul Irish

Connor Clark voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Paul Irish
Submit Requirements:
    • requirement satisfiedCode-Owners
    • requirement satisfiedCode-Review
    • requirement satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: devtools/devtools-frontend
    Gerrit-Branch: main
    Gerrit-Change-Id: I62e656d9f92892d8b051e0c7b98c663b432499d1
    Gerrit-Change-Number: 7037208
    Gerrit-PatchSet: 3
    Gerrit-Owner: Connor Clark <cja...@chromium.org>
    Gerrit-Reviewer: Connor Clark <cja...@chromium.org>
    Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
    Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
    Gerrit-Attention: Paul Irish <paul...@chromium.org>
    Gerrit-Comment-Date: Mon, 13 Oct 2025 23:01:06 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Devtools-frontend LUCI CQ (Gerrit)

    unread,
    Oct 13, 2025, 7:03:00 PM (7 hours ago) Oct 13
    to Connor Clark, Paul Irish, devtools-rev...@chromium.org

    Devtools-frontend LUCI CQ submitted the change

    Unreviewed changes

    1 is the latest approved patch-set.
    No files were changed between the latest approved patch-set and the submitted one.

    Change information

    Commit message:
    [RPP] Force tab targetType in trace_app

    This is required for some features to work, such as linkifying the
    console source location in the bottom-up / call-tree tables. The short
    summary is that `primaryPageTarget()` does not return the correct target
    for rehydrated connections unless the first / root target is of type
    tab, which breaks linkifying.
    Bug: 427455148
    Change-Id: I62e656d9f92892d8b051e0c7b98c663b432499d1
    Reviewed-by: Paul Irish <paul...@chromium.org>
    Auto-Submit: Connor Clark <cja...@chromium.org>
    Commit-Queue: Connor Clark <cja...@chromium.org>
    Files:
    • M front_end/entrypoints/inspector_main/InspectorMain.ts
    Change size: XS
    Delta: 1 file changed, 3 insertions(+), 1 deletion(-)
    Branch: refs/heads/main
    Submit Requirements:
    • requirement satisfiedCode-Review: +1 by Paul Irish
    Open in Gerrit
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: merged
    Gerrit-Project: devtools/devtools-frontend
    Gerrit-Branch: main
    Gerrit-Change-Id: I62e656d9f92892d8b051e0c7b98c663b432499d1
    Gerrit-Change-Number: 7037208
    Gerrit-PatchSet: 4
    Gerrit-Owner: Connor Clark <cja...@chromium.org>
    Gerrit-Reviewer: Connor Clark <cja...@chromium.org>
    Gerrit-Reviewer: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
    Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
    open
    diffy
    satisfied_requirement
    Reply all
    Reply to author
    Forward
    0 new messages