[AdDensity] Add Mojo boundary check for outermost main frame metadata [chromium/src : main]

0 views
Skip to first unread message

Yao Xiao (Gerrit)

unread,
Apr 28, 2026, 7:59:20 PM (20 hours ago) Apr 28
to Shivani Sharma, Chromium LUCI CQ, chromium...@chromium.org, bmcquad...@chromium.org, csharris...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org
Attention needed from Shivani Sharma

Yao Xiao added 1 comment

Patchset-level comments
File-level comment, Patchset 15 (Latest):
Yao Xiao . resolved

shivanisha@: PTAL. Thanks! Adding you here given your prior context on the bug report and the FF/MPArch aspects of the CL (e.g., PageLoadMetricsForwardObserver).

Open in Gerrit

Related details

Attention is currently required from:
  • Shivani Sharma
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not 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: I05bcefb2bd801840cfc44017b3d109273b4d91c1
Gerrit-Change-Number: 7796521
Gerrit-PatchSet: 15
Gerrit-Owner: Yao Xiao <yao...@chromium.org>
Gerrit-Reviewer: Shivani Sharma <shiva...@chromium.org>
Gerrit-Reviewer: Yao Xiao <yao...@chromium.org>
Gerrit-Attention: Shivani Sharma <shiva...@chromium.org>
Gerrit-Comment-Date: Tue, 28 Apr 2026 23:59:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages