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

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

Issue 1306773003: Make earcon ids strings instead of numbers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove_unused_earcons
Patch Set: Fixed test Created 5 years, 4 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 ac6a42924b9f5af4c7dd4cce1c44e0e6d2e6ca72..9c8175a6627750581e88b4991ffc31133897e1ab 100644
--- a/chrome/browser/resources/chromeos/chromevox/extensions/searchvox/search.js
+++ b/chrome/browser/resources/chromeos/chromevox/extensions/searchvox/search.js
@@ -238,7 +238,7 @@ cvox.Search.keyhandler = function(evt) {
cvox.Search.index = cvox.SearchUtil.subOneWrap(cvox.Search.index,
cvox.Search.results.length);
if (cvox.Search.index === cvox.Search.results.length - 1) {
- cvox.ChromeVox.earcons.playEarconByName('WRAP');
+ cvox.ChromeVox.earcons.playEarcon(cvox.Earcon.WRAP);
}
cvox.Search.syncToIndex();
break;
@@ -247,7 +247,7 @@ cvox.Search.keyhandler = function(evt) {
cvox.Search.index = cvox.SearchUtil.addOneWrap(cvox.Search.index,
cvox.Search.results.length);
if (cvox.Search.index === 0) {
- cvox.ChromeVox.earcons.playEarconByName('WRAP');
+ cvox.ChromeVox.earcons.playEarcon(cvox.Earcon.WRAP);
}
cvox.Search.syncToIndex();
break;

Powered by Google App Engine
This is Rietveld 408576698