[M146] Speculative fix for another BadMessage class in RCM. [chromium/src : refs/branch-heads/7680]

0 views
Skip to first unread message

Rubber Stamper (Gerrit)

unread,
Mar 2, 2026, 12:46:11 PM (yesterday) Mar 2
to Chrome Cherry Picker, Maks Orlovich, AyeAye, network-ser...@chromium.org

Rubber Stamper voted

Bot-Commit+1
Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedLint
  • requirement 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: refs/branch-heads/7680
Gerrit-Change-Id: I3a83570e2ba24c01a4eb0c9c2c30b8794c2544c7
Gerrit-Change-Number: 7623870
Gerrit-PatchSet: 2
Gerrit-Reviewer: Chrome Cherry Picker <chrome-che...@chops-service-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Gerrit-CC: Maks Orlovich <morl...@chromium.org>
Gerrit-Comment-Date: Mon, 02 Mar 2026 17:46:06 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
Mar 2, 2026, 3:07:36 PM (22 hours ago) Mar 2
to Chrome Cherry Picker, Maks Orlovich, Rubber Stamper, AyeAye, network-ser...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:
[M146] Speculative fix for another BadMessage class in RCM.

Original change's description:
> Speculative fix for another BadMessage class in RCM.
>
> We somehow receive listener add requests with an empty
> URL. I am not sure how we get into this state, but reports
> in wild and release schedule require a rapid response, so
> just ignore this case; that's behaviorally correct anyway.
> The origin in the renderer probably isn't opaque since there is
> a report on 147.0.7703.0
>
> Bug: 488084020
> Change-Id: I3a83570e2ba24c01a4eb0c9c2c30b8794c2544c7
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7617215
> Commit-Queue: Maks Orlovich <morl...@chromium.org>
> Reviewed-by: Chris Fredrickson <cfre...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1591734}

(cherry picked from commit 04e954426c86152708bd62420562c991e0411282)
Bug: 488792907,488084020
Change-Id: I3a83570e2ba24c01a4eb0c9c2c30b8794c2544c7
Commit-Queue: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/7680@{#1770}
Cr-Branched-From: 76b7d80e5cda23fe6537eed26d68c92e995c7f39-refs/heads/main@{#1582197}
Files:
  • M services/network/restricted_cookie_manager.cc
Change size: XS
Delta: 1 file changed, 7 insertions(+), 0 deletions(-)
Branch: refs/branch-heads/7680
Submit Requirements:
  • requirement satisfiedCode-Review: Bot-Commit+1 by Rubber Stamper
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: chromium/src
Gerrit-Branch: refs/branch-heads/7680
Gerrit-Change-Id: I3a83570e2ba24c01a4eb0c9c2c30b8794c2544c7
Gerrit-Change-Number: 7623870
Gerrit-PatchSet: 3
Gerrit-Reviewer: Chrome Cherry Picker <chrome-che...@chops-service-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages