[M148] Revert "Report mojo bad message for incorrect site_for_cookies and top_frame_origin" [chromium/src : refs/branch-heads/7766]

0 views

Skip to first unread message

Krishna Govind (Gerrit)

unread,

Apr 1, 2026, 2:40:56 PM (3 days ago) Apr 1

to Lily Chen, Chromium LUCI CQ, AyeAye, network-ser...@chromium.org

Attention needed from Lily Chen

Krishna Govind voted

Code-Review	+1
Owners-Override	+1

Open in Gerrit

Related details

Attention is currently required from:

Lily Chen

Submit Requirements:

Code-Owners
Code-Review
Lint
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

open

diffy

Krishna Govind (Gerrit)

unread,

Apr 1, 2026, 2:41:11 PM (3 days ago) Apr 1

to Lily Chen, Chromium LUCI CQ, AyeAye, network-ser...@chromium.org

Krishna Govind submitted the change

Change information

Commit message:

[M148] Revert "Report mojo bad message for incorrect site_for_cookies and top_frame_origin"

This reverts commit 225000ab6d13272cf32d41ea8cdba1ff65835a86.

Reason for revert: crashes from this change block release; root cause of
inconsistencies is not resolved yet so killing the renderer is not safe
for stability.

Original change's description:
> Report mojo bad message for incorrect site_for_cookies and top_frame_origin
>
> RestrictedCookieManager now explicitly reports a bad message and
> terminates the connection if the renderer provides an incorrect
> site_for_cookies or top_frame_origin. Previously, these mismatches
> were only logged as errors without further action.
>
> This change ensures that inconsistencies between the renderer's
> provided context and the browser's bound state are treated as
> protocol violations.
>
> Fixed: 495405493
> Change-Id: Ifcecd1250f4fc9d5015369ac9df7914e317d8974
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7707502
> Reviewed-by: Kenichi Ishibashi <ba...@chromium.org>
> Commit-Queue: Yoichi Osato <yoi...@chromium.org>
> Reviewed-by: Charlie Reis <cr...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1607622}

(cherry picked from commit c243f51102a4b8fe850ebb1757861e277bc8052b)