Don't run DefaultEventHandler on disabled input elements [chromium/src : main]
0 views
Skip to first unread message
David Baron (Gerrit)
Jul 31, 2026, 4:58:00 PM (10 hours ago) Jul 31
to David Baron, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, blink-rev...@chromium.org, blink-...@chromium.org
Attention needed from Joey Arhar
David Baron added 1 comment![]( "Open in Gerrit")
Patchset-level comments
File-level comment, Patchset 3 (Latest):
David Baron . resolved
This basically looks fine -- except I wonder if this is risky enough that it should have a killswitch flag. (I didn't really look in to what's in the default event handling code, but I suspect there are a decent number of things in there...)
Related details
Attention is currently required from:
- Joey Arhar
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
David Baron (Gerrit)
Jul 31, 2026, 4:58:13 PM (10 hours ago) Jul 31
to David Baron, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, blink-rev...@chromium.org, blink-...@chromium.org
Attention needed from Joey Arhar
David Baron added 1 comment![]( "Open in Gerrit")
Patchset-level comments
David Baron . unresolved
This basically looks fine -- except I wonder if this is risky enough that it should have a killswitch flag. (I didn't really look in to what's in the default event handling code, but I suspect there are a decent number of things in there...)
David Baron
.
Related details
Attention is currently required from:
- Joey Arhar
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
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