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

Unified Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js

Issue 2059913002: Remove keyIdentifier usage in chrome/browser/resources/* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix chromeos build 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: chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js
index 1c5f122f78e8eebcf416591605c6fa38f1ec3eaa..b525f444a78109543e0277ba061cc590cd5c99c2 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js
@@ -494,24 +494,24 @@ Panel.onKeyDown = function(event) {
if (event.altKey || event.ctrlKey || event.metaKey || event.shiftKey)
return;
- switch (event.keyIdentifier) {
- case 'Left':
+ switch (event.key) {
+ case 'ArrowLeft':
Panel.advanceActiveMenuBy(-1);
break;
- case 'Right':
+ case 'ArrowRight':
Panel.advanceActiveMenuBy(1);
break;
- case 'Up':
+ case 'ArrowUp':
Panel.advanceItemBy(-1);
break;
- case 'Down':
+ case 'ArrowDown':
Panel.advanceItemBy(1);
break;
- case 'U+001B': // Escape
+ case 'Escape':
Panel.closeMenusAndRestoreFocus();
break;
- case 'Enter': // Enter
- case 'U+0020': // Space
+ case 'Enter':
+ case ' ': // Space
Panel.pendingCallback_ = Panel.getCallbackForCurrentItem();
Panel.closeMenusAndRestoreFocus();
break;

Powered by Google App Engine
This is Rietveld 408576698