Enable HTTP-bad Phase 1 by default except iOS

Enable HTTP-bad Phase 1 by default except iOS

This feature is launching in Chrome 56 on all platforms except iOS.

BUG=657143
Review-Url: https://codereview.chromium.org/2635423002
Cr-Commit-Position: refs/heads/master@{#444945}
Committed: https://chromium.googlesource.com/chromium/src/+/bcfa0007f638e46826cf0803746dfd90c53d6fc4

[estark, 2017-01-18 02:23:09]

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

[commit-bot: I haz the power, 2017-01-18 02:23:31]

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

[estark, 2017-01-18 02:24:03]

estark@chromium.org changed reviewers:
+ lgarron@chromium.org

[estark, 2017-01-18 02:24:04]

lgarron: take 2, this time just enabling the feature by default on non-iOS

[commit-bot: I haz the power, 2017-01-18 02:54:51]

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

[commit-bot: I haz the power, 2017-01-18 02:54:52]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[estark, 2017-01-18 03:41:07]

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

[commit-bot: I haz the power, 2017-01-18 03:41:29]

Dry run: 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, 2017-01-18 04:15:21]

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

[commit-bot: I haz the power, 2017-01-18 04:15:22]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[estark, 2017-01-18 04:19:01]

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

[commit-bot: I haz the power, 2017-01-18 04:19:17]

Dry run: 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, 2017-01-18 05:11:50]

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

[commit-bot: I haz the power, 2017-01-18 05:11:51]

Dry run: This issue passed the CQ dry run.

[estark, 2017-01-19 05:38:33]

On 2017/01/18 02:24:04, estark wrote:
> lgarron: take 2, this time just enabling the feature by default on non-iOS

Friendly ping

[lgarron, 2017-01-19 22:05:53]

LGTM for the code change.

Why remove the tests? Because you'd have to change them to handle the ifdef-ed
default values?

[estark, 2017-01-19 22:07:03]

On 2017/01/19 22:05:53, lgarron wrote:
> LGTM for the code change.
> 
> Why remove the tests? Because you'd have to change them to handle the ifdef-ed
> default values?

Yeah, I don't think it's worth updating them when we're going to remove them
imminently anyway (once iOS is launched).

[estark, 2017-01-19 22:15:47]

The CQ bit was checked by estark@chromium.org

[commit-bot: I haz the power, 2017-01-19 22:16:17]

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, 2017-01-19 23:15:28]

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

[commit-bot: I haz the power, 2017-01-19 23:15:29]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[estark, 2017-01-19 23:37:13]

The CQ bit was checked by estark@chromium.org

[commit-bot: I haz the power, 2017-01-19 23:38:13]

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, 2017-01-20 01:59:48]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484869033487620,
"parent_rev": "6e463a0dac6c782c5054432807feacf3dc7e6e2d", "commit_rev":
"bcfa0007f638e46826cf0803746dfd90c53d6fc4"}

[commit-bot: I haz the power, 2017-01-20 02:00:17]

Description was changed from

==========
Enable HTTP-bad Phase 1 by default except iOS

This feature is launching in Chrome 56 on all platforms except iOS.

BUG=657143
==========

to

==========
Enable HTTP-bad Phase 1 by default except iOS

This feature is launching in Chrome 56 on all platforms except iOS.

BUG=657143

Review-Url: https://codereview.chromium.org/2635423002
Cr-Commit-Position: refs/heads/master@{#444945}
Committed:
https://chromium.googlesource.com/chromium/src/+/bcfa0007f638e46826cf0803746d...
==========

[commit-bot: I haz the power, 2017-01-20 02:00:18]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/bcfa0007f638e46826cf0803746d...