Disable old hotword extension when new hotwording is enabled.

Disable old hotword extension when new hotwording is enabled.

BUG=397019
Committed: https://crrev.com/26c09aac0e5c2018f232bb9d77105731034d0bc2
Cr-Commit-Position: refs/heads/master@{#300977}

[Anand Mistry (off Chromium), 2014-10-23 05:29:21]

amistry@chromium.org changed reviewers:
+ rlp@chromium.org

[rpetterson, 2014-10-23 18:41:17]

LGTM

I assume the new extension is enabled by default? What about the shared module?
From within the external_component_loader?

[Anand Mistry (off Chromium), 2014-10-23 21:29:44]

On 2014/10/23 18:41:17, rpetterson wrote:
> LGTM
> 
> I assume the new extension is enabled by default?
Yup, and never disabled. The edge cases are handled by unregistering from the
appropriate events, so we don't need to disable it.

> What about the shared module? From within the external_component_loader?
Installed by external_component_loader and never disabled. Also re-installed on
language change.

[Anand Mistry (off Chromium), 2014-10-23 21:29:50]

The CQ bit was checked by amistry@chromium.org

[commit-bot: I haz the power, 2014-10-23 21:31:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/671113002/1

[rpetterson, 2014-10-23 21:32:32]

On 2014/10/23 21:29:44, Anand Mistry wrote:
> On 2014/10/23 18:41:17, rpetterson wrote:
> > LGTM
> > 
> > I assume the new extension is enabled by default?
> Yup, and never disabled. The edge cases are handled by unregistering from the
> appropriate events, so we don't need to disable it.
> 
> > What about the shared module? From within the external_component_loader?
> Installed by external_component_loader and never disabled. Also re-installed
on
> language change.

Cool.

[commit-bot: I haz the power, 2014-10-23 23:26:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-23 23:27:25]

Patchset 1 (id:??) landed as
https://crrev.com/26c09aac0e5c2018f232bb9d77105731034d0bc2
Cr-Commit-Position: refs/heads/master@{#300977}