Fix attaching to worklet twice [chromium/src : main]

0 views
Skip to first unread message

Andrey Kosyakov (Gerrit)

unread,
Dec 22, 2025, 11:10:26 PM (9 hours ago) Dec 22
to Peter Kvitek, Maksim Sadym, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org
Attention needed from Maksim Sadym and Peter Kvitek

Andrey Kosyakov added 1 comment

File third_party/blink/renderer/core/inspector/worker_inspector_controller.cc
Line 138, Patchset 1 (Latest): inspector_agents_.insert(session, inspector_agent);
Andrey Kosyakov . resolved

Right now we do not dispatch `Inspector.workerLoaded`, but we'll add this as a follow-up for consistency wrt workers.

Open in Gerrit

Related details

Attention is currently required from:
  • Maksim Sadym
  • Peter Kvitek
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • 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: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: Ibe49b1f30b1c55c68dcd4f8202fa848f168e12b6
Gerrit-Change-Number: 7306210
Gerrit-PatchSet: 1
Gerrit-Owner: Andrey Kosyakov <ca...@chromium.org>
Gerrit-Reviewer: Andrey Kosyakov <ca...@chromium.org>
Gerrit-Reviewer: Maksim Sadym <sa...@chromium.org>
Gerrit-Reviewer: Peter Kvitek <kvi...@chromium.org>
Gerrit-Attention: Maksim Sadym <sa...@chromium.org>
Gerrit-Attention: Peter Kvitek <kvi...@chromium.org>
Gerrit-Comment-Date: Tue, 23 Dec 2025 04:10:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages