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

Unified Diff: LayoutTests/editing/undo/undo-forward-delete.html

Issue 43523002: Make tests related to undo word deletion platform independent (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: All tests modified Created 7 years, 2 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
Index: LayoutTests/editing/undo/undo-forward-delete.html
diff --git a/LayoutTests/editing/undo/undo-forward-delete.html b/LayoutTests/editing/undo/undo-forward-delete.html
index 20d937f39dc863995f8601b0cad0f4cfb223bad3..93038c95ee84e46b0254373ce0959c63166c805b 100644
--- a/LayoutTests/editing/undo/undo-forward-delete.html
+++ b/LayoutTests/editing/undo/undo-forward-delete.html
@@ -1,40 +1,45 @@
-<html>
-<head>
-
-<style>
-.editing {
- border: 2px solid red;
- padding: 12px;
- font-size: 24px;
-}
-</style>
-<script src="../editing.js" language="JavaScript" type="text/JavaScript" ></script>
-
+<!DOCTYPE html>
+<html>
+<body>
+<div id="container">
+<p id="description"></p>
+<p>To test manually, place the cursor before 'w' of 'word' and delete it completely character by character. Do ctrl+z. On Mac, 'word' should be selected. On other platforms 'word' should not be selected and the cursor should be placed before 'w' of 'word'.</p>
+<div id="test" style="border: 2px solid red;" contenteditable>This word should be selected.</div>
+</div>
+<div id="console"></div>
+<script src="../../fast/js/resources/js-test-pre.js"></script>
<script>
+description('Verifies the selection behavior on undoing a text deletion.');
+var selectionNode = document.getElementById('test').firstChild;
+var selectionOffset = selectionNode.data.indexOf(' ') + 1;
+var endOffset = selectionNode.data.indexOf('d') + 1;
+var selection = window.getSelection();
-function editingTest() {
- moveSelectionForwardByWordCommand();
- moveSelectionForwardByCharacterCommand();
- forwardDeleteCommand();
- forwardDeleteCommand();
- forwardDeleteCommand();
- forwardDeleteCommand();
- undoCommand();
-}
+function undoTest(platform, expectedStartNode, expectedStartOffset, expectedEndNode, expectedEndOffset, selectedText) {
+ debug(platform);
+ internals.settings.setEditingBehavior(platform);
-</script>
+ selection.collapse(selectionNode, selectionOffset);
+ for (var i = 0; i < 4; i++)
+ document.execCommand('forwarddelete');
+ document.execCommand('undo');
-</head>
-<body>
-<div contenteditable id="root" class="editing">
-<span id="test">This word should be selected, since the test deleted it a character at a time and then did an undo.</span>
-</div>
+ shouldBeEqualToString('selection.anchorNode.data', expectedStartNode.data);
+ shouldBe('selection.anchorOffset', expectedStartOffset + '');
+ shouldBeEqualToString('selection.focusNode.data', expectedEndNode.data);
+ shouldBe('selection.focusOffset', expectedEndOffset + '');
+ shouldBeEqualToString('selection.toString()', selectedText);
+}
-<script>
-if (window.internals)
- internals.settings.setEditingBehavior("mac");
-runEditingTest();
+if (window.internals) {
+ undoTest('mac', selectionNode, selectionOffset, selectionNode, endOffset, 'word');
+ undoTest('win', selectionNode, selectionOffset, selectionNode, endOffset, 'word');
+ undoTest('unix', selectionNode, selectionOffset, selectionNode, endOffset, 'word');
+ undoTest('android', selectionNode, selectionOffset, selectionNode, endOffset, 'word');
+}
+if (window.testRunner)
+ document.getElementById('container').outerHTML = '';
</script>
-
+<script src="../../fast/js/resources/js-test-post.js"></script>
</body>
</html>
« no previous file with comments | « LayoutTests/editing/undo/undo-delete-expected.txt ('k') | LayoutTests/editing/undo/undo-forward-delete-boundary.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698