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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.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/LayoutBlockFlowLine.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp b/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
index 7328c89a284079e8cfeb459a15e62b61c3fbb8b9..ab7e8b01ebab2e21126cbee1ec5a99d4c9e871d7 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
@@ -2420,8 +2420,8 @@ void LayoutBlockFlow::checkLinesForTextOverflow() {
LayoutUnit width(indentText == IndentText ? firstLineEllipsisWidth
: ellipsisWidth);
LayoutUnit blockEdge = ltr ? blockRightEdge : blockLeftEdge;
- if (curr->lineCanAccommodateEllipsis(
- ltr, blockEdge.toInt(), lineBoxEdge.toInt(), width.toInt())) {
+ if (curr->lineCanAccommodateEllipsis(ltr, blockEdge, lineBoxEdge,
+ width)) {
LayoutUnit totalLogicalWidth = curr->placeEllipsis(
selectedEllipsisStr, ltr, blockLeftEdge, blockRightEdge, width);
LayoutUnit logicalLeft; // We are only interested in the delta from the
« no previous file with comments | « third_party/WebKit/LayoutTests/TestExpectations ('k') | third_party/WebKit/Source/core/layout/LayoutDeprecatedFlexibleBox.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698