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

Unified Diff: chrome/browser/resources/chromeos/chromevox/host/chrome/earcons.js

Issue 1319093003: Use new earcons in ChromeVox Next. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@automation_node_id_fix_2
Patch Set: Rebase Created 5 years, 2 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/host/chrome/earcons.js
diff --git a/chrome/browser/resources/chromeos/chromevox/host/chrome/earcons.js b/chrome/browser/resources/chromeos/chromevox/host/chrome/earcons.js
index c250e8c3967780a89d5c27b8c16d18c2da8b9211..5ac0f81479783ac1bf707082c750e1b0fcc31e43 100644
--- a/chrome/browser/resources/chromeos/chromevox/host/chrome/earcons.js
+++ b/chrome/browser/resources/chromeos/chromevox/host/chrome/earcons.js
@@ -31,7 +31,7 @@ goog.inherits(cvox.ChromeEarcons, cvox.AbstractEarcons);
*/
cvox.ChromeEarcons.prototype.playEarcon = function(earcon) {
goog.base(this, 'playEarcon', earcon);
- if (!this.enabled) {
+ if (!cvox.AbstractEarcons.enabled) {
return;
}
@@ -51,9 +51,9 @@ cvox.ChromeEarcons.prototype.toggle = function() {
'target': 'Prefs',
'action': 'setPref',
'pref': 'earcons',
- 'value': this.enabled
+ 'value': cvox.AbstractEarcons.enabled
});
- if (!this.enabled) {
+ if (!cvox.AbstractEarcons.enabled) {
cvox.ChromeVox.host.sendToBackgroundPage({
'target': 'Prefs',
'action': 'setPref',
@@ -61,8 +61,11 @@ cvox.ChromeEarcons.prototype.toggle = function() {
'value': true
});
}
- return this.enabled;
+ return cvox.AbstractEarcons.enabled;
};
+/**
+ * @override
+ */
cvox.HostFactory.earconsConstructor = cvox.ChromeEarcons;

Powered by Google App Engine
This is Rietveld 408576698