[Hotword] Update helper extension to listen for change to audio logging preference.

[Hotword] Update helper extension to listen for change to audio logging preference.
Helper extension then notifies chrome and tells the extension to turn on/off.

BUG=345811
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260195

[saurya1, 2014-03-27 16:29:22]

https://codereview.chromium.org/212253003/diff/20001/chrome/browser/resources...
File chrome/browser/resources/hotword_helper/manager.js (right):

https://codereview.chromium.org/212253003/diff/20001/chrome/browser/resources...
chrome/browser/resources/hotword_helper/manager.js:171: {'cmd':
OptInManager.CommandFromHelper.AUDIO_LOGGING_OFF});
AUDIO_LOGGING_ON

https://codereview.chromium.org/212253003/diff/20001/chrome/browser/resources...
File chrome/browser/resources/hotword_helper/optin_client.js (right):

https://codereview.chromium.org/212253003/diff/20001/chrome/browser/resources...
chrome/browser/resources/hotword_helper/optin_client.js:79: {'type': command});
If you want to clean this up you can just remove the
chrome.runtime.sendMessage() and breaks above and have a default case that just
passes on the message.

[rpetterson, 2014-03-27 16:37:18]

https://codereview.chromium.org/212253003/diff/20001/chrome/browser/resources...
File chrome/browser/resources/hotword_helper/manager.js (right):

https://codereview.chromium.org/212253003/diff/20001/chrome/browser/resources...
chrome/browser/resources/hotword_helper/manager.js:171: {'cmd':
OptInManager.CommandFromHelper.AUDIO_LOGGING_OFF});
On 2014/03/27 16:29:22, saurya1 wrote:
> AUDIO_LOGGING_ON

Derp. Copy/paste. Fixed.

https://codereview.chromium.org/212253003/diff/20001/chrome/browser/resources...
File chrome/browser/resources/hotword_helper/optin_client.js (right):

https://codereview.chromium.org/212253003/diff/20001/chrome/browser/resources...
chrome/browser/resources/hotword_helper/optin_client.js:79: {'type': command});
I assume I don't actually need the switch statement anymore? I think I've
cleaned it up, but PTAL.

[saurya1, 2014-03-27 21:55:10]

lgtm

[rpetterson, 2014-03-27 21:56:24]

For owners approval.

[rpetterson, 2014-03-27 21:57:04]

Actually adding James for owners approval this time.

[James Hawkins, 2014-03-27 21:58:54]

lgtm

[rpetterson, 2014-03-28 00:41:13]

The CQ bit was checked by rlp@chromium.org

[commit-bot: I haz the power, 2014-03-28 00:43:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rlp@chromium.org/212253003/40001

[commit-bot: I haz the power, 2014-03-28 02:53:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-28 02:53:57]

Retried try job too often on win_rel for step(s) app_list_unittests,
ash_unittests, aura_unittests, base_unittests, browser_tests,
cacheinvalidation_unittests, cc_unittests, check_deps, check_deps2git,
chrome_elf_unittests, chromedriver_unittests, components_unittests,
compositor_unittests, content_browsertests, content_unittests, crypto_unittests,
events_unittests, gfx_unittests, google_apis_unittests, gpu_unittests,
installer_util_unittests, interactive_ui_tests, ipc_tests, jingle_unittests,
media_unittests, mini_installer_test, nacl_integration, net_unittests,
ppapi_unittests, printing_unittests, remoting_unittests, sql_unittests,
sync_integration_tests, sync_unit_tests, telemetry_perf_unittests,
telemetry_unittests, unit_tests, views_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[rpetterson, 2014-03-28 03:34:52]

The CQ bit was checked by rlp@chromium.org

[commit-bot: I haz the power, 2014-03-28 03:34:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rlp@chromium.org/212253003/40001

[commit-bot: I haz the power, 2014-03-28 04:35:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-28 04:35:07]

Retried try job too often on ios_dbg_simulator for step(s) ui_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[rpetterson, 2014-03-28 05:04:24]

The CQ bit was checked by rlp@chromium.org

[commit-bot: I haz the power, 2014-03-28 05:06:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rlp@chromium.org/212253003/40001

[commit-bot: I haz the power, 2014-03-28 09:28:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-28 09:28:05]

Retried try job too often on ios_dbg_simulator for step(s) components_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[rpetterson, 2014-03-28 15:32:05]

The CQ bit was checked by rlp@chromium.org

[commit-bot: I haz the power, 2014-03-28 15:33:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rlp@chromium.org/212253003/40001

[commit-bot: I haz the power, 2014-03-28 18:08:59]

Change committed as 260195