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.
See also bug 231219 for similar changes in other tests.
Created attachment 440428 [details] Patch
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..
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?
(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.
Created attachment 440434 [details] [fast-cq] Patch
Committed r283657 (242602@main): <https://commits.webkit.org/242602@main>
<rdar://problem/83950034>