Revert part of r244141
Created attachment 374313 [details] Patch
rdar://problem/50060561
Comment on attachment 374313 [details] Patch This needs a changelog with a minimum of 500 words.
Created attachment 374322 [details] Patch
Comment on attachment 374322 [details] Patch This is a so much better change
Comment on attachment 374322 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=374322&action=review Yes, much, much better, thanks for looking into this! > LayoutTests/ChangeLog:5 > + Radar?
Comment on attachment 374322 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=374322&action=review > LayoutTests/fast/forms/ios/delete-in-input-in-iframe-expected.txt:3 > +PASS: page scrolled to 1143 Do we want to consider just testing to see if the page scrolled at all, rather than to a specific number? This seems prone to regressions still.
Comment on attachment 374322 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=374322&action=review >> LayoutTests/fast/forms/ios/delete-in-input-in-iframe-expected.txt:3 >> +PASS: page scrolled to 1143 > > Do we want to consider just testing to see if the page scrolled at all, rather than to a specific number? This seems prone to regressions still. I thought this was better so we can verify that it scrolled the expected amount. If this becomes flaky or we change the scrolling amount too much we can change the results later
Created attachment 374398 [details] Patch
Comment on attachment 374398 [details] Patch Clearing flags on attachment: 374398 Committed r247571: <https://trac.webkit.org/changeset/247571>
All reviewed patches have been landed. Closing bug.
*** Bug 199920 has been marked as a duplicate of this bug. ***
Reopening to attach new patch.
Created attachment 374422 [details] Patch
Comment on attachment 374422 [details] Patch Clearing flags on attachment: 374422 Committed r247648: <https://trac.webkit.org/changeset/247648>