Title: [139032] trunk/Source/WebCore
- Revision
- 139032
- Author
- [email protected]
- Date
- 2013-01-07 22:07:55 -0800 (Mon, 07 Jan 2013)
Log 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 <[email protected]> on 2013-01-07
Reviewed by Kentaro Hara.
Refactoring only, no functional change.
* rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (139031 => 139032)
--- trunk/Source/WebCore/ChangeLog 2013-01-08 06:03:20 UTC (rev 139031)
+++ trunk/Source/WebCore/ChangeLog 2013-01-08 06:07:55 UTC (rev 139032)
@@ -1,3 +1,15 @@
+2013-01-07 Steve Block <[email protected]>
+
+ 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 <[email protected]>
[EFL][WebGL] Crash of WebKitTestRunner when running webgl layout tests
Modified: trunk/Source/WebCore/rendering/RenderLayerBacking.cpp (139031 => 139032)
--- trunk/Source/WebCore/rendering/RenderLayerBacking.cpp 2013-01-08 06:03:20 UTC (rev 139031)
+++ trunk/Source/WebCore/rendering/RenderLayerBacking.cpp 2013-01-08 06:07:55 UTC (rev 139032)
@@ -624,7 +624,7 @@
RenderLayer::ClipRectsContext clipRectsContext(compAncestor, 0, TemporaryClipRects, IgnoreOverlayScrollbarSize, RenderLayer::IgnoreOverflowClip);
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());
// backgroundRect is relative to compAncestor, so subtract deltaX/deltaY to get back to local coords.
@@ -634,7 +634,7 @@
graphicsLayerParentLocation = parentClipRect.location();
}
- m_graphicsLayer->setPosition(FloatPoint() + (relativeCompositingBounds.location() - graphicsLayerParentLocation));
+ m_graphicsLayer->setPosition(FloatPoint(relativeCompositingBounds.location() - graphicsLayerParentLocation));
m_graphicsLayer->setOffsetFromRenderer(localCompositingBounds.location() - IntPoint());
FloatSize oldSize = m_graphicsLayer->size();
@@ -652,7 +652,7 @@
IntRect clippingBox;
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());
}
@@ -728,7 +728,7 @@
// but the reflected layer is the bounds of this layer, so we need to position it appropriately.
FloatRect layerBounds = compositedBounds();
FloatRect reflectionLayerBounds = reflectionBacking->compositedBounds();
- reflectionBacking->graphicsLayer()->setReplicatedLayerPosition(FloatPoint() + (layerBounds.location() - reflectionLayerBounds.location()));
+ reflectionBacking->graphicsLayer()->setReplicatedLayerPosition(FloatPoint(layerBounds.location() - reflectionLayerBounds.location()));
}
if (m_scrollingLayer) {
@@ -737,7 +737,7 @@
IntRect paddingBox(renderBox->borderLeft(), renderBox->borderTop(), renderBox->width() - renderBox->borderLeft() - renderBox->borderRight(), renderBox->height() - renderBox->borderTop() - renderBox->borderBottom());
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());
m_scrollingContentsLayer->setPosition(FloatPoint(-scrollOffset.width(), -scrollOffset.height()));
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
