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

Unified Diff: chrome/browser/resources/chromeos/chromevox/extensions/searchvox/search.js

Issue 604423002: Use an enum for ChromeVox queue mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 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
Index: chrome/browser/resources/chromeos/chromevox/extensions/searchvox/search.js
diff --git a/chrome/browser/resources/chromeos/chromevox/extensions/searchvox/search.js b/chrome/browser/resources/chromeos/chromevox/extensions/searchvox/search.js
index 7ef10ea88dc18b3276f8b2b9ba4decb9d7659984..4b53144adee34d58dc0e726e9510b4231cd63b5a 100644
--- a/chrome/browser/resources/chromeos/chromevox/extensions/searchvox/search.js
+++ b/chrome/browser/resources/chromeos/chromevox/extensions/searchvox/search.js
@@ -102,9 +102,9 @@ cvox.Search.syncToIndex = function() {
cvox.ChromeVox.tts.stop();
var prop = { endCallback: cvox.Search.speakSync_ };
if (cvox.Search.index === 0) {
- cvox.ChromeVox.tts.speak('First result', 1, prop);
+ cvox.ChromeVox.tts.speak('First result', cvox.QueueMode.QUEUE, prop);
} else if (cvox.Search.index === cvox.Search.results.length - 1) {
- cvox.ChromeVox.tts.speak('Last result', 1, prop);
+ cvox.ChromeVox.tts.speak('Last result', cvox.QueueMode.QUEUE, prop);
} else {
cvox.Search.speakSync_();
}
@@ -173,9 +173,10 @@ cvox.Search.navigatePage = function(next) {
if (url) {
var pageNumber = cvox.Search.getPageNumber(url);
if (!isNaN(pageNumber)) {
- cvox.ChromeVox.tts.speak('Page ' + pageNumber, 0, prop);
+ cvox.ChromeVox.tts.speak('Page ' + pageNumber, cvox.QueueMode.FLUSH,
+ prop);
} else {
- cvox.ChromeVox.tts.speak('Unknown page.', 0, prop);
+ cvox.ChromeVox.tts.speak('Unknown page.', cvox.QueueMode.FLUSH, prop);
}
}
};
@@ -186,11 +187,12 @@ cvox.Search.navigatePage = function(next) {
cvox.Search.goToPane = function() {
var pane = cvox.Search.panes[cvox.Search.paneIndex];
if (pane.className === cvox.Search.SELECTED_PANE_CLASS) {
- cvox.ChromeVox.tts.speak('You are already on that page.');
+ cvox.ChromeVox.tts.speak('You are already on that page.',
+ cvox.QueueMode.QUEUE);
return;
}
var anchor = pane.querySelector('a');
- cvox.ChromeVox.tts.speak(anchor.textContent);
+ cvox.ChromeVox.tts.speak(anchor.textContent, cvox.QueueMode.QUEUE);
var url = cvox.SearchUtil.extractURL(pane);
if (url) {
window.location = url;

Powered by Google App Engine
This is Rietveld 408576698