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

Unified Diff: LayoutTests/editing/spelling/spelling-exactly-selected-word.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-exactly-selected-word.html
diff --git a/LayoutTests/editing/spelling/spelling-exactly-selected-word.html b/LayoutTests/editing/spelling/spelling-exactly-selected-word.html
index 72a1a251246d9fc931d107145c4a638d79094508..13ad582f2e4e7ee8d8a6675c4ff3344929da5bc3 100644
--- a/LayoutTests/editing/spelling/spelling-exactly-selected-word.html
+++ b/LayoutTests/editing/spelling/spelling-exactly-selected-word.html
@@ -35,7 +35,7 @@ initSpellTest("destination", "wellcome home.", 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);
spellingMarkerRange = internals.markerRangeForNode(textNode, "spelling", 0);
shouldBeEqualToString("spellingMarkerRange.toString()", "wellcome");
@@ -55,7 +55,7 @@ initSpellTest("destination", "wellcome home.", 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);
shouldBeEqualToString("window.getSelection().toString()", "home");
shouldBeEqualToString("contextMenuElements[contextMenuElements.length - 1]", "<separator>");

Powered by Google App Engine
This is Rietveld 408576698