Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(754)

Unified Diff: Source/core/layout/Layer.cpp

Issue 952273006: Make the constructor of a LayoutRect from an IntRect explicit. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/layout/HitTestLocation.cpp ('k') | Source/core/layout/LayerClipper.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/layout/Layer.cpp
diff --git a/Source/core/layout/Layer.cpp b/Source/core/layout/Layer.cpp
index b044e25701b85822866d8f99424bf5dabfec857b..57cc430948e5134bec8df79280ea1a9c0b42d3d2 100644
--- a/Source/core/layout/Layer.cpp
+++ b/Source/core/layout/Layer.cpp
@@ -1585,7 +1585,7 @@ void Layer::collectFragments(LayerFragments& fragments, const Layer* rootLayer,
ClipRect backgroundRectInFlowThread;
ClipRect foregroundRectInFlowThread;
ClipRect outlineRectInFlowThread;
- clipper().calculateRects(paginationClipRectsContext, LayoutRect::infiniteIntRect(), layerBoundsInFlowThread, backgroundRectInFlowThread, foregroundRectInFlowThread,
+ clipper().calculateRects(paginationClipRectsContext, LayoutRect(LayoutRect::infiniteIntRect()), layerBoundsInFlowThread, backgroundRectInFlowThread, foregroundRectInFlowThread,
outlineRectInFlowThread, &offsetWithinPaginatedLayer);
// Take our bounding box within the flow thread and clip it.
@@ -1651,7 +1651,7 @@ static inline LayoutRect frameVisibleRect(LayoutObject* renderer)
if (!frameView)
return LayoutRect();
- return frameView->visibleContentRect();
+ return LayoutRect(frameView->visibleContentRect());
}
bool Layer::hitTest(const HitTestRequest& request, HitTestResult& result)
@@ -1670,7 +1670,7 @@ bool Layer::hitTest(const HitTestRequest& request, const HitTestLocation& hitTes
// Start with frameVisibleRect to ensure we include the scrollbars.
LayoutRect hitTestArea = frameVisibleRect(renderer());
if (request.ignoreClipping())
- hitTestArea.unite(renderer()->view()->documentRect());
+ hitTestArea.unite(LayoutRect(renderer()->view()->documentRect()));
Layer* insideLayer = hitTestLayer(this, 0, request, result, hitTestArea, hitTestLocation, false);
if (!insideLayer) {
@@ -2203,7 +2203,7 @@ Layer* Layer::hitTestChildLayerColumns(Layer* childLayer, Layer* rootLayer, cons
newTransformState->translate(offset.width(), offset.height(), HitTestingTransformState::AccumulateTransform);
FloatPoint localPoint = newTransformState->mappedPoint();
FloatQuad localPointQuad = newTransformState->mappedQuad();
- LayoutRect localHitTestRect = newTransformState->mappedArea().enclosingBoundingBox();
+ LayoutRect localHitTestRect(newTransformState->mappedArea().enclosingBoundingBox());
HitTestLocation newHitTestLocation;
if (hitTestLocation.isRectBasedTest())
newHitTestLocation = HitTestLocation(localPoint, localPointQuad);
@@ -2253,7 +2253,7 @@ void Layer::invalidatePaintForBlockSelectionGaps()
if (m_blockSelectionGapsBounds.isEmpty())
return;
- LayoutRect rect = m_blockSelectionGapsBounds;
+ LayoutRect rect(m_blockSelectionGapsBounds);
if (renderer()->hasOverflowClip()) {
LayoutBox* box = layoutBox();
rect.move(-box->scrolledContentOffset());
@@ -2436,7 +2436,7 @@ LayoutRect Layer::boundingBoxForCompositing(const Layer* ancestorLayer, Calculat
// The root layer is always just the size of the document.
if (isRootLayer())
- return m_renderer->view()->unscaledDocumentRect();
+ return LayoutRect(m_renderer->view()->unscaledDocumentRect());
// The layer created for the LayoutFlowThread is just a helper for painting and hit-testing,
// and should not contribute to the bounding box. The LayoutMultiColumnSets will contribute
« no previous file with comments | « Source/core/layout/HitTestLocation.cpp ('k') | Source/core/layout/LayerClipper.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698