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

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

Issue 2880653002: Display ellipsis correctly in inline blocks adjacent to floats (Closed)
Patch Set: bug 720377 Created 3 years, 7 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 88b64403cd49fc669e971434d9519a3f74768351..f3373c9e0dc71c64056fa35b23cde4b6b518188e 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
@@ -2499,8 +2499,8 @@ void LayoutBlockFlow::CheckLinesForTextOverflow() {
curr->MoveInInlineDirection(
logical_left - (available_logical_width - total_logical_width));
} else {
- TryPlacingEllipsisOnAtomicInlines(curr, block_right_edge,
- block_left_edge, width,
+ TryPlacingEllipsisOnAtomicInlines(curr, LogicalRightOffsetForContent(),
+ LogicalLeftOffsetForContent(), width,
selected_ellipsis_str);
}
}
@@ -2569,7 +2569,7 @@ void LayoutBlockFlow::TryPlacingEllipsisOnAtomicInlines(
logical_left_offset += max_root_box_width - curr->LogicalWidth();
curr->PlaceEllipsis(selected_ellipsis_str, ltr, block_left_edge,
block_right_edge, ellipsis_width,
- LayoutUnit(logical_left_offset.Ceil()), found_box);
+ logical_left_offset, found_box);
placed_ellipsis = true;
}
}

Powered by Google App Engine
This is Rietveld 408576698