RESOLVED FIXED 231327
Rewrite LayoutTests/plugins/object-onfocus-mutation-crash.html to stop relying on autofocus
https://bugs.webkit.org/show_bug.cgi?id=231327
Summary Rewrite LayoutTests/plugins/object-onfocus-mutation-crash.html to stop relyin...
Tim Nguyen (:ntim)
Reported 2021-10-06 13:21:01 PDT
This test mostly tests onfocus and <object>, we don't need it to use autofocus. The new autofocus timing will be async so we don't want this test to break with the new autofocus behavior implemented.
Attachments
Patch (1.83 KB, patch)
2021-10-06 13:25 PDT, Tim Nguyen (:ntim)
cdumez: review+
[fast-cq] Patch (1.73 KB, patch)
2021-10-06 13:39 PDT, Tim Nguyen (:ntim)
no flags
Tim Nguyen (:ntim)
Comment 1 2021-10-06 13:21:42 PDT
See also bug 231219 for similar changes in other tests.
Tim Nguyen (:ntim)
Comment 2 2021-10-06 13:25:04 PDT
Chris Dumez
Comment 3 2021-10-06 13:29:07 PDT
Comment on attachment 440428 [details] Patch This seems to completely disable the test. This test was introduced to cover a code change in HTMLObjectElement/HTMLPlugInImageElement and it is no longer covered since you're dropping the object element..
Chris Dumez
Comment 4 2021-10-06 13:35:28 PDT
Comment on attachment 440428 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=440428&action=review > LayoutTests/plugins/object-onfocus-mutation-crash.html:4 > +<object data="a"> Oh, I misread. You merely moved the <object> > LayoutTests/plugins/object-onfocus-mutation-crash.html:22 > + finish(); Any reason we can't keep calling finish asynchronously?
Tim Nguyen (:ntim)
Comment 5 2021-10-06 13:36:37 PDT
(In reply to Chris Dumez from comment #4) > Comment on attachment 440428 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=440428&action=review > > > LayoutTests/plugins/object-onfocus-mutation-crash.html:4 > > +<object data="a"> > > Oh, I misread. You merely moved the <object> > > > LayoutTests/plugins/object-onfocus-mutation-crash.html:22 > > + finish(); > > Any reason we can't keep calling finish asynchronously? No reason, just looks cleaner without the setTimeout, but I can add it back.
Tim Nguyen (:ntim)
Comment 6 2021-10-06 13:39:48 PDT
Created attachment 440434 [details] [fast-cq] Patch
Tim Nguyen (:ntim)
Comment 7 2021-10-06 13:47:25 PDT
Radar WebKit Bug Importer
Comment 8 2021-10-06 13:48:22 PDT
Note You need to log in before you can comment on or make changes to this bug.