[LayoutNG] Not paint document marker on ellipsis.

This patch stops painting document markers on ellipsis character. Old implementation is bit strange : painting background of ellipsis as culled characters are not hided only for find-in-page marker. See: https://docs.google.com/document/d/1Osd-W9IMxHrrgJ2Kvo7HfpVTms7EDUbOHhsmL1c3_Yo/ This patch implements simply: do not paint any marker on ellipsis because characters are hide. That can cause a bit weird situation: if find-in-page hits such hide texts, it shows numbers of found texts but blink doesn't paint anything. However old implementation already has similar situation: if found text are clipped by text-overflow: clip, also blink doesn't show them. Thus this is acceptable change. This patch fixes following tests: paint/markers/ellipsis-ltr-text-in-ltr-flow-with-markers.html paint/markers/ellipsis-ltr-text-in-rtl-flow-with-markers.html paint/markers/ellipsis-mixed-text-in-ltr-flow-with-markers.html paint/markers/ellipsis-mixed-text-in-rtl-flow-with-markers.html paint/markers/ellipsis-rtl-text-in-ltr-flow-with-markers.html paint/markers/ellipsis-rtl-text-in-rtl-flow-with-markers.html Bug: 850448 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Ibdd7d485e2b5ac4b5db665d54b64131e6c1bcdb9 Reviewed-on: https://chromium-review.googlesource.com/1105681 Commit-Queue: Yoichi Osato <yoichio@chromium.org> Reviewed-by: Koji Ishii <kojii@chromium.org> Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#569152}

[LayoutNG] Not paint document marker on ellipsis.
This patch stops painting document markers on ellipsis character. Old implementation is bit strange : painting background of ellipsis as culled characters are not hided only for find-in-page marker. See: https://docs.google.com/document/d/1Osd-W9IMxHrrgJ2Kvo7HfpVTms7EDUbOHhsmL1c3_Yo/ This patch implements simply: do not paint any marker on ellipsis because characters are hide. That can cause a bit weird situation: if find-in-page hits such hide texts, it shows numbers of found texts but blink doesn't paint anything. However old implementation already has similar situation: if found text are clipped by text-overflow: clip, also blink doesn't show them. Thus this is acceptable change. This patch fixes following tests: paint/markers/ellipsis-ltr-text-in-ltr-flow-with-markers.html paint/markers/ellipsis-ltr-text-in-rtl-flow-with-markers.html paint/markers/ellipsis-mixed-text-in-ltr-flow-with-markers.html paint/markers/ellipsis-mixed-text-in-rtl-flow-with-markers.html paint/markers/ellipsis-rtl-text-in-ltr-flow-with-markers.html paint/markers/ellipsis-rtl-text-in-rtl-flow-with-markers.html Bug: 850448 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Ibdd7d485e2b5ac4b5db665d54b64131e6c1bcdb9 Reviewed-on: https://chromium-review.googlesource.com/1105681 Commit-Queue: Yoichi Osato <yoichio@chromium.org> Reviewed-by: Koji Ishii <kojii@chromium.org> Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#569152}
1805b4dc · Yoichi Osato · Commit Bot · a424f9e0 · 1805b4dc · 1805b4dc
Commit 1805b4dc authored Jun 21, 2018 by Yoichi Osato Committed by Commit Bot Jun 21, 2018
13 changed files
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-ltr-text-in-ltr-flow-with-markers-expected.png
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-ltr-text-in-ltr-flow-with-markers-expected.png
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-ltr-text-in-ltr-flow-with-markers-expected.txt
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-ltr-text-in-ltr-flow-with-markers-expected.txt
+layer at (0,0) size 800x600
+  LayoutView at (0,0) size 800x600
+layer at (0,0) size 800x196
+  LayoutNGBlockFlow {HTML} at (0,0) size 800x196
+    LayoutNGBlockFlow {BODY} at (8,8) size 784x180
+layer at (8,8) size 100x20 scrollWidth 133
+  LayoutNGBlockFlow {DIV} at (0,0) size 100x20
+    LayoutText {#text} at (0,0) size 99x19
+      text run at (0,0) width 83: "markAllCom"
+      text run at (83,0) width 16: "\x{2026}"
+layer at (8,28) size 100x20 scrollWidth 142
+  LayoutNGBlockFlow {DIV} at (0,20) size 100x20
+    LayoutText {#text} at (0,0) size 96x19
+      text run at (0,0) width 80: "markStartCo"
+      text run at (80,0) width 16: "\x{2026}"
+layer at (8,48) size 100x20 scrollWidth 139
+  LayoutNGBlockFlow {DIV} at (0,40) size 100x20
+    LayoutText {#text} at (0,0) size 93x19
+      text run at (0,0) width 77: "markEndCo"
+      text run at (77,0) width 16: "\x{2026}"
+layer at (8,68) size 100x20 scrollWidth 104
+  LayoutNGBlockFlow {DIV} at (0,60) size 100x20
+    LayoutText {#text} at (0,0) size 100x19
+      text run at (0,0) width 84: "markAllSpell"
+      text run at (84,0) width 16: "\x{2026}"
+layer at (8,88) size 100x20 scrollWidth 113
+  LayoutNGBlockFlow {DIV} at (0,80) size 100x20
+    LayoutText {#text} at (0,0) size 94x19
+      text run at (0,0) width 78: "markStartSp"
+      text run at (78,0) width 16: "\x{2026}"
+layer at (8,108) size 100x20 scrollWidth 110
+  LayoutNGBlockFlow {DIV} at (0,100) size 100x20
+    LayoutText {#text} at (0,0) size 98x19
+      text run at (0,0) width 82: "markEndSpe"
+      text run at (82,0) width 16: "\x{2026}"
+layer at (8,128) size 100x20 scrollWidth 120
+  LayoutNGBlockFlow {DIV} at (0,120) size 100x20
+    LayoutText {#text} at (0,0) size 96x19
+      text run at (0,0) width 80: "markAllText"
+      text run at (80,0) width 16: "\x{2026}"
+layer at (8,148) size 100x20 scrollWidth 129
+  LayoutNGBlockFlow {DIV} at (0,140) size 100x20
+    LayoutText {#text} at (0,0) size 93x19
+      text run at (0,0) width 77: "markStartTe"
+      text run at (77,0) width 16: "\x{2026}"
+layer at (8,168) size 100x20 scrollWidth 126
+  LayoutNGBlockFlow {DIV} at (0,160) size 100x20
+    LayoutText {#text} at (0,0) size 98x19
+      text run at (0,0) width 82: "markEndTex"
+      text run at (82,0) width 16: "\x{2026}"
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-ltr-text-in-rtl-flow-with-markers-expected.png
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-ltr-text-in-rtl-flow-with-markers-expected.png
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-ltr-text-in-rtl-flow-with-markers-expected.txt
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-ltr-text-in-rtl-flow-with-markers-expected.txt
+layer at (0,0) size 800x600
+  LayoutView at (0,0) size 800x600
+layer at (0,0) size 800x196
+  LayoutNGBlockFlow {HTML} at (0,0) size 800x196
+    LayoutNGBlockFlow {BODY} at (8,8) size 784x180
+layer at (8,8) size 100x20 scrollX 33.00 scrollWidth 133
+  LayoutNGBlockFlow {DIV} at (0,0) size 100x20
+    LayoutText {#text} at (19,0) size 97x19
+      text run at (19,0) width 81: "Composition"
+      text run at (3,0) width 16: "\x{2026}"
+layer at (8,28) size 100x20 scrollX 42.00 scrollWidth 142
+  LayoutNGBlockFlow {DIV} at (0,20) size 100x20
+    LayoutText {#text} at (19,0) size 97x19
+      text run at (19,0) width 81: "Composition"
+      text run at (3,0) width 16: "\x{2026}"
+layer at (8,48) size 100x20 scrollX 39.00 scrollWidth 139
+  LayoutNGBlockFlow {DIV} at (0,40) size 100x20
+    LayoutText {#text} at (19,0) size 97x19
+      text run at (19,0) width 81: "Composition"
+      text run at (3,0) width 16: "\x{2026}"
+layer at (8,68) size 100x20 scrollX 4.00 scrollWidth 104
+  LayoutNGBlockFlow {DIV} at (0,60) size 100x20
+    LayoutText {#text} at (20,0) size 96x19
+      text run at (20,0) width 80: "kAllSpelling"
+      text run at (4,0) width 16: "\x{2026}"
+layer at (8,88) size 100x20 scrollX 13.00 scrollWidth 113
+  LayoutNGBlockFlow {DIV} at (0,80) size 100x20
+    LayoutText {#text} at (19,0) size 97x19
+      text run at (19,0) width 81: "StartSpelling"
+      text run at (3,0) width 16: "\x{2026}"
+layer at (8,108) size 100x20 scrollX 10.00 scrollWidth 110
+  LayoutNGBlockFlow {DIV} at (0,100) size 100x20
+    LayoutText {#text} at (22,0) size 94x19
+      text run at (22,0) width 78: "EndSpelling"
+      text run at (6,0) width 16: "\x{2026}"
+layer at (8,128) size 100x20 scrollX 20.00 scrollWidth 120
+  LayoutNGBlockFlow {DIV} at (0,120) size 100x20
+    LayoutText {#text} at (24,0) size 92x19
+      text run at (24,0) width 76: "llTextMatch"
+      text run at (8,0) width 16: "\x{2026}"
+layer at (8,148) size 100x20 scrollX 29.00 scrollWidth 129
+  LayoutNGBlockFlow {DIV} at (0,140) size 100x20
+    LayoutText {#text} at (23,0) size 93x19
+      text run at (23,0) width 77: "rtTextMatch"
+      text run at (7,0) width 16: "\x{2026}"
+layer at (8,168) size 100x20 scrollX 26.00 scrollWidth 126
+  LayoutNGBlockFlow {DIV} at (0,160) size 100x20
+    LayoutText {#text} at (24,0) size 92x19
+      text run at (24,0) width 76: "dTextMatch"
+      text run at (8,0) width 16: "\x{2026}"
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-mixed-text-in-ltr-flow-with-markers-expected.png
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-mixed-text-in-ltr-flow-with-markers-expected.png
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-mixed-text-in-ltr-flow-with-markers-expected.txt
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-mixed-text-in-ltr-flow-with-markers-expected.txt
+layer at (0,0) size 800x600
+  LayoutView at (0,0) size 800x600
+layer at (0,0) size 800x256
+  LayoutNGBlockFlow {HTML} at (0,0) size 800x256
+    LayoutNGBlockFlow {BODY} at (8,8) size 784x240
+layer at (8,8) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,0) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,28) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,20) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,48) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,40) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,68) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,60) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,88) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,80) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,108) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,100) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,128) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,120) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,148) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,140) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,168) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,160) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,188) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,180) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,208) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,200) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,228) size 100x20 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,220) size 100x20
+    LayoutText {#text} at (0,0) size 134x19
+      text run at (0,0) width 35: "Hello"
+      text run at (35,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (67,0) width 12: "H"
+      text run at (102,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (79,0) width 16: "\x{2026}"
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-mixed-text-in-rtl-flow-with-markers-expected.png
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-mixed-text-in-rtl-flow-with-markers-expected.png
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-mixed-text-in-rtl-flow-with-markers-expected.txt
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-mixed-text-in-rtl-flow-with-markers-expected.txt
+layer at (0,0) size 800x600
+  LayoutView at (0,0) size 800x600
+layer at (0,0) size 800x256
+  LayoutNGBlockFlow {HTML} at (0,0) size 800x256
+    LayoutNGBlockFlow {BODY} at (8,8) size 784x240
+layer at (8,8) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,0) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,28) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,20) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,48) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,40) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,68) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,60) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,88) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,80) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,108) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,100) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,128) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,120) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,148) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,140) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,168) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,160) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,188) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,180) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,208) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,200) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
+layer at (8,228) size 100x20 scrollX 34.00 scrollWidth 134
+  LayoutNGBlockFlow {DIV} at (0,220) size 100x20
+    LayoutText {#text} at (-34,0) size 134x19
+      text run at (-34,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (17,0) width 16: "llo"
+      text run at (33,0) width 32: "\x{5E9}\x{5DC}\x{5D5}\x{5DD}"
+      text run at (65,0) width 35: "Hello"
+      text run at (1,0) width 16: "\x{2026}"
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-rtl-text-in-ltr-flow-with-markers-expected.png
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-rtl-text-in-ltr-flow-with-markers-expected.png
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-rtl-text-in-ltr-flow-with-markers-expected.txt
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-rtl-text-in-ltr-flow-with-markers-expected.txt
+layer at (0,0) size 800x600
+  LayoutView at (0,0) size 800x600
+layer at (0,0) size 800x196
+  LayoutNGBlockFlow {HTML} at (0,0) size 800x196
+    LayoutNGBlockFlow {BODY} at (8,8) size 784x180
+layer at (8,8) size 100x20 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,0) size 100x20
+    LayoutText {#text} at (0,0) size 95x19
+      text run at (0,0) width 79: "\x{5E2}\x{5E4}\x{5E3}\x{5E6}\x{5E5}\x{5E7}\x{5E8}\x{5E9}\x{5EA}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,28) size 100x20 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,20) size 100x20
+    LayoutText {#text} at (0,0) size 95x19
+      text run at (0,0) width 79: "\x{5E2}\x{5E4}\x{5E3}\x{5E6}\x{5E5}\x{5E7}\x{5E8}\x{5E9}\x{5EA}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,48) size 100x20 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,40) size 100x20
+    LayoutText {#text} at (0,0) size 95x19
+      text run at (0,0) width 79: "\x{5E2}\x{5E4}\x{5E3}\x{5E6}\x{5E5}\x{5E7}\x{5E8}\x{5E9}\x{5EA}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,68) size 100x20 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,60) size 100x20
+    LayoutText {#text} at (0,0) size 95x19
+      text run at (0,0) width 79: "\x{5E2}\x{5E4}\x{5E3}\x{5E6}\x{5E5}\x{5E7}\x{5E8}\x{5E9}\x{5EA}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,88) size 100x20 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,80) size 100x20
+    LayoutText {#text} at (0,0) size 95x19
+      text run at (0,0) width 79: "\x{5E2}\x{5E4}\x{5E3}\x{5E6}\x{5E5}\x{5E7}\x{5E8}\x{5E9}\x{5EA}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,108) size 100x20 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,100) size 100x20
+    LayoutText {#text} at (0,0) size 95x19
+      text run at (0,0) width 79: "\x{5E2}\x{5E4}\x{5E3}\x{5E6}\x{5E5}\x{5E7}\x{5E8}\x{5E9}\x{5EA}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,128) size 100x20 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,120) size 100x20
+    LayoutText {#text} at (0,0) size 95x19
+      text run at (0,0) width 79: "\x{5E2}\x{5E4}\x{5E3}\x{5E6}\x{5E5}\x{5E7}\x{5E8}\x{5E9}\x{5EA}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,148) size 100x20 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,140) size 100x20
+    LayoutText {#text} at (0,0) size 95x19
+      text run at (0,0) width 79: "\x{5E2}\x{5E4}\x{5E3}\x{5E6}\x{5E5}\x{5E7}\x{5E8}\x{5E9}\x{5EA}"
+      text run at (79,0) width 16: "\x{2026}"
+layer at (8,168) size 100x20 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,160) size 100x20
+    LayoutText {#text} at (0,0) size 95x19
+      text run at (0,0) width 79: "\x{5E2}\x{5E4}\x{5E3}\x{5E6}\x{5E5}\x{5E7}\x{5E8}\x{5E9}\x{5EA}"
+      text run at (79,0) width 16: "\x{2026}"
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-rtl-text-in-rtl-flow-with-markers-expected.png
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-rtl-text-in-rtl-flow-with-markers-expected.png
--- a/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-rtl-text-in-rtl-flow-with-markers-expected.txt
+++ b/third_party/WebKit/LayoutTests/flag-specific/enable-blink-features=LayoutNG/paint/markers/ellipsis-rtl-text-in-rtl-flow-with-markers-expected.txt
+layer at (0,0) size 800x600
+  LayoutView at (0,0) size 800x600
+layer at (0,0) size 800x196
+  LayoutNGBlockFlow {HTML} at (0,0) size 800x196
+    LayoutNGBlockFlow {BODY} at (8,8) size 784x180
+layer at (8,8) size 100x20 scrollX 101.00 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,0) size 100x20
+    LayoutText {#text} at (21,0) size 95x19
+      text run at (21,0) width 79: "\x{5D0}\x{5D1}\x{5D2}\x{5D3}\x{5D4}\x{5D5}\x{5D6}\x{5D7}\x{5D8}\x{5D9}\x{5DB}"
+      text run at (5,0) width 16: "\x{2026}"
+layer at (8,28) size 100x20 scrollX 101.00 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,20) size 100x20
+    LayoutText {#text} at (21,0) size 95x19
+      text run at (21,0) width 79: "\x{5D0}\x{5D1}\x{5D2}\x{5D3}\x{5D4}\x{5D5}\x{5D6}\x{5D7}\x{5D8}\x{5D9}\x{5DB}"
+      text run at (5,0) width 16: "\x{2026}"
+layer at (8,48) size 100x20 scrollX 101.00 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,40) size 100x20
+    LayoutText {#text} at (21,0) size 95x19
+      text run at (21,0) width 79: "\x{5D0}\x{5D1}\x{5D2}\x{5D3}\x{5D4}\x{5D5}\x{5D6}\x{5D7}\x{5D8}\x{5D9}\x{5DB}"
+      text run at (5,0) width 16: "\x{2026}"
+layer at (8,68) size 100x20 scrollX 101.00 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,60) size 100x20
+    LayoutText {#text} at (21,0) size 95x19
+      text run at (21,0) width 79: "\x{5D0}\x{5D1}\x{5D2}\x{5D3}\x{5D4}\x{5D5}\x{5D6}\x{5D7}\x{5D8}\x{5D9}\x{5DB}"
+      text run at (5,0) width 16: "\x{2026}"
+layer at (8,88) size 100x20 scrollX 101.00 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,80) size 100x20
+    LayoutText {#text} at (21,0) size 95x19
+      text run at (21,0) width 79: "\x{5D0}\x{5D1}\x{5D2}\x{5D3}\x{5D4}\x{5D5}\x{5D6}\x{5D7}\x{5D8}\x{5D9}\x{5DB}"
+      text run at (5,0) width 16: "\x{2026}"
+layer at (8,108) size 100x20 scrollX 101.00 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,100) size 100x20
+    LayoutText {#text} at (21,0) size 95x19
+      text run at (21,0) width 79: "\x{5D0}\x{5D1}\x{5D2}\x{5D3}\x{5D4}\x{5D5}\x{5D6}\x{5D7}\x{5D8}\x{5D9}\x{5DB}"
+      text run at (5,0) width 16: "\x{2026}"
+layer at (8,128) size 100x20 scrollX 101.00 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,120) size 100x20
+    LayoutText {#text} at (21,0) size 95x19
+      text run at (21,0) width 79: "\x{5D0}\x{5D1}\x{5D2}\x{5D3}\x{5D4}\x{5D5}\x{5D6}\x{5D7}\x{5D8}\x{5D9}\x{5DB}"
+      text run at (5,0) width 16: "\x{2026}"
+layer at (8,148) size 100x20 scrollX 101.00 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,140) size 100x20
+    LayoutText {#text} at (21,0) size 95x19
+      text run at (21,0) width 79: "\x{5D0}\x{5D1}\x{5D2}\x{5D3}\x{5D4}\x{5D5}\x{5D6}\x{5D7}\x{5D8}\x{5D9}\x{5DB}"
+      text run at (5,0) width 16: "\x{2026}"
+layer at (8,168) size 100x20 scrollX 101.00 scrollWidth 201
+  LayoutNGBlockFlow {DIV} at (0,160) size 100x20
+    LayoutText {#text} at (21,0) size 95x19
+      text run at (21,0) width 79: "\x{5D0}\x{5D1}\x{5D2}\x{5D3}\x{5D4}\x{5D5}\x{5D6}\x{5D7}\x{5D8}\x{5D9}\x{5DB}"
+      text run at (5,0) width 16: "\x{2026}"
--- a/third_party/blink/renderer/core/paint/ng/ng_text_fragment_painter.cc
+++ b/third_party/blink/renderer/core/paint/ng/ng_text_fragment_painter.cc
@@ -73,6 +73,11 @@ DocumentMarkerVector ComputeMarkersToPaint(
  Node* const node = paint_fragment.GetNode();
  if (!node)
    return DocumentMarkerVector();
+  // We don't paint any marker on ellipsis.
+  if (paint_fragment.PhysicalFragment().StyleVariant() ==
+      NGStyleVariant::kEllipsis)
+    return DocumentMarkerVector();
+
  DocumentMarkerController& document_marker_controller =
      node->GetDocument().Markers();
  return document_marker_controller.ComputeMarkersToPaint(*node);