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

Unified Diff: third_party/WebKit/LayoutTests/editing/selection/modify_extend/extend_by_word_002.html

Issue 2754543004: Make Selection#modify() with word granularity not to skip punctuation (Closed)
Patch Set: 2017-03-26T01:40:34 rebase Created 3 years, 9 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/editing/text-iterator/first-letter-word-boundary.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/editing/selection/modify_extend/extend_by_word_002.html
diff --git a/third_party/WebKit/LayoutTests/editing/selection/modify_extend/extend_by_word_002.html b/third_party/WebKit/LayoutTests/editing/selection/modify_extend/extend_by_word_002.html
index c267b56b0dce9fa166ec9927bbaa7c239a2296b4..301741f6e60a619a50d4855445e86afa01913eb3 100644
--- a/third_party/WebKit/LayoutTests/editing/selection/modify_extend/extend_by_word_002.html
+++ b/third_party/WebKit/LayoutTests/editing/selection/modify_extend/extend_by_word_002.html
@@ -29,12 +29,7 @@ test(() => assert_selection(
`<li>${middot} <a href="foo.html">Soups & Salads</a></li>`,
`<li>${middot} <a href="foo.html">Sandwiches & Burgers</a></li>`,
isMac ? `<li>${middot} <a href="foo.html">Steak| & Ribs</a></li>`
- // The current non-Mac behavior is wrong. The correct focus
- // should be at 'Steak & |Ribs'. A deeper reason is that Blink
- // performs backward word boundary searches to refine the final
- // selection, which doesn't work with first-letter
- // (crbug.com/671104).
- : `<li>${middot} <a href="foo.html">Steak &| Ribs</a></li>`,
+ : `<li>${middot} <a href="foo.html">Steak |& Ribs</a></li>`,
`<li>${middot} <a href="foo.html">Seafood</a></li>`,
`<li>${middot} <a href="foo.html">Combos</a></li>`,
'</ul>'
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/editing/text-iterator/first-letter-word-boundary.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698