Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(152)

Unified Diff: third_party/WebKit/Source/web/tests/WebFrameTest.cpp

Issue 2658253003: [RootLayerScrolls] Fix ScrollingCoordinatorTest (Closed)
Patch Set: Remove unused function Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/web/tests/WebFrameTest.cpp
diff --git a/third_party/WebKit/Source/web/tests/WebFrameTest.cpp b/third_party/WebKit/Source/web/tests/WebFrameTest.cpp
index d123a03ceffb17247de312433aaba890b5e7b4b9..bc04711adada3b2d8e4d839052020552031e1c95 100644
--- a/third_party/WebKit/Source/web/tests/WebFrameTest.cpp
+++ b/third_party/WebKit/Source/web/tests/WebFrameTest.cpp
@@ -7761,6 +7761,16 @@ TEST_P(ParameterizedWebFrameTest, FullscreenMainFrame) {
webViewHelper.resize(WebSize(viewportWidth, viewportHeight));
webViewImpl->updateAllLifecyclePhases();
+ WebLayer* webScrollLayer = webViewImpl->mainFrameImpl()
+ ->frame()
+ ->view()
+ ->layoutViewportScrollableArea()
+ ->layerForScrolling()
+ ->platformLayer();
+ ASSERT_TRUE(webScrollLayer->scrollable());
+ ASSERT_TRUE(webScrollLayer->userScrollableHorizontal());
+ ASSERT_TRUE(webScrollLayer->userScrollableVertical());
+
Document* document = webViewImpl->mainFrameImpl()->frame()->document();
UserGestureIndicator gesture(DocumentUserGestureToken::create(document));
Fullscreen::requestFullscreen(*document->documentElement());
@@ -7780,8 +7790,12 @@ TEST_P(ParameterizedWebFrameTest, FullscreenMainFrame) {
Fullscreen::fullscreenElementFrom(*document));
// Verify that the main frame is still scrollable.
- WebLayer* webScrollLayer =
- webViewImpl->compositor()->scrollLayer()->platformLayer();
+ webScrollLayer = webViewImpl->mainFrameImpl()
+ ->frame()
+ ->view()
+ ->layoutViewportScrollableArea()
+ ->layerForScrolling()
+ ->platformLayer();
ASSERT_TRUE(webScrollLayer->scrollable());
ASSERT_TRUE(webScrollLayer->userScrollableHorizontal());
ASSERT_TRUE(webScrollLayer->userScrollableVertical());

Powered by Google App Engine
This is Rietveld 408576698