diff --git a/third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp b/third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp
index e4d74e8..3f62c7d 100644
--- a/third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp
+++ b/third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp
@@ -419,9 +418,14 @@ LayoutUnit InlineTextBox::PlaceEllipsisBox(bool flow_is_ltr,
: LogicalRight() - visible_box_width;
}
// We measure the text using the second half of the previous character and
// the first half of the current one when the text is rtl. This gives a
// more accurate position in rtl text.
+ // TODO(crbug.com/722043: This doesn't always give the best results.
int offset = OffsetForPosition(ellipsis_x, !ltr);
// Full truncation is only necessary when we're flowing left-to-right.
if (flow_is_ltr && offset == 0 && ltr == flow_is_ltr) {
Comment 1 by robho...@gmail.com
, May 13 2017diff --git a/third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp b/third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp index e4d74e8..3f62c7d 100644 --- a/third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp +++ b/third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp @@ -419,9 +418,14 @@ LayoutUnit InlineTextBox::PlaceEllipsisBox(bool flow_is_ltr, : LogicalRight() - visible_box_width; } // We measure the text using the second half of the previous character and // the first half of the current one when the text is rtl. This gives a // more accurate position in rtl text. + // TODO(crbug.com/722043: This doesn't always give the best results. int offset = OffsetForPosition(ellipsis_x, !ltr); // Full truncation is only necessary when we're flowing left-to-right. if (flow_is_ltr && offset == 0 && ltr == flow_is_ltr) {