[Lighthouse] Add agentic browsing category [devtools/devtools-frontend : main]

0 views
Skip to first unread message

Alex Rudenko (Gerrit)

unread,
11:14 AM (4 hours ago) 11:14 AM
to Lusa Zhan, Paul Irish, android-bu...@system.gserviceaccount.com, devtools-rev...@chromium.org, lighthouse-en...@google.com
Attention needed from Lusa Zhan and Paul Irish

Alex Rudenko voted and added 1 comment

Votes added by Alex Rudenko

Code-Review+1

1 comment

File front_end/panels/lighthouse/LighthouseController.ts
Line 90, Patchset 2 (Latest): agenticBrowsing: 'Agentic Browsing',
Alex Rudenko . unresolved
```suggestion
agenticBrowsing: 'Agentic browsing',
```

I think per UX writing guidelines it should start with a lowercase letter. Feel free to ignore the suggestion if the title was approved elsewhere with the capital Browsing.

Open in Gerrit

Related details

Attention is currently required from:
  • Lusa Zhan
  • Paul Irish
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedNo-Unresolved-Comments
  • 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: I199a87fd59cf6cdaaacd5205259aa29d4b6ff99b
Gerrit-Change-Number: 7806870
Gerrit-PatchSet: 2
Gerrit-Owner: Lusa Zhan <lusa...@chromium.org>
Gerrit-Reviewer: Alex Rudenko <alexr...@chromium.org>
Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
Gerrit-Attention: Paul Irish <paul...@chromium.org>
Gerrit-Attention: Lusa Zhan <lusa...@chromium.org>
Gerrit-Comment-Date: Mon, 04 May 2026 15:14:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Lusa Zhan (Gerrit)

unread,
11:23 AM (4 hours ago) 11:23 AM
to Alex Rudenko, Paul Irish, android-bu...@system.gserviceaccount.com, devtools-rev...@chromium.org, lighthouse-en...@google.com
Attention needed from Alex Rudenko and Paul Irish

Lusa Zhan added 1 comment

File front_end/panels/lighthouse/LighthouseController.ts
Line 90, Patchset 2: agenticBrowsing: 'Agentic Browsing',
Alex Rudenko . resolved
```suggestion
agenticBrowsing: 'Agentic browsing',
```

I think per UX writing guidelines it should start with a lowercase letter. Feel free to ignore the suggestion if the title was approved elsewhere with the capital Browsing.

Lusa Zhan

Done

Open in Gerrit

Related details

Attention is currently required from:
  • Alex Rudenko
  • 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: I199a87fd59cf6cdaaacd5205259aa29d4b6ff99b
    Gerrit-Change-Number: 7806870
    Gerrit-PatchSet: 3
    Gerrit-Owner: Lusa Zhan <lusa...@chromium.org>
    Gerrit-Reviewer: Alex Rudenko <alexr...@chromium.org>
    Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
    Gerrit-Attention: Paul Irish <paul...@chromium.org>
    Gerrit-Attention: Alex Rudenko <alexr...@chromium.org>
    Gerrit-Comment-Date: Mon, 04 May 2026 15:23:10 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    Comment-In-Reply-To: Alex Rudenko <alexr...@chromium.org>
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Paul Irish (Gerrit)

    unread,
    1:49 PM (2 hours ago) 1:49 PM
    to Lusa Zhan, Alex Rudenko, android-bu...@system.gserviceaccount.com, devtools-rev...@chromium.org, lighthouse-en...@google.com
    Attention needed from Alex Rudenko and Lusa Zhan

    Paul Irish added 1 comment

    File front_end/panels/lighthouse/LighthouseProtocolService.ts
    Line 156, Patchset 3 (Latest): if (!config && categoryIDs.includes('agentic-browsing')) {
    Paul Irish . unresolved

    something is weird with this.


    so we have 'self.createConfig' which is exposed manually and gets used here in the worker: https://source.chromium.org/chromium/chromium/src/+/main:third_party/devtools-frontend/src/front_end/entrypoints/lighthouse_worker/LighthouseWorkerService.ts;l=79?q=f:lighthouse%20config%20-f:locales&ss=chromium%2Fchromium%2Fsrc:third_party%2Fdevtools-frontend%2Fsrc%2Ffront_end%2F

    with that.. we should just need to pass the categoryid here from protocolservice over to workerservice and then everything _should_ work.

    if agenticbrowsing still requires custom config... then.. that complicates this scenario. (but even if so.. i'm sure there's another solution than defining the category again outside of the LH codebase)

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Alex Rudenko
    • Lusa Zhan
    Submit Requirements:
      • requirement satisfiedCode-Owners
      • requirement is not satisfiedCode-Review
      • requirement is not satisfiedNo-Unresolved-Comments
      • 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: I199a87fd59cf6cdaaacd5205259aa29d4b6ff99b
      Gerrit-Change-Number: 7806870
      Gerrit-PatchSet: 3
      Gerrit-Owner: Lusa Zhan <lusa...@chromium.org>
      Gerrit-Reviewer: Alex Rudenko <alexr...@chromium.org>
      Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
      Gerrit-Attention: Lusa Zhan <lusa...@chromium.org>
      Gerrit-Attention: Alex Rudenko <alexr...@chromium.org>
      Gerrit-Comment-Date: Mon, 04 May 2026 17:49:23 +0000
      Gerrit-HasComments: Yes
      Gerrit-Has-Labels: No
      satisfied_requirement
      unsatisfied_requirement
      open
      diffy
      Reply all
      Reply to author
      Forward
      0 new messages