Title: [221813] trunk/Source/WebCore
Revision
221813
Author
[email protected]
Date
2017-09-09 02:35:33 -0700 (Sat, 09 Sep 2017)

Log Message

Use RenderLayerBacking::renderer() when possible
https://bugs.webkit.org/show_bug.cgi?id=176585

The private member renderer() is a shorthand for m_owningLayer.renderer(). This patch uses
it in RenderLayerBacking when possible.

Patch by Frederic Wang <[email protected]> on 2017-09-09
Reviewed by Carlos Garcia Campos.

No new tests, behavior not changed.

* rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::updateGeometry):
(WebCore::RenderLayerBacking::setContentsNeedDisplay):
(WebCore::RenderLayerBacking::setContentsNeedDisplayInRect):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (221812 => 221813)


--- trunk/Source/WebCore/ChangeLog	2017-09-09 07:46:30 UTC (rev 221812)
+++ trunk/Source/WebCore/ChangeLog	2017-09-09 09:35:33 UTC (rev 221813)
@@ -1,3 +1,20 @@
+2017-09-09  Frederic Wang  <[email protected]>
+
+        Use RenderLayerBacking::renderer() when possible
+        https://bugs.webkit.org/show_bug.cgi?id=176585
+
+        The private member renderer() is a shorthand for m_owningLayer.renderer(). This patch uses
+        it in RenderLayerBacking when possible.
+
+        Reviewed by Carlos Garcia Campos.
+
+        No new tests, behavior not changed.
+
+        * rendering/RenderLayerBacking.cpp:
+        (WebCore::RenderLayerBacking::updateGeometry):
+        (WebCore::RenderLayerBacking::setContentsNeedDisplay):
+        (WebCore::RenderLayerBacking::setContentsNeedDisplayInRect):
+
 2017-09-08  Sam Weinig  <[email protected]>
 
         Replace JS builtin implementation of the FetchResponse constructor with a C++ one

Modified: trunk/Source/WebCore/rendering/RenderLayerBacking.cpp (221812 => 221813)


--- trunk/Source/WebCore/rendering/RenderLayerBacking.cpp	2017-09-09 07:46:30 UTC (rev 221812)
+++ trunk/Source/WebCore/rendering/RenderLayerBacking.cpp	2017-09-09 09:35:33 UTC (rev 221813)
@@ -1052,7 +1052,7 @@
     if (m_maskLayer)
         updateMaskingLayerGeometry();
     
-    if (m_owningLayer.renderer().hasTransformRelatedProperty()) {
+    if (renderer().hasTransformRelatedProperty()) {
         // Update properties that depend on layer dimensions.
         FloatPoint3D transformOrigin = computeTransformOriginForPainting(downcast<RenderBox>(renderer()).borderBoxRect());
         FloatPoint layerOffset = roundPointToDevicePixels(toLayoutPoint(rendererOffset.fromParentGraphicsLayer()), deviceScaleFactor());
@@ -2433,7 +2433,7 @@
 {
     ASSERT(!paintsIntoCompositedAncestor());
 
-    FrameView& frameView = owningLayer().renderer().view().frameView();
+    FrameView& frameView = renderer().view().frameView();
     if (m_isMainFrameRenderViewLayer && frameView.isTrackingRepaints())
         frameView.addTrackedRepaintRect(owningLayer().absoluteBoundingBoxForPainting());
     
@@ -2468,7 +2468,7 @@
     ASSERT(!paintsIntoCompositedAncestor());
 
     FloatRect pixelSnappedRectForPainting = snapRectToDevicePixels(r, deviceScaleFactor());
-    FrameView& frameView = owningLayer().renderer().view().frameView();
+    FrameView& frameView = renderer().view().frameView();
     if (m_isMainFrameRenderViewLayer && frameView.isTrackingRepaints())
         frameView.addTrackedRepaintRect(pixelSnappedRectForPainting);
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to