Apply an `<iframe>` security check to other frame-owning elements. [chromium/src : main]
0 views
Skip to first unread message
Mike West (Gerrit)
3:26 AM (10 hours ago) 3:26 AM
to Antonio Sartori, Chromium LUCI CQ, chromium...@chromium.org, blink-rev...@chromium.org, blink-...@chromium.org
Attention needed from Antonio Sartori
Mike West added 1 comment![]( "Open in Gerrit")
Patchset-level comments
File-level comment, Patchset 3 (Latest):
Mike West . resolved
Hey Antonio, would you mind looking at this reland? The delta between 1 and 2 is the only chance from the original, which drops the checks against the stack trace itself as that looks to be flakily reordered on ChromeOS and Android.
Related details
Attention is currently required from:
- Antonio Sartori
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Antonio Sartori (Gerrit)
7:54 AM (6 hours ago) 7:54 AM
to Mike West, Chromium LUCI CQ, chromium...@chromium.org, blink-rev...@chromium.org, blink-...@chromium.org
Attention needed from Mike West
Antonio Sartori voted Code-Review+1![]( "Open in Gerrit")
Attention is currently required from:
- Mike West
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages