Diff
Modified: branches/safari-608.1.27-branch/LayoutTests/ChangeLog (245984 => 245985)
--- branches/safari-608.1.27-branch/LayoutTests/ChangeLog 2019-05-31 21:11:32 UTC (rev 245984)
+++ branches/safari-608.1.27-branch/LayoutTests/ChangeLog 2019-05-31 21:31:57 UTC (rev 245985)
@@ -1,5 +1,9 @@
2019-05-31 Kocsen Chung <[email protected]>
+ Revert r245953. rdar://problem/51314019
+
+2019-05-31 Kocsen Chung <[email protected]>
+
Revert r245946. rdar://problem/47390407
2019-05-30 Ryosuke Niwa <[email protected]>
Deleted: branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-keyboard-expected.txt (245984 => 245985)
--- branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-keyboard-expected.txt 2019-05-31 21:11:32 UTC (rev 245984)
+++ branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-keyboard-expected.txt 2019-05-31 21:31:57 UTC (rev 245985)
@@ -1,4 +0,0 @@
-This document shouldn't be scrollable normally but should be scrollable when the software keyboard is shown.
-To manually test, tap on the text field below to bring up the docked software keyboard.
-The document should become scrollable.
-PASS - the document did scroll
Deleted: branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-keyboard.html (245984 => 245985)
--- branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-keyboard.html 2019-05-31 21:11:32 UTC (rev 245984)
+++ branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-keyboard.html 2019-05-31 21:31:57 UTC (rev 245985)
@@ -1,50 +0,0 @@
-<!DOCTYPE html> <!-- webkit-test-runner [ useFlexibleViewport=true internal:AsyncOverflowScrollingEnabled=true internal:AsyncFrameScrollingEnabled=true ] -->
-<html>
-<head>
-<meta name="viewport" content="width=device-width, initial-scale=1, user-scalable=no">
-<script src=""
-<script src=""
-<style>
-html, body { width: 100%; height: 100%; margin: 0px; padding: 0px; }
-body { overflow: hidden; }
-#content { width: 100%; height: 100%; box-sizing: border-box; padding: 20px; background: #ccc; }
-</style>
-<script>
-if (window.testRunner) {
- testRunner.waitUntilDone();
- testRunner.dumpAsText();
-}
-
-async function runTest() {
- if (!window.testRunner)
- return;
-
- await UIHelper.setHardwareKeyboardAttached(false);
-
- // FIXME: <rdar://problem/51289800> Scrolling down by touch gestures does not work immediately after software keyboard is brought up for the first time
- await UIHelper.activateElementAndWaitForInputSession(document.querySelector('input'));
- document.activeElement.blur();
- await UIHelper.waitForKeyboardToHide();
-
- await UIHelper.activateElementAndWaitForInputSession(document.querySelector('input'));
-
- const result = document.getElementById('result');
- const y = result.getBoundingClientRect().top + 10;
-
- await touchAndDragFromPointToPoint(200, y + 100, 200, y + 10);
- await liftUpAtPoint(200, y + 10);
- await UIHelper.delayFor(100);
-
- result.textContent = document.documentElement.scrollTop >= 50 ? 'PASS - the document did scroll' : 'FAIL - the document did not scroll';
-
- testRunner.notifyDone();
-}
-
-</script>
-<body _onload_="runTest()"><div id="content">
-This document shouldn't be scrollable normally but should be scrollable when the software keyboard is shown.<br>
-To manually test, tap on the text field below to bring up the docked software keyboard.<br>
-The document should become scrollable.<br>
-<div id="result"><input placeholder="Click here"></div>
-</div></body>
-</html>
Deleted: branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-1-expected.txt (245984 => 245985)
--- branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-1-expected.txt 2019-05-31 21:11:32 UTC (rev 245984)
+++ branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-1-expected.txt 2019-05-31 21:31:57 UTC (rev 245985)
@@ -1,5 +0,0 @@
-This document shouldn't be scrollable normally but should be scrollable when pinch zoomed.
-To manually test, pinch zoom on the page. The document should become scrollable.
-PASS - the document did scroll with zooming
-PASS - the document scrolled back to the top
-
Deleted: branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-1.html (245984 => 245985)
--- branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-1.html 2019-05-31 21:11:32 UTC (rev 245984)
+++ branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-1.html 2019-05-31 21:31:57 UTC (rev 245985)
@@ -1,48 +0,0 @@
-<!DOCTYPE html> <!-- webkit-test-runner [ useFlexibleViewport=true internal:AsyncOverflowScrollingEnabled=true internal:AsyncFrameScrollingEnabled=true ] -->
-<html>
-<head>
-<meta name="viewport" content="width=device-width, initial-scale=1">
-<script src=""
-<script src=""
-<style>
-html, body { width: 100%; height: 100%; margin: 0px; padding: 0px; }
-body { overflow: hidden; }
-#content { width: 100%; height: 100%; box-sizing: border-box; padding: 20px; background: #ccc; }
-</style>
-<script>
-if (window.testRunner) {
- testRunner.waitUntilDone();
- testRunner.dumpAsText();
-}
-
-async function runTest() {
- if (!window.testRunner)
- return;
-
- let result = '';
- const log = (text) => result += text + '\n';
-
- await UIHelper.zoomToScale(1.5);
-
- log(visualViewport.pageTop > 50 ? 'PASS - the document did scroll with zooming' : `FAIL - the document did not scroll with zooming: ${visualViewport.pageTop}`);
-
- await touchAndDragFromPointToPoint(200, 100, 200, 200);
- await liftUpAtPoint(200, 200);
- await UIHelper.delayFor(200);
-
- log(visualViewport.pageTop <= 0 ? 'PASS - the document scrolled back to the top' : `FAIL - the document did not scroll back to the top: ${visualViewport.pageTop}`);
-
- document.getElementById('log').textContent = result;
-
- testRunner.notifyDone();
-}
-
-</script>
-<body _onload_="runTest()">
-<div id="content">
-This document shouldn't be scrollable normally but should be scrollable when pinch zoomed.<br>
-To manually test, pinch zoom on the page. The document should become scrollable.
-</div>
-<pre id="log"></pre>
-</body>
-</html>
Deleted: branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-2-expected.txt (245984 => 245985)
--- branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-2-expected.txt 2019-05-31 21:11:32 UTC (rev 245984)
+++ branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-2-expected.txt 2019-05-31 21:31:57 UTC (rev 245985)
@@ -1,5 +0,0 @@
-This document shouldn't be scrollable normally but should be scrollable when pinch zoomed.
-To manually test, pinch zoom on the page. The document should become scrollable.
-PASS - the document did scroll with zooming
-PASS - the document scrolled back to the top
-
Deleted: branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-2.html (245984 => 245985)
--- branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-2.html 2019-05-31 21:11:32 UTC (rev 245984)
+++ branches/safari-608.1.27-branch/LayoutTests/fast/scrolling/ios/body-overflow-hidden-height-100-percent-zoomed-2.html 2019-05-31 21:31:57 UTC (rev 245985)
@@ -1,53 +0,0 @@
-<!DOCTYPE html> <!-- webkit-test-runner [ useFlexibleViewport=true internal:AsyncOverflowScrollingEnabled=true internal:AsyncFrameScrollingEnabled=true ] -->
-<html>
-<head>
-<script src=""
-<script src=""
-<style>
-html, body { width: 100%; height: 100%; margin: 0px; padding: 0px; }
-body { overflow: hidden; }
-#content { width: 100%; height: 100%; box-sizing: border-box; padding: 20px; background: #ccc; }
-</style>
-<script>
-if (window.testRunner) {
- testRunner.waitUntilDone();
- testRunner.dumpAsText();
-}
-
-async function runTest() {
- if (!window.testRunner)
- return;
-
- let result = '';
- const log = (text) => result += text + '\n';
-
- const initialScale = await UIHelper.zoomScale();
- await UIHelper.zoomToScale(initialScale * 1.5);
- const finalScale = await UIHelper.zoomScale();
-
- log(visualViewport.pageTop > 100 ? 'PASS - the document did scroll with zooming' : `FAIL - the document did not scroll with zooming: ${visualViewport.pageTop}`);
-
- const x = visualViewport.pageLeft;
- const y = visualViewport.pageTop;
-
- await touchAndDragFromPointToPoint(x + 100, y + 100, x + 100, y + 200);
- await liftUpAtPoint(x + 100, y + 200);
- await UIHelper.delayFor(200);
-
- const expectedY = y - 80 * finalScale;
- log(visualViewport.pageTop < expectedY ? 'PASS - the document scrolled back to the top' : `FAIL - the document did not scroll back to the top: expected ${visualViewport.pageTop} < ${expectedY}`);
-
- document.getElementById('log').textContent = result;
-
- testRunner.notifyDone();
-}
-
-</script>
-<body _onload_="runTest()">
-<div id="content">
-This document shouldn't be scrollable normally but should be scrollable when pinch zoomed.<br>
-To manually test, pinch zoom on the page. The document should become scrollable.
-</div>
-<pre id="log"></pre>
-</body>
-</html>
Modified: branches/safari-608.1.27-branch/Source/WebKit/ChangeLog (245984 => 245985)
--- branches/safari-608.1.27-branch/Source/WebKit/ChangeLog 2019-05-31 21:11:32 UTC (rev 245984)
+++ branches/safari-608.1.27-branch/Source/WebKit/ChangeLog 2019-05-31 21:31:57 UTC (rev 245985)
@@ -1,5 +1,9 @@
2019-05-31 Kocsen Chung <[email protected]>
+ Revert r245953. rdar://problem/51314019
+
+2019-05-31 Kocsen Chung <[email protected]>
+
Revert r245946. rdar://problem/47390407
2019-05-31 Carlos Garcia Campos <[email protected]>
Modified: branches/safari-608.1.27-branch/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm (245984 => 245985)
--- branches/safari-608.1.27-branch/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm 2019-05-31 21:11:32 UTC (rev 245984)
+++ branches/safari-608.1.27-branch/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm 2019-05-31 21:31:57 UTC (rev 245985)
@@ -1988,9 +1988,7 @@
[_scrollView setMaximumZoomScale:layerTreeTransaction.maximumScaleFactor()];
[_scrollView setZoomEnabled:layerTreeTransaction.allowsUserScaling()];
#if ENABLE(ASYNC_SCROLLING)
- bool hasDockedInputView = !CGRectIsEmpty(_inputViewBounds);
- bool isZoomed = layerTreeTransaction.pageScaleFactor() > layerTreeTransaction.initialScaleFactor();
- [_scrollView setScrollEnabled:_page->scrollingCoordinatorProxy()->hasScrollableMainFrame() || hasDockedInputView || isZoomed];
+ [_scrollView setScrollEnabled:_page->scrollingCoordinatorProxy()->hasScrollableMainFrame()];
#endif
if (!layerTreeTransaction.scaleWasSetByUIProcess() && ![_scrollView isZooming] && ![_scrollView isZoomBouncing] && ![_scrollView _isAnimatingZoom] && [_scrollView zoomScale] != layerTreeTransaction.pageScaleFactor()) {
LOG_WITH_STREAM(VisibleRects, stream << " updating scroll view with pageScaleFactor " << layerTreeTransaction.pageScaleFactor());
