Chromium Code Reviews Chromium Code Reviews
Issue 2175433002:
Remove the unnecessary blockquote after indenting an empty blockquote (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: third_party/WebKit/LayoutTests/editing/execCommand/indent-empty-quote.html |
| diff --git a/third_party/WebKit/LayoutTests/editing/execCommand/indent-empty-quote.html b/third_party/WebKit/LayoutTests/editing/execCommand/indent-empty-quote.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..f5788d2d52ed19d88a7fb623cbaa566a803388b7 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/editing/execCommand/indent-empty-quote.html |
| @@ -0,0 +1,19 @@ |
| +<!doctype html> |
|
yosin_UTC9
2016/07/22 01:21:55
Could you put this test file in "LayoutTests/editi
joone
2016/07/22 03:29:54
Done.
|
| +<script src="../../resources/testharness.js"></script> |
| +<script src="../../resources/testharnessreport.js"></script> |
| +<script src="../assert_selection.js"></script> |
| +<div id="log"></div> |
| +<script> |
| +test(() => { |
| + assert_selection( |
| + '<div contenteditable><blockquote>|<br></blockquote></div>', |
| + 'indent true', |
|
yosin_UTC9
2016/07/22 01:21:55
nit: I think we don't need to have |true|, since "
joone
2016/07/22 03:29:54
Done.
|
| + '<div contenteditable><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;"><blockquote>|<br></blockquote></blockquote></div>'); |
| +}, 'indent empty <blockquote>'); |
| +test(() => { |
| + assert_selection( |
| + '<div contenteditable><blockquote>1|</blockquote></div>', |
| + 'indent true', |
|
yosin_UTC9
2016/07/22 01:21:55
nit: I think we don't need to have |true|, since "
joone
2016/07/22 03:29:54
Done.
|
| + '<div contenteditable><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;"><blockquote>1|</blockquote></blockquote></div>'); |
| +}, 'indent <blockquote> with text'); |
| +</script> |
