Title: [144710] trunk/Source/WebCore
Revision
144710
Author
commit-qu...@webkit.org
Date
2013-03-04 19:23:35 -0800 (Mon, 04 Mar 2013)

Log Message

Fix some style issues in RenderLayerCompositor
https://bugs.webkit.org/show_bug.cgi?id=111369

Patch by Min Qin <qin...@chromium.org> on 2013-03-04
Reviewed by Simon Fraser.

Just style fix, no test needed

* rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::frameViewDidScroll):
(WebCore::RenderLayerCompositor::requiresCompositingForPosition):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (144709 => 144710)


--- trunk/Source/WebCore/ChangeLog	2013-03-05 03:17:34 UTC (rev 144709)
+++ trunk/Source/WebCore/ChangeLog	2013-03-05 03:23:35 UTC (rev 144710)
@@ -1,3 +1,16 @@
+2013-03-04  Min Qin  <qin...@chromium.org>
+
+        Fix some style issues in RenderLayerCompositor
+        https://bugs.webkit.org/show_bug.cgi?id=111369
+
+        Reviewed by Simon Fraser.
+
+        Just style fix, no test needed
+
+        * rendering/RenderLayerCompositor.cpp:
+        (WebCore::RenderLayerCompositor::frameViewDidScroll):
+        (WebCore::RenderLayerCompositor::requiresCompositingForPosition):
+
 2013-03-04  Terry Anderson  <tdander...@chromium.org>
 
         Revert changes to RenderLayer::scrollByRecursively() that were added for gesture-scrolling

Modified: trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp (144709 => 144710)


--- trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp	2013-03-05 03:17:34 UTC (rev 144709)
+++ trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp	2013-03-05 03:23:35 UTC (rev 144710)
@@ -1220,9 +1220,10 @@
     if (ScrollingCoordinator* scrollingCoordinator = this->scrollingCoordinator()) {
         if (scrollingCoordinator->coordinatesScrollingForFrameView(frameView))
             return;
-        if (Settings* settings = m_renderView->document()->settings())
+        if (Settings* settings = m_renderView->document()->settings()) {
             if (settings->compositedScrollingForFramesEnabled())
                 scrollingCoordinator->scrollableAreaScrollLayerDidChange(frameView);
+        }
     }
 
     m_scrollLayer->setPosition(FloatPoint(-scrollPosition.x(), -scrollPosition.y()));
@@ -2148,9 +2149,10 @@
         return false;
 
     // FIXME: acceleratedCompositingForFixedPositionEnabled should probably be renamed acceleratedCompositingForViewportConstrainedPositionEnabled().
-    if (Settings* settings = m_renderView->document()->settings())
+    if (Settings* settings = m_renderView->document()->settings()) {
         if (!settings->acceleratedCompositingForFixedPositionEnabled())
             return false;
+    }
 
     if (isSticky)
         return true;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to