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

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

Issue 2888093006: Place ellipsis correctly in inline blocks separated by spaces (Closed)
Patch Set: bug 133700 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 f3373c9e0dc71c64056fa35b23cde4b6b518188e..6844312616b265a7f9ca0ff87ee9085eb6a2d92d 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
@@ -2526,8 +2526,11 @@ void LayoutBlockFlow::TryPlacingEllipsisOnAtomicInlines(
for (InlineBox* box = ltr ? root->FirstChild() : root->LastChild(); box;
box = ltr ? box->NextOnLine() : box->PrevOnLine()) {
if (!box->GetLineLayoutItem().IsAtomicInlineLevel() ||
- !box->GetLineLayoutItem().IsLayoutBlockFlow())
+ !box->GetLineLayoutItem().IsLayoutBlockFlow()) {
+ if (box->GetLineLayoutItem().IsText())
+ logical_left_offset += box->LogicalWidth();
continue;
+ }
RootInlineBox* first_root_box =
LineLayoutBlockFlow(box->GetLineLayoutItem()).FirstRootBox();

Powered by Google App Engine
This is Rietveld 408576698