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

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/base_automation_handler.js

Issue 1693713004: Don't fire a FOCUS accessibility event on a View that doesn't have focus. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Missed one call to SendFocusAlert Created 4 years, 10 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
« no previous file with comments | « ash/wm/overview/window_selector_item.cc ('k') | chrome/browser/ui/views/autofill/password_generation_popup_view_views.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 /** 5 /**
6 * @fileoverview Basic facillities to handle events from a single automation 6 * @fileoverview Basic facillities to handle events from a single automation
7 * node. 7 * node.
8 */ 8 */
9 9
10 goog.provide('BaseAutomationHandler'); 10 goog.provide('BaseAutomationHandler');
(...skipping 17 matching lines...) Expand all
28 * Maps an automation event to its listener. 28 * Maps an automation event to its listener.
29 * @type {!Object<EventType, function(!AutomationEvent) : void>} 29 * @type {!Object<EventType, function(!AutomationEvent) : void>}
30 */ 30 */
31 this.listenerMap_ = { 31 this.listenerMap_ = {
32 alert: this.onAlert, 32 alert: this.onAlert,
33 focus: this.onFocus, 33 focus: this.onFocus,
34 hover: this.onEventDefault, 34 hover: this.onEventDefault,
35 loadComplete: this.onLoadComplete, 35 loadComplete: this.onLoadComplete,
36 menuStart: this.onEventDefault, 36 menuStart: this.onEventDefault,
37 menuEnd: this.onEventDefault, 37 menuEnd: this.onEventDefault,
38 selection: this.onEventDefault,
38 scrollPositionChanged: this.onScrollPositionChanged, 39 scrollPositionChanged: this.onScrollPositionChanged,
39 textChanged: this.onTextChanged, 40 textChanged: this.onTextChanged,
40 textSelectionChanged: this.onTextSelectionChanged, 41 textSelectionChanged: this.onTextSelectionChanged,
41 valueChanged: this.onValueChanged 42 valueChanged: this.onValueChanged
42 }; 43 };
43 44
44 /** @type {!Object<string, function(!AutomationEvent): void>} @private */ 45 /** @type {!Object<string, function(!AutomationEvent): void>} @private */
45 this.listeners_ = {}; 46 this.listeners_ = {};
46 47
47 this.register_(); 48 this.register_();
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 */ 137 */
137 onTextSelectionChanged: function(evt) {}, 138 onTextSelectionChanged: function(evt) {},
138 139
139 /** 140 /**
140 * @param {!AutomationEvent} evt 141 * @param {!AutomationEvent} evt
141 */ 142 */
142 onValueChanged: function(evt) {} 143 onValueChanged: function(evt) {}
143 }; 144 };
144 145
145 }); // goog.scope 146 }); // goog.scope
OLDNEW
« no previous file with comments | « ash/wm/overview/window_selector_item.cc ('k') | chrome/browser/ui/views/autofill/password_generation_popup_view_views.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698