Changeset 144710 in webkit

Timestamp:

Mar 4, 2013, 7:23:35 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

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

Patch by Min Qin <​qinmin@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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderLayerCompositor.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-03-04  Min Qin  <qinmin@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  <tdanderson@chromium.org>
 

trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp

@@ -1221 +1221 @@
         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);
+        }
     }
 
@@ -2149 +2150 @@
 
     // 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)