Chromium Code Reviews Chromium Code Reviews
Issue 2525273002:
[Editing][CodeHealth] Convert editing/selection to utilize w3c test harness part 1. (Closed)
| Index: third_party/WebKit/LayoutTests/editing/selection/modify_move/move-by-paragraph.html |
| diff --git a/third_party/WebKit/LayoutTests/editing/selection/modify_move/move-by-paragraph.html b/third_party/WebKit/LayoutTests/editing/selection/modify_move/move-by-paragraph.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..5ba2ff87e07deca5d6ca99c9699bf9e33073a5e5 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/editing/selection/modify_move/move-by-paragraph.html |
| @@ -0,0 +1,26 @@ |
| +<!doctype html> |
| +<script src="../../../resources/testharness.js"></script> |
| +<script src="../../../resources/testharnessreport.js"></script> |
| +<script src="../../assert_selection.js"></script> |
| +<script> |
| +test(() => { |
| + assert_selection( |
| + '<div>foo<div>bar|</div>baz</div>', |
| + selection => selection.modify('move', 'backward', 'paragraph'), |
| + '<div>foo|<div>bar</div>baz</div>'); |
| + assert_selection( |
| + '<div>foo<span>bar|</span>baz</div>', |
| + selection => selection.modify('move', 'backward', 'paragraph'), |
| + '<div>|foo<span>bar</span>baz</div>'); |
| + |
|
yosin_UTC9
2017/01/11 05:59:54
Could you have two |test()| rather than adding bla
yoichio
2017/01/11 07:18:57
Done.
|
| + assert_selection( |
| + '<div>foo<div>|bar</div>baz</div>', |
| + selection => selection.modify('move', 'forward', 'paragraph'), |
| + '<div>foo<div>bar</div>|baz</div>'); |
| + assert_selection( |
| + '<div>foo<span>|bar</span>baz</div>', |
| + selection => selection.modify('move', 'forward', 'paragraph'), |
| + '<div>foo<span>bar</span>baz|</div>'); |
| + |
|
yosin_UTC9
2017/01/11 05:59:54
nit: no need to have an extra blank line.
yoichio
2017/01/11 07:18:57
Done.
|
| +}, 'Selection.modify move by paragraph'); |
| +</script> |
