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