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

Side by Side Diff: chrome/browser/resources/chromeos/switch_access/keyboard_handler.js

Issue 2905373002: Set keys for SwitchAccessEventHandler to capture using accessibiltyPrivate API (Closed)
Patch Set: Responded to comment Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // TODO(elichtenberg): Move into custom logger class or somewhere else. 5 // TODO(elichtenberg): Move into custom logger class or somewhere else.
6 let debuggingEnabled = true; 6 let debuggingEnabled = true;
7 7
8 /** 8 /**
9 * Class to handle keyboard input. 9 * Class to handle keyboard input.
10 * 10 *
(...skipping 10 matching lines...) Expand all
21 this.init_(); 21 this.init_();
22 } 22 }
23 23
24 KeyboardHandler.prototype = { 24 KeyboardHandler.prototype = {
25 /** 25 /**
26 * Set up key listener. 26 * Set up key listener.
27 * 27 *
28 * @private 28 * @private
29 */ 29 */
30 init_: function() { 30 init_: function() {
31 // Capture keycodes for keys 1 through 4, and 6 through 9.
32 chrome.accessibilityPrivate.setSwitchAccessKeys(
33 ['1'.charCodeAt(0), '2'.charCodeAt(0), '3'.charCodeAt(0),
Devlin 2017/05/30 19:22:03 optional: to me, this would potentially be more re
elichtenberg 2017/05/31 00:58:43 Good idea. I changed it to an arrow function too,
34 '4'.charCodeAt(0), '6'.charCodeAt(0), '7'.charCodeAt(0),
35 '8'.charCodeAt(0), '9'.charCodeAt(0)]);
31 document.addEventListener('keyup', this.handleKeyEvent_.bind(this)); 36 document.addEventListener('keyup', this.handleKeyEvent_.bind(this));
32 }, 37 },
33 38
34 /** 39 /**
35 * Handle a keyboard event by calling the appropriate SwitchAccess functions. 40 * Handle a keyboard event by calling the appropriate SwitchAccess functions.
36 * 41 *
37 * @param {!Event} event 42 * @param {!Event} event
38 * @private 43 * @private
39 */ 44 */
40 handleKeyEvent_: function(event) { 45 handleKeyEvent_: function(event) {
(...skipping 30 matching lines...) Expand all
71 break; 76 break;
72 case '9': 77 case '9':
73 console.log('9 = go to parent element (debug mode)'); 78 console.log('9 = go to parent element (debug mode)');
74 this.switchAccess_.debugMoveToParent(); 79 this.switchAccess_.debugMoveToParent();
75 break; 80 break;
76 } 81 }
77 } 82 }
78 this.switchAccess_.performedUserAction(); 83 this.switchAccess_.performedUserAction();
79 } 84 }
80 }; 85 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698