Changeset 150798 in webkit

Timestamp:

May 28, 2013 6:07:36 AM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[Qt] REGRESSION(r149292): broke scrolling in Qt WK2 desktop mode
​https://bugs.webkit.org/show_bug.cgi?id=116597

ScrollingCoordinatorCoordinatedGraphics is tailored specifically for
fixedLayout mode. When fixedLayout is not set, it should be turned off.

Patch by Noam Rosenthal <Noam Rosenthal> on 2013-05-28
Reviewed by Allan Sandfeld Jensen.

• WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp:

(WebKit::CoordinatedLayerTreeHost::CoordinatedLayerTreeHost):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2013-05-28  Noam Rosenthal  <noam@webkit.org>
+
+        [Qt] REGRESSION(r149292): broke scrolling in Qt WK2 desktop mode
+        https://bugs.webkit.org/show_bug.cgi?id=116597
+
+        ScrollingCoordinatorCoordinatedGraphics is tailored specifically for
+        fixedLayout mode. When fixedLayout is not set, it should be turned off.
+
+        Reviewed by Allan Sandfeld Jensen.
+
+        * WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp:
+        (WebKit::CoordinatedLayerTreeHost::CoordinatedLayerTreeHost):
+
 2013-05-28  Andreas Kling  <akling@apple.com>
 

trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp

@@ -100 +100 @@
 #endif
 {
-    m_webPage->corePage()->settings()->setScrollingCoordinatorEnabled(true);
+    if (m_webPage->useFixedLayout())
+        m_webPage->corePage()->settings()->setScrollingCoordinatorEnabled(true);
     m_webPage->corePage()->settings()->setApplyDeviceScaleFactorInCompositor(true);