Do not adjust viewport if editing selection is already visible
Created attachment 376733 [details] Patch
Comment on attachment 376733 [details] Patch Attachment 376733 [details] did not pass mac-ews (mac): Output: https://webkit-queues.webkit.org/results/12944127 New failing tests: fast/events/autoscroll-input-when-very-zoomed.html
Created attachment 376739 [details] Archive of layout-test-results from ews100 for mac-highsierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews100 Port: mac-highsierra Platform: Mac OS X 10.13.6
Comment on attachment 376733 [details] Patch Attachment 376733 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/12944206 New failing tests: fast/events/autoscroll-input-when-very-zoomed.html
Created attachment 376741 [details] Archive of layout-test-results from ews115 for mac-highsierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews115 Port: mac-highsierra Platform: Mac OS X 10.13.6
Comment on attachment 376733 [details] Patch Attachment 376733 [details] did not pass win-ews (win): Output: https://webkit-queues.webkit.org/results/12944300 New failing tests: fast/events/autoscroll-input-when-very-zoomed.html
Created attachment 376742 [details] Archive of layout-test-results from ews214 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews214 Port: win-future Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Created attachment 376822 [details] Patch
Created attachment 376908 [details] Patch
Created attachment 376924 [details] Patch
Comment on attachment 376924 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=376924&action=review > Source/WebCore/ChangeLog:11 > + Currently due to scrolling being mostly handled by integers, we are getting "scrolling being mostly handled by integers" -> scrollPositions being integral > Source/WebCore/ChangeLog:14 > + start dealing with scrolling with floats/doubles, but until such time, to make scrollPositions be LayoutUnits or floating point > Source/WebCore/rendering/RenderLayer.cpp:2704 > + // This will likely be unnecessary once we convert scrolling to work with floats/doubles instead of ints I would change this comment to "Avoid scrolling to the rounded value of revealRect.location() if we don't actually need to scroll" > LayoutTests/fast/scrolling/ios/autoscroll-input-when-very-zoomed.html:25 > + Remove the blank line > LayoutTests/fast/scrolling/ios/autoscroll-input-when-very-zoomed.html:28 > + await UIHelper.delayFor(200); What's the reason for the delay? > LayoutTests/fast/scrolling/ios/autoscroll-input-when-very-zoomed.html:38 > + // Everything isn't quite set yet if we don't delay That's a bit vague. What are we waiting for? > LayoutTests/fast/scrolling/ios/autoscroll-input-when-very-zoomed.html:42 > + > + Two blank lines. > LayoutTests/fast/scrolling/ios/autoscroll-input-when-very-zoomed.html:52 > + output += 'FAIL: page has scrolled on the secont input'; secont > LayoutTests/fast/scrolling/ios/autoscroll-input-when-very-zoomed.html:58 > + await UIHelper.immediateZoomToScale(1.0); > + await UIHelper.immediateScrollTo(0, 0); These shouldn't be necessary. > LayoutTests/fast/scrolling/ios/autoscroll-input-when-very-zoomed.html:81 > +This test focuses a form, them zoomes and scrolls the page. zoomes > LayoutTests/fast/scrolling/ios/autoscroll-input-when-very-zoomed.html:85 > +<div id="result"></div> This seems unused
Created attachment 376947 [details] Patch for landing
Comment on attachment 376947 [details] Patch for landing Clearing flags on attachment: 376947 Committed r248977: <https://trac.webkit.org/changeset/248977>
All reviewed patches have been landed. Closing bug.
<rdar://problem/54579095>