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

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

Issue 1303413004: Convert some call sites to use explicit LayoutRect->FloatRect conversion. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 4 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
Index: Source/core/layout/LayoutInline.cpp
diff --git a/Source/core/layout/LayoutInline.cpp b/Source/core/layout/LayoutInline.cpp
index e3044721f6f9e24785264a853c24c927b1262ad4..76ce2a151e177eb769bca2077e492a7198d77a17 100644
--- a/Source/core/layout/LayoutInline.cpp
+++ b/Source/core/layout/LayoutInline.cpp
@@ -1026,7 +1026,7 @@ LayoutRect LayoutInline::absoluteClippedOverflowRect() const
for (LayoutBlock* currBlock = containingBlock(); currBlock && currBlock->isAnonymousBlock(); currBlock = toLayoutBlock(currBlock->nextSibling())) {
for (LayoutObject* curr = currBlock->firstChild(); curr; curr = curr->nextSibling()) {
- LayoutRect rect = curr->clippedOverflowRectForPaintInvalidation(view());
+ FloatRect rect(curr->clippedOverflowRectForPaintInvalidation(view()));
eae 2015/08/25 17:47:16 Why are we using a FloatRect here given that the c
chrishtr 2015/08/25 18:30:43 The context() function on line 1030 requires it. I
context(rect);
if (curr == endContinuation)
return LayoutRect(enclosingIntRect(floatResult));

Powered by Google App Engine
This is Rietveld 408576698