Only disable hotwording if the VoiceTrigger field trial is set to 'Disabled'.

Only disable hotwording if the VoiceTrigger field trial is set to 'Disabled'.

Also, delete the 'Experimental' field trial group since it was never
used.

BUG=465174
Committed: https://crrev.com/6b006fd3f1e00e4920ed22755edce6b9ce64f024
Cr-Commit-Position: refs/heads/master@{#320824}

[Anand Mistry (off Chromium), 2015-03-16 03:01:01]

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

[kcarattini, 2015-03-16 03:24:21]

lgtm

[Anand Mistry (off Chromium), 2015-03-16 03:30:25]

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

[Anand Mistry (off Chromium), 2015-03-16 03:30:25]

thestig@chromium.org: For OWNERS.

[Lei Zhang, 2015-03-16 21:48:30]

lgtm

Drive by comment below. Feel free to address separately.

https://codereview.chromium.org/1008303002/diff/1/chrome/browser/search/hotwo...
File chrome/browser/search/hotword_service_unittest.cc (right):

https://codereview.chromium.org/1008303002/diff/1/chrome/browser/search/hotwo...
chrome/browser/search/hotword_service_unittest.cc:265:
EXPECT_TRUE(hotword_service != NULL);
BTW, these should be ASSERT_TRUE(), otherwise the entire unit_tests binary
crashes if you dereference NULL on the next line.

[Anand Mistry (off Chromium), 2015-03-16 22:27:28]

https://codereview.chromium.org/1008303002/diff/1/chrome/browser/search/hotwo...
File chrome/browser/search/hotword_service_unittest.cc (right):

https://codereview.chromium.org/1008303002/diff/1/chrome/browser/search/hotwo...
chrome/browser/search/hotword_service_unittest.cc:265:
EXPECT_TRUE(hotword_service != NULL);
On 2015/03/16 21:48:30, Lei Zhang wrote:
> BTW, these should be ASSERT_TRUE(), otherwise the entire unit_tests binary
> crashes if you dereference NULL on the next line.

Done.

[Anand Mistry (off Chromium), 2015-03-16 22:28:26]

The CQ bit was checked by amistry@chromium.org

[Anand Mistry (off Chromium), 2015-03-16 22:28:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
thestig@chromium.org, kcarattini@chromium.org
Link to the patchset: https://codereview.chromium.org/1008303002/#ps20001
(title: "Change some EXPECTs to ASSERTs.")

[commit-bot: I haz the power, 2015-03-16 22:29:11]

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

[commit-bot: I haz the power, 2015-03-16 23:44:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-16 23:45:07]

Patchset 2 (id:??) landed as
https://crrev.com/6b006fd3f1e00e4920ed22755edce6b9ce64f024
Cr-Commit-Position: refs/heads/master@{#320824}