| Index: Source/web/tests/WebFrameTest.cpp
|
| diff --git a/Source/web/tests/WebFrameTest.cpp b/Source/web/tests/WebFrameTest.cpp
|
| index d174a6ec5bcff68719b4a7c71f89ddc20f2f823b..a7b2ee7ae86eab302d80e932e49cc8c7b37c89b4 100644
|
| --- a/Source/web/tests/WebFrameTest.cpp
|
| +++ b/Source/web/tests/WebFrameTest.cpp
|
| @@ -2372,7 +2372,7 @@ void simulatePageScale(WebViewImpl* webViewImpl, float& scale)
|
| {
|
| blink::IntSize scrollDelta = webViewImpl->fakePageScaleAnimationTargetPositionForTesting() - webViewImpl->mainFrameImpl()->frameView()->scrollPosition();
|
| float scaleDelta = webViewImpl->fakePageScaleAnimationPageScaleForTesting() / webViewImpl->pageScaleFactor();
|
| - webViewImpl->applyScrollAndScale(scrollDelta, scaleDelta);
|
| + webViewImpl->applyViewportDeltas(scrollDelta, scaleDelta, 0);
|
| scale = webViewImpl->pageScaleFactor();
|
| }
|
|
|
| @@ -2484,14 +2484,14 @@ TEST_F(WebFrameTest, DivAutoZoomMultipleDivsTest)
|
| // If the user pinch zooms after double tap, a second double tap should zoom back to the div.
|
| simulateDoubleTap(webViewHelper.webViewImpl(), topPoint, scale);
|
| EXPECT_FLOAT_EQ(1, scale);
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(), 0.6f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(), 0.6f, 0);
|
| simulateDoubleTap(webViewHelper.webViewImpl(), bottomPoint, scale);
|
| EXPECT_FLOAT_EQ(1, scale);
|
| simulateDoubleTap(webViewHelper.webViewImpl(), bottomPoint, scale);
|
| EXPECT_FLOAT_EQ(webViewHelper.webViewImpl()->minimumPageScaleFactor(), scale);
|
|
|
| // If we didn't yet get an auto-zoom update and a second double-tap arrives, should go back to minimum scale.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(), 1.1f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(), 1.1f, 0);
|
| webViewHelper.webViewImpl()->animateDoubleTapZoom(topPoint);
|
| EXPECT_TRUE(webViewHelper.webViewImpl()->fakeDoubleTapAnimationPendingForTesting());
|
| simulateDoubleTap(webViewHelper.webViewImpl(), bottomPoint, scale);
|
| @@ -2531,7 +2531,7 @@ TEST_F(WebFrameTest, DivAutoZoomScaleBoundsTest)
|
| EXPECT_FLOAT_EQ(1, scale);
|
|
|
| // Zoom in to reset double_tap_zoom_in_effect flag.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(), 1.1f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(), 1.1f, 0);
|
| // 1 < minimumPageScale < doubleTapZoomAlreadyLegibleScale
|
| webViewHelper.webView()->setPageScaleFactorLimits(1.1f, 4);
|
| webViewHelper.webView()->layout();
|
| @@ -2545,7 +2545,7 @@ TEST_F(WebFrameTest, DivAutoZoomScaleBoundsTest)
|
| EXPECT_FLOAT_EQ(doubleTapZoomAlreadyLegibleScale, scale);
|
|
|
| // Zoom in to reset double_tap_zoom_in_effect flag.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(), 1.1f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(), 1.1f, 0);
|
| // minimumPageScale < 1 < doubleTapZoomAlreadyLegibleScale
|
| webViewHelper.webView()->setPageScaleFactorLimits(0.95f, 4);
|
| webViewHelper.webView()->layout();
|
| @@ -2595,7 +2595,7 @@ TEST_F(WebFrameTest, DivAutoZoomScaleFontScaleFactorTest)
|
| EXPECT_FLOAT_EQ(legibleScale, scale);
|
|
|
| // Zoom in to reset double_tap_zoom_in_effect flag.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(), 1.1f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(), 1.1f, 0);
|
| // 1 < accessibilityFontScaleFactor < minimumPageScale < doubleTapZoomAlreadyLegibleScale
|
| webViewHelper.webView()->setPageScaleFactorLimits(1.0f, 4);
|
| webViewHelper.webView()->layout();
|
| @@ -2609,7 +2609,7 @@ TEST_F(WebFrameTest, DivAutoZoomScaleFontScaleFactorTest)
|
| EXPECT_FLOAT_EQ(doubleTapZoomAlreadyLegibleScale, scale);
|
|
|
| // Zoom in to reset double_tap_zoom_in_effect flag.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(), 1.1f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(), 1.1f, 0);
|
| // minimumPageScale < 1 < accessibilityFontScaleFactor < doubleTapZoomAlreadyLegibleScale
|
| webViewHelper.webView()->setPageScaleFactorLimits(0.95f, 4);
|
| webViewHelper.webView()->layout();
|
| @@ -2623,7 +2623,7 @@ TEST_F(WebFrameTest, DivAutoZoomScaleFontScaleFactorTest)
|
| EXPECT_FLOAT_EQ(doubleTapZoomAlreadyLegibleScale, scale);
|
|
|
| // Zoom in to reset double_tap_zoom_in_effect flag.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(), 1.1f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(), 1.1f, 0);
|
| // minimumPageScale < 1 < doubleTapZoomAlreadyLegibleScale < accessibilityFontScaleFactor
|
| webViewHelper.webView()->setPageScaleFactorLimits(0.9f, 4);
|
| webViewHelper.webView()->layout();
|
| @@ -4968,7 +4968,7 @@ TEST_F(WebFrameTest, CompositorScrollIsUserScrollLongPage)
|
| EXPECT_FALSE(client.wasProgrammaticScroll());
|
|
|
| // Do a compositor scroll, verify that this is counted as a user scroll.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(0, 1), 1.1f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(0, 1), 1.1f, 0);
|
| EXPECT_TRUE(client.wasUserScroll());
|
| client.reset();
|
|
|
| @@ -4976,18 +4976,18 @@ TEST_F(WebFrameTest, CompositorScrollIsUserScrollLongPage)
|
| EXPECT_FALSE(client.wasProgrammaticScroll());
|
|
|
| // The page scale 1.0f and scroll.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(0, 1), 1.0f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(0, 1), 1.0f, 0);
|
| EXPECT_TRUE(client.wasUserScroll());
|
| client.reset();
|
|
|
| // No scroll event if there is no scroll delta.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(), 1.0f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(), 1.0f, 0);
|
| EXPECT_FALSE(client.wasUserScroll());
|
| EXPECT_FALSE(client.wasProgrammaticScroll());
|
| client.reset();
|
|
|
| // Non zero page scale and scroll.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(9, 13), 0.6f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(9, 13), 0.6f, 0);
|
| EXPECT_TRUE(client.wasUserScroll());
|
| client.reset();
|
|
|
| @@ -5022,7 +5022,7 @@ TEST_F(WebFrameTest, CompositorScrollIsUserScrollShortPage)
|
| EXPECT_FALSE(client.wasProgrammaticScroll());
|
|
|
| // Non zero page scale and scroll.
|
| - webViewHelper.webViewImpl()->applyScrollAndScale(WebSize(9, 13), 2.0f);
|
| + webViewHelper.webViewImpl()->applyViewportDeltas(WebSize(9, 13), 2.0f, 0);
|
| EXPECT_FALSE(client.wasProgrammaticScroll());
|
| EXPECT_TRUE(client.wasUserScroll());
|
| client.reset();
|
|
|