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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/LayoutTests/editing/selection/user-select/user-select-all-contenteditable.html

Issue 2120913002: [Editing][Regression] Contenteditable w/ "-webkit-user-select:all" should be editable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add selection test Created 4 years, 5 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/Source/core/dom/Node.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/editing/selection/user-select/user-select-all-contenteditable.html
diff --git a/third_party/WebKit/LayoutTests/editing/selection/user-select/user-select-all-contenteditable.html b/third_party/WebKit/LayoutTests/editing/selection/user-select/user-select-all-contenteditable.html
new file mode 100644
index 0000000000000000000000000000000000000000..c251d9dde142f05521a81da3a2b3d4795665dad1
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/editing/selection/user-select/user-select-all-contenteditable.html
@@ -0,0 +1,73 @@
+<!doctype HTML>
+<script src="../../../resources/testharness.js"></script>
+<script src="../../../resources/testharnessreport.js"></script>
+<script src="../../assert_selection.js"></script>
+<div id="log"></div>
+<script>
+test(function(){
yosin_UTC9 2016/07/08 09:30:50 It seems there are not test for HTMLElement.isCont
yoichio 2016/07/11 06:54:04 Done.
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">foo</div>',
+ selection => selection.collapse(selection.document.getElementById('div'), 0),
yosin_UTC9 2016/07/08 09:30:50 Please use other than zero or adding another test
yoichio 2016/07/11 06:54:04 Done.
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">|foo</div>');
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">|foo</div>',
+ selection => selection.extend(selection.document.getElementById('div'), 1),
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">^foo|</div>');
yosin_UTC9 2016/07/08 09:30:50 We should check attempting partial selection becom
yoichio 2016/07/11 06:54:04 Done.
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">foo</div>',
+ selection => {
+ var div = selection.document.getElementById('div');
+ selection.setBaseAndExtent(div, 0, div, 1);
+ },
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">^foo|</div>');
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">foo</div>',
+ selection => {
+ var range = selection.document.createRange();
+ var div = selection.document.getElementById('div');
+ range.setStart(div, 0);
+ range.setEnd(div, 1);
+ selection.addRange(range);
+ },
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">^foo|</div>');
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">|foo</div>',
+ selection => selection.modify('move', 'forward', 'character'),
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">f|oo</div>');
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">f|oo</div>',
+ selection => selection.modify('move', 'backward', 'character'),
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">|foo</div>');
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">|foo</div>',
+ selection => selection.modify('extend', 'forward', 'character'),
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">^foo|</div>');
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">foo|</div>',
+ selection => selection.modify('extend', 'backward', 'character'),
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">|foo^</div>');
+ },
+'Selection API can edit in -webkit-user-select:all contenteditable element');
+
+test(function(){
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">|foo</div>',
+ selection => selection.document.execCommand('insertText', false, 'bar'),
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">bar|foo</div>');
+ },
+'Execcommand inserttext in -webkit-user-select:all contenteditable element');
+
+test(function(){
yosin_UTC9 2016/07/11 01:38:41 Could you move this test case to another file to u
yoichio 2016/07/11 06:54:04 Done.
+ assert_selection(
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">bar</div>',
+ selection => {
+ var div = selection.document.getElementById('div');
yosin_UTC9 2016/07/08 09:30:50 Please add assert_not_undefined(window.eventSende
yoichio 2016/07/11 06:54:04 Done.
+ eventSender.mouseMoveTo(selection.document.offsetLeft + div.offsetLeft + 5, selection.document.offsetTop + div.offsetTop + 5);
+ eventSender.mouseDown();
+ eventSender.mouseUp();
+ eventSender.leapForward(100);
+ },
+ '<div contenteditable="true" id="div" style="-webkit-user-select:all">|bar</div>');
+ },
+'User can focus in -webkit-user-select:all contenteditable element');
+</script>
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/dom/Node.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698