Chromium Code Reviews| Index: chrome/browser/resources/hotword/always_on_manager.js |
| diff --git a/chrome/browser/resources/hotword/always_on_manager.js b/chrome/browser/resources/hotword/always_on_manager.js |
| index f99d284cadec1d14aa4de78dfff9ae43b608b778..b3c2266792dbb467f7b995b5112156416abc7508 100644 |
| --- a/chrome/browser/resources/hotword/always_on_manager.js |
| +++ b/chrome/browser/resources/hotword/always_on_manager.js |
| @@ -12,7 +12,7 @@ cr.define('hotword', function() { |
| * @param {!hotword.StateManager} stateManager |
| * @constructor |
| * @extends {hotword.BaseSessionManager} |
| - * @struct |
| + * @class |
|
Dan Beam
2014/10/21 03:07:39
why both @class and @constructor?
Anand Mistry (off Chromium)
2014/10/21 03:42:21
Dunno. I was following some other code which did t
|
| */ |
| function AlwaysOnManager(stateManager) { |
| hotword.BaseSessionManager.call(this, |
| @@ -23,12 +23,12 @@ cr.define('hotword', function() { |
| AlwaysOnManager.prototype = { |
| __proto__: hotword.BaseSessionManager.prototype, |
| - /** override */ |
| + /** @override */ |
| enabled: function() { |
| return this.stateManager.isAlwaysOnEnabled(); |
| }, |
| - /** override */ |
| + /** @override */ |
| updateListeners: function() { |
| hotword.BaseSessionManager.prototype.updateListeners.call(this); |
| if (this.enabled()) |