RESOLVED FIXED 228845
preventDefault() on keydown event should prevent dialog cancelation from happening 
https://bugs.webkit.org/show_bug.cgi?id=228845
Summary preventDefault() on keydown event should prevent dialog cancelation from happ...
Tim Nguyen (:ntim)
Reported 2021-08-05 14:56:44 PDT
This is covered by dialog-keydown-preventDefault.html: https://github.com/web-platform-tests/wpt/blob/master/html/semantics/interactive-elements/the-dialog-element/dialog-keydown-preventDefault.html Not sure if there are any other events that should also prevent dialog cancelation, but they should be covered by tests as well if so.
Attachments
Patch (5.99 KB, patch)
2021-08-24 03:36 PDT, Tim Nguyen (:ntim) 		koivisto: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-08-11 13:40:44 PDT
<rdar://problem/81811203>
Tim Nguyen (:ntim)
Comment 2 2021-08-24 03:36:54 PDT
Created attachment 436273 [details] Patch
Tim Nguyen (:ntim)
Comment 3 2021-08-24 05:57:54 PDT
Committed r281492 (240866@main): <https://commits.webkit.org/240866@main>
Ryosuke Niwa
Comment 4 2021-08-24 11:01:55 PDT
Comment on attachment 436273 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=436273&action=review > Source/WebCore/page/EventHandler.cpp:3804 > + if (auto* activeModalDialog = m_frame.document()->activeModalDialog()) Use RefPtr > Source/WebCore/page/EventHandler.cpp:3812 > + Why are we adding a blank line here?
Note You need to log in before you can comment on or make changes to this bug.