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

Unified Diff: third_party/WebKit/LayoutTests/fast/events/menu-key-context-menu-document-pinch-zoom.html

Issue 2100243002: Remove non-standardize key code names from event_sender. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix inspector tests that were missed by sed Created 4 years, 6 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: third_party/WebKit/LayoutTests/fast/events/menu-key-context-menu-document-pinch-zoom.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/menu-key-context-menu-document-pinch-zoom.html b/third_party/WebKit/LayoutTests/fast/events/menu-key-context-menu-document-pinch-zoom.html
index bb9e83d950b17336c5c875920ff0884bac806def..0317b6ea21196b4e209ef37a753d83321e700e45 100644
--- a/third_party/WebKit/LayoutTests/fast/events/menu-key-context-menu-document-pinch-zoom.html
+++ b/third_party/WebKit/LayoutTests/fast/events/menu-key-context-menu-document-pinch-zoom.html
@@ -89,23 +89,23 @@ var runTest = function() {
// With nothing focused, the context menu should appear in the top left
// corner of the viewport.
document.addEventListener('contextmenu', handleContextMenuNoFocus, true);
- eventSender.keyDown("menu");
- eventSender.keyDown("escape");
+ eventSender.keyDown("ContextMenu");
+ eventSender.keyDown("Escape");
document.removeEventListener('contextmenu', handleContextMenuNoFocus, true);
// Focus the anchor, make sure the client coordinates and screen coordinates
// take the viewport's position and scale into account.
anchor.focus();
document.addEventListener('contextmenu', handleContextMenuFocus, true);
- eventSender.keyDown("menu");
- eventSender.keyDown("escape");
+ eventSender.keyDown("ContextMenu");
+ eventSender.keyDown("Escape");
document.removeEventListener('contextmenu', handleContextMenuFocus, true);
// Place selection and make sure it still works.
window.getSelection().selectAllChildren(anchor);
document.addEventListener('contextmenu', handleContextMenuEditable, true);
- eventSender.keyDown("menu");
- eventSender.keyDown("escape");
+ eventSender.keyDown("ContextMenu");
+ eventSender.keyDown("Escape");
document.removeEventListener('contextmenu', handleContextMenuEditable, true);
testRunner.notifyDone();

Powered by Google App Engine
This is Rietveld 408576698