Revert "[html] Fix body margin precedence and iframe margin injection" [chromium/src : refs/branch-heads/7727]
0 views
Skip to first unread message
Mason Freed (Gerrit)
Mar 31, 2026, 1:24:49 PM (2 days ago) Mar 31
to Hyowon Kim, AyeAye, chromium...@chromium.org, Chromium LUCI CQ, blink-revie...@chromium.org, blink-...@chromium.org, kinuko...@chromium.org, blink-rev...@chromium.org, jmedle...@chromium.org
Attention needed from Hyowon Kim
Mason Freed added 1 comment![]( "Open in Gerrit")
Patchset-level comments
File-level comment, Patchset 4 (Latest):
Mason Freed . resolved
The revert looks good, but you'll need to request merge approval first:
Related details
Attention is currently required from:
- Hyowon Kim
Submit Requirements:
Code-Owners
Code-Review
Lint
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Hyowon Kim (Gerrit)
Apr 1, 2026, 8:05:04 PM (10 hours ago) Apr 1
to Mason Freed, AyeAye, chromium...@chromium.org, Chromium LUCI CQ, blink-revie...@chromium.org, blink-...@chromium.org, kinuko...@chromium.org, blink-rev...@chromium.org, jmedle...@chromium.org
Attention needed from Mason Freed
Hyowon Kim added 1 comment![]( "Open in Gerrit")
Patchset-level comments
Mason Freed . unresolved
The revert looks good, but you'll need to request merge approval first:
Hyowon Kim
I've submitted the merge request for M147: https://issues.chromium.org/issues/498381867
Is there anything else I need to do on my side?
Related details
Attention is currently required from:
- Mason Freed
Submit Requirements:
Code-Owners
Code-Review
Lint
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Mason Freed (Gerrit)
Apr 1, 2026, 8:09:53 PM (10 hours ago) Apr 1
to Hyowon Kim, AyeAye, chromium...@chromium.org, Chromium LUCI CQ, blink-revie...@chromium.org, blink-...@chromium.org, kinuko...@chromium.org, blink-rev...@chromium.org, jmedle...@chromium.org
Attention needed from Hyowon Kim
Mason Freed added 1 comment![]( "Open in Gerrit")
Patchset-level comments
Mason Freed . unresolved
Hyowon KimThe revert looks good, but you'll need to request merge approval first:
I've submitted the merge request for M147: https://issues.chromium.org/issues/498381867
Is there anything else I need to do on my side?
Mason Freed
Yeah, you'll have to wait for that request (which looks good now) to get approved. Also, if you set the "Auto Cherry Pick" field to "Yes", then the system will create this merge CL for you and automatically land it.
Related details
Attention is currently required from:
- Hyowon Kim
Submit Requirements:
Code-Owners
Code-Review
Lint
No-Unresolved-Comments
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