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

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

Issue 2480913002: Fix a variety of panel issues (Closed)
Patch Set: Fix a variety of panel issues Created 4 years, 1 month 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 ChromeVox commands. 6 * @fileoverview ChromeVox commands.
7 */ 7 */
8 8
9 goog.provide('CommandHandler'); 9 goog.provide('CommandHandler');
10 10
(...skipping 162 matching lines...) Expand 10 before | Expand all | Expand 10 after
173 !cvox.BrailleCaptionsBackground.isEnabled()); 173 !cvox.BrailleCaptionsBackground.isEnabled());
174 return false; 174 return false;
175 case 'toggleChromeVoxVersion': 175 case 'toggleChromeVoxVersion':
176 if (!ChromeVoxState.instance.toggleNext()) 176 if (!ChromeVoxState.instance.toggleNext())
177 return false; 177 return false;
178 if (ChromeVoxState.instance.currentRange) { 178 if (ChromeVoxState.instance.currentRange) {
179 ChromeVoxState.instance.navigateToRange( 179 ChromeVoxState.instance.navigateToRange(
180 ChromeVoxState.instance.currentRange); 180 ChromeVoxState.instance.currentRange);
181 } 181 }
182 break; 182 break;
183 case 'help':
183 case 'showNextUpdatePage': 184 case 'showNextUpdatePage':
dmazzoni 2016/11/07 07:46:01 Should we have two commands that do the same thing
David Tseng 2016/11/07 19:43:29 Done.
184 (new PanelCommand(PanelCommandType.TUTORIAL)).send(); 185 (new PanelCommand(PanelCommandType.TUTORIAL)).send();
185 return false; 186 return false;
186 default: 187 default:
187 break; 188 break;
188 } 189 }
189 190
190 // Require a current range. 191 // Require a current range.
191 if (!ChromeVoxState.instance.currentRange_) 192 if (!ChromeVoxState.instance.currentRange_)
192 return true; 193 return true;
193 194
(...skipping 543 matching lines...) Expand 10 before | Expand all | Expand 10 after
737 break; 738 break;
738 } 739 }
739 if (announcement) { 740 if (announcement) {
740 cvox.ChromeVox.tts.speak( 741 cvox.ChromeVox.tts.speak(
741 announcement, cvox.QueueMode.FLUSH, 742 announcement, cvox.QueueMode.FLUSH,
742 cvox.AbstractTts.PERSONALITY_ANNOTATION); 743 cvox.AbstractTts.PERSONALITY_ANNOTATION);
743 } 744 }
744 }; 745 };
745 746
746 }); // goog.scope 747 }); // goog.scope
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698