Changeset 139032 in webkit

Timestamp:

Jan 7, 2013 10:07:55 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

Use explicit constructor to convert from Int/FloatSize to Int/FloatPoint
​https://bugs.webkit.org/show_bug.cgi?id=106209

Patch by Steve Block <​steveblock@chromium.org> on 2013-01-07
Reviewed by Kentaro Hara.

Refactoring only, no functional change.

• rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderLayerBacking.cpp (modified) (5 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-01-07  Steve Block  <steveblock@chromium.org>
+
+        Use explicit constructor to convert from Int/FloatSize to Int/FloatPoint
+        https://bugs.webkit.org/show_bug.cgi?id=106209
+
+        Reviewed by Kentaro Hara.
+
+        Refactoring only, no functional change.
+
+        * rendering/RenderLayerBacking.cpp:
+        (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
+
 2013-01-07  Viatcheslav Ostapenko  <sl.ostapenko@samsung.com>
 

trunk/Source/WebCore/rendering/RenderLayerBacking.cpp

@@ -625 +625 @@
         IntRect parentClipRect = pixelSnappedIntRect(m_owningLayer->backgroundClipRect(clipRectsContext).rect()); // FIXME: Incorrect for CSS regions.
         ASSERT(parentClipRect != PaintInfo::infiniteRect());
-        m_ancestorClippingLayer->setPosition(FloatPoint() + (parentClipRect.location() - graphicsLayerParentLocation));
+        m_ancestorClippingLayer->setPosition(FloatPoint(parentClipRect.location() - graphicsLayerParentLocation));
         m_ancestorClippingLayer->setSize(parentClipRect.size());
 
@@ -635 +635 @@
     }
 
-    m_graphicsLayer->setPosition(FloatPoint() + (relativeCompositingBounds.location() - graphicsLayerParentLocation));
+    m_graphicsLayer->setPosition(FloatPoint(relativeCompositingBounds.location() - graphicsLayerParentLocation));
     m_graphicsLayer->setOffsetFromRenderer(localCompositingBounds.location() - IntPoint());
     
@@ -653 +653 @@
     if (GraphicsLayer* clipLayer = clippingLayer()) {
         clippingBox = clipBox(toRenderBox(renderer()));
-        clipLayer->setPosition(FloatPoint() + (clippingBox.location() - localCompositingBounds.location()));
+        clipLayer->setPosition(FloatPoint(clippingBox.location() - localCompositingBounds.location()));
         clipLayer->setSize(clippingBox.size());
         clipLayer->setOffsetFromRenderer(clippingBox.location() - IntPoint());
@@ -729 +729 @@
         FloatRect layerBounds = compositedBounds();
         FloatRect reflectionLayerBounds = reflectionBacking->compositedBounds();
-        reflectionBacking->graphicsLayer()->setReplicatedLayerPosition(FloatPoint() + (layerBounds.location() - reflectionLayerBounds.location()));
+        reflectionBacking->graphicsLayer()->setReplicatedLayerPosition(FloatPoint(layerBounds.location() - reflectionLayerBounds.location()));
     }
 
@@ -738 +738 @@
         IntSize scrollOffset = m_owningLayer->scrolledContentOffset();
 
-        m_scrollingLayer->setPosition(FloatPoint() + (paddingBox.location() - localCompositingBounds.location()));
+        m_scrollingLayer->setPosition(FloatPoint(paddingBox.location() - localCompositingBounds.location()));
 
         m_scrollingLayer->setSize(paddingBox.size());