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

Unified Diff: LayoutTests/editing/spelling/spelling-changed-text.html

Issue 484353004: Test tidying: use "escape" and "backspace" with eventSender.keyDown() (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: More quoting consistency wibbles Created 6 years, 4 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/spelling/spelling-changed-text.html
diff --git a/LayoutTests/editing/spelling/spelling-changed-text.html b/LayoutTests/editing/spelling/spelling-changed-text.html
index 9583c305825cd22c63298b3de88b6b5a1691bd12..7c4b48e92bd8241f4217c4bc33a134bc66b1ba2e 100644
--- a/LayoutTests/editing/spelling/spelling-changed-text.html
+++ b/LayoutTests/editing/spelling/spelling-changed-text.html
@@ -24,7 +24,7 @@ initSpellTest("destination", "Spell wellcome. Is it broken?", function(textNode)
window.getSelection().addRange(deleteRange);
shouldBeEqualToString("window.getSelection().toString()", " Is it broken?");
// Del key to delete the text "Is it broken?".
- eventSender.keyDown(String.fromCharCode(0x007F), null);
+ eventSender.keyDown("delete", null);
// Context click to show the context menu.
var x = destination.offsetParent.offsetLeft + destination.offsetLeft + 50;
@@ -32,7 +32,7 @@ initSpellTest("destination", "Spell wellcome. Is it broken?", function(textNode)
eventSender.mouseMoveTo(x, y);
contextMenuElements = eventSender.contextClick();
// Esc key to hide the context menu.
- eventSender.keyDown(String.fromCharCode(0x001B), null);
+ eventSender.keyDown("escape", null);
document.getElementById("destination").innerHTML = "";
});
« no previous file with comments | « LayoutTests/editing/spelling/input-type-text.html ('k') | LayoutTests/editing/spelling/spelling-double-clicked-word.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698