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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutDeprecatedFlexibleBox.cpp

Issue 2702043002: Use LayoutUnit for positioning ellipses (Closed)
Patch Set: bug 693905-2 Created 3 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
Index: third_party/WebKit/Source/core/layout/LayoutDeprecatedFlexibleBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutDeprecatedFlexibleBox.cpp b/third_party/WebKit/Source/core/layout/LayoutDeprecatedFlexibleBox.cpp
index bd33f446ad274b14193d845c47261a3bb94a06ee..ae2eadfc902ce6e935f2d21b54182063f5168300 100644
--- a/third_party/WebKit/Source/core/layout/LayoutDeprecatedFlexibleBox.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutDeprecatedFlexibleBox.cpp
@@ -1173,9 +1173,9 @@ void LayoutDeprecatedFlexibleBox::applyLineClamp(FlexBoxIterator& iterator,
LayoutUnit blockRightEdge = destBlock.logicalRightOffsetForLine(
lastVisibleLine->y(), DoNotIndentText);
if (!lastVisibleLine->lineCanAccommodateEllipsis(
- leftToRight, blockRightEdge.toInt(),
- (lastVisibleLine->x() + lastVisibleLine->logicalWidth()).toInt(),
- totalWidth))
+ leftToRight, blockRightEdge,
+ lastVisibleLine->x() + lastVisibleLine->logicalWidth(),
+ LayoutUnit(totalWidth)))
continue;
// Let the truncation code kick in.

Powered by Google App Engine
This is Rietveld 408576698