Instant cleanup: Remove unused Android-only cmdline switches

Instant cleanup: Remove unused Android-only cmdline switches

These features were never launched on Android; no need to keep the switches around any longer.
The field trials remain (for now), so it's still possible to use the --force-fieldtrials param.

BUG=627747
Committed: https://crrev.com/db738db41ec50d536c7c4a70c994f6cabcffeb23
Cr-Commit-Position: refs/heads/master@{#433915}

[Marc Treib, 2016-11-18 09:50:01]

The CQ bit was checked by treib@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-18 09:50:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Marc Treib, 2016-11-18 10:49:28]

treib@chromium.org changed reviewers:
+ jered@chromium.org

[Marc Treib, 2016-11-18 10:49:28]

PTAL!

[commit-bot: I haz the power, 2016-11-18 10:56:28]

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

[commit-bot: I haz the power, 2016-11-18 10:56:29]

Dry run: This issue passed the CQ dry run.

[Marc Treib, 2016-11-22 10:15:39]

Ping!

[Jered, 2016-11-22 16:21:01]

On 2016/11/22 10:15:39, Marc Treib wrote:
> Ping!

lgtm

[Jered, 2016-11-22 16:21:13]

lgtm

https://codereview.chromium.org/2513913002/diff/1/components/search/search_an...
File components/search/search_android_unittest.cc (left):

https://codereview.chromium.org/2513913002/diff/1/components/search/search_an...
components/search/search_android_unittest.cc:18:
EXPECT_FALSE(IsInstantExtendedAPIEnabled());
I guess if we're going to bother with keeping L17, we may as well keep L18 too.

[Marc Treib, 2016-11-22 16:59:10]

The CQ bit was checked by treib@chromium.org to run a CQ dry run

[Marc Treib, 2016-11-22 16:59:15]

https://codereview.chromium.org/2513913002/diff/1/components/search/search_an...
File components/search/search_android_unittest.cc (left):

https://codereview.chromium.org/2513913002/diff/1/components/search/search_an...
components/search/search_android_unittest.cc:18:
EXPECT_FALSE(IsInstantExtendedAPIEnabled());
On 2016/11/22 16:21:13, Jered wrote:
> I guess if we're going to bother with keeping L17, we may as well keep L18
too.

Good catch! Done.

[commit-bot: I haz the power, 2016-11-22 16:59:25]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Marc Treib, 2016-11-22 17:47:04]

The CQ bit was unchecked by treib@chromium.org

[Marc Treib, 2016-11-22 17:47:04]

The CQ bit was checked by treib@chromium.org

[Marc Treib, 2016-11-22 17:47:05]

The patchset sent to the CQ was uploaded after l-g-t-m from jered@chromium.org
Link to the patchset: https://codereview.chromium.org/2513913002/#ps20001
(title: "review")

[commit-bot: I haz the power, 2016-11-22 17:47:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-22 18:15:30]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479836824895760,
"parent_rev": "5fb6a216bb1df1ba0c36c498814e43ec3e5e91c0", "commit_rev":
"ca3c4eda69e16c08bad7cb463929ea25175e3f86"}

[commit-bot: I haz the power, 2016-11-22 18:15:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-22 18:19:31]

Description was changed from

==========
Instant cleanup: Remove unused Android-only cmdline switches

These features were never launched on Android; no need to keep the switches
around any longer.
The field trials remain (for now), so it's still possible to use the
--force-fieldtrials param.

BUG=627747
==========

to

==========
Instant cleanup: Remove unused Android-only cmdline switches

These features were never launched on Android; no need to keep the switches
around any longer.
The field trials remain (for now), so it's still possible to use the
--force-fieldtrials param.

BUG=627747

Committed: https://crrev.com/db738db41ec50d536c7c4a70c994f6cabcffeb23
Cr-Commit-Position: refs/heads/master@{#433915}
==========

[commit-bot: I haz the power, 2016-11-22 18:19:32]

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