Hotword Audio Verification app: control the hotword settings

chrome/browser/extensions/api/hotword_private/hotword_private_api.cc

Index: chrome/browser/extensions/api/hotword_private/hotword_private_api.cc
diff --git a/chrome/browser/extensions/api/hotword_private/hotword_private_api.cc b/chrome/browser/extensions/api/hotword_private/hotword_private_api.cc
index 9f5b89705f74385ec6ecd25d32f4e224cc08ccc4..fb5ee50f856fc85f6065fb771ddad289c42c332e 100644
--- a/chrome/browser/extensions/api/hotword_private/hotword_private_api.cc
+++ b/chrome/browser/extensions/api/hotword_private/hotword_private_api.cc
@@ -93,11 +93,23 @@ bool HotwordPrivateSetAudioLoggingEnabledFunction::RunSync() {
       api::hotword_private::SetEnabled::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get());
 
+  // TODO(kcarattini): Sync the chrome pref with the account-level
+  // Audio History setting.
   PrefService* prefs = GetProfile()->GetPrefs();
   prefs->SetBoolean(prefs::kHotwordAudioLoggingEnabled, params->state);
   return true;
 }
 
+bool HotwordPrivateSetHotwordAlwaysOnSearchEnabledFunction::RunSync() {
+  scoped_ptr<api::hotword_private::SetEnabled::Params> params(

[rpetterson, 2014/09/11 20:14:17]

This should be:
scoped_ptr<api::hotword_private::SetHotwordAlwaysOnSearchEnabled::Params> params

And if you could also fix it in SetAudioLogging (which was my bad) that would be
super. :)

[kcarattini, 2014/09/12 01:02:07]

Done.

+      api::hotword_private::SetEnabled::Params::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());
+
+  PrefService* prefs = GetProfile()->GetPrefs();
+  prefs->SetBoolean(prefs::kHotwordAlwaysOnSearchEnabled, params->state);
+  return true;
+}
+
 bool HotwordPrivateGetStatusFunction::RunSync() {
   api::hotword_private::StatusDetails result;