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

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

Issue 2331253002: Use category flush for focus events. (Closed)
Patch Set: fix test Created 4 years, 3 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
« no previous file with comments | « no previous file | chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
index c464f1cfb76d8316bb2ba55d335da481d447daad..70a50675ccda9d2187f05b1a7033998d2ec2dd15 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
@@ -127,10 +127,10 @@ Background = function() {
cvox.ExtensionBridge.addMessageListener(this.onMessage_);
- /** @type {!BackgroundKeyboardHandler} */
+ /** @type {!BackgroundKeyboardHandler} @private */
this.keyboardHandler_ = new BackgroundKeyboardHandler();
- /** @type {!LiveRegions} */
+ /** @type {!LiveRegions} @private */
this.liveRegions_ = new LiveRegions(this);
/** @type {boolean} @private */
@@ -139,6 +139,7 @@ Background = function() {
/**
* Stores the mode as computed the last time a current range was set.
* @type {?ChromeVoxMode}
+ * @private
*/
this.mode_ = null;
@@ -519,6 +520,7 @@ Background.prototype = {
* 1. a url is blacklisted for Classic.
* 2. the current range is not within web content.
* @param {string} url
+ * @return {boolean}
*/
isWhitelistedForCompat_: function(url) {
return this.isBlacklistedForClassic_(url) || (this.getCurrentRange() &&
@@ -626,7 +628,7 @@ Background.prototype = {
} else if (action == 'onCommand') {
CommandHandler.onCommand(msg['command']);
} else if (action == 'flushNextUtterance') {
- Output.flushNextSpeechUtterance();
+ Output.forceModeForNextSpeechUtterance(cvox.QueueMode.FLUSH);
}
break;
}
« no previous file with comments | « no previous file | chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698