Enable HTTP/0.9 on non-standard ports by default

Enable HTTP/0.9 on non-standard ports by default

This is a temporary patch to enable HTTP/0.9 on non-standard ports by
default.  After merging it with Chrome 56, it will be reverted, in favor
of specifically adding detection of Shoutcast servers on HTTP ports
other than port 80.

BUG=669800
Review-Url: https://codereview.chromium.org/2651753002
Cr-Commit-Position: refs/heads/master@{#445743}
Committed: https://chromium.googlesource.com/chromium/src/+/2617c4a136497244376e1bcf11aa356afb2a1e06

[mmenke, 2017-01-23 17:54:11]

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

[commit-bot: I haz the power, 2017-01-23 17:54:47]

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-23 18:52:23]

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

[commit-bot: I haz the power, 2017-01-23 18:52:24]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[mmenke, 2017-01-23 20:34:11]

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

[commit-bot: I haz the power, 2017-01-23 20:34:48]

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-23 21:45:43]

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

[commit-bot: I haz the power, 2017-01-23 21:45:43]

Dry run: This issue passed the CQ dry run.

[mmenke, 2017-01-23 22:30:35]

mmenke@chromium.org changed reviewers:
+ eroman@chromium.org

[mmenke, 2017-01-23 22:30:36]

Eric:  I want to land this, merge it with M56, and revert it, so it's
deliberately pretty simplistic.

[mmenke, 2017-01-23 23:09:22]

Description was changed from

==========
ut # Enter a description of the change.
Enable HTTP/0.9 on non-standard ports by default

This is a temporary patch to enable HTTP/0.9 on non-standard ports by
default.  After merging it with Chrome 56, it will be reverted, in favor
of specifically adding detection of Shoutcast servers on HTTP ports
other than port 80.

BUG=869725
==========

to

==========
Enable HTTP/0.9 on non-standard ports by default

This is a temporary patch to enable HTTP/0.9 on non-standard ports by
default.  After merging it with Chrome 56, it will be reverted, in favor
of specifically adding detection of Shoutcast servers on HTTP ports
other than port 80.

BUG=869725
==========

[eroman, 2017-01-24 00:22:25]

LGTM.

Note that you could patch directly to the 56 branch

[mmenke, 2017-01-24 15:45:41]

On 2017/01/24 00:22:25, eroman (slow) wrote:
> LGTM.
> 
> Note that you could patch directly to the 56 branch

Given how close we are to release, was thinking they may prefer to have it on
trunk for a day or two before merge.  Guess there's no harm in asking.

[mmenke, 2017-01-24 15:46:02]

Description was changed from

==========
Enable HTTP/0.9 on non-standard ports by default

This is a temporary patch to enable HTTP/0.9 on non-standard ports by
default.  After merging it with Chrome 56, it will be reverted, in favor
of specifically adding detection of Shoutcast servers on HTTP ports
other than port 80.

BUG=869725
==========

to

==========
Enable HTTP/0.9 on non-standard ports by default

This is a temporary patch to enable HTTP/0.9 on non-standard ports by
default.  After merging it with Chrome 56, it will be reverted, in favor
of specifically adding detection of Shoutcast servers on HTTP ports
other than port 80.

BUG=669800
==========

[mmenke, 2017-01-24 15:49:26]

On 2017/01/24 15:45:41, mmenke wrote:
> On 2017/01/24 00:22:25, eroman (slow) wrote:
> > LGTM.
> > 
> > Note that you could patch directly to the 56 branch
> 
> Given how close we are to release, was thinking they may prefer to have it on
> trunk for a day or two before merge.  Guess there's no harm in asking.

Actually, let's just merge to trunk.  I'm lazy, and don't feel like messing with
git.

[mmenke, 2017-01-24 16:49:34]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2017-01-24 16:49:40]

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-24 16:53:55]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485276574325070,
"parent_rev": "36324125a1d77f4d720687be28f6751e7087cf8a", "commit_rev":
"2617c4a136497244376e1bcf11aa356afb2a1e06"}

[commit-bot: I haz the power, 2017-01-24 16:54:37]

Description was changed from

==========
Enable HTTP/0.9 on non-standard ports by default

This is a temporary patch to enable HTTP/0.9 on non-standard ports by
default.  After merging it with Chrome 56, it will be reverted, in favor
of specifically adding detection of Shoutcast servers on HTTP ports
other than port 80.

BUG=669800
==========

to

==========
Enable HTTP/0.9 on non-standard ports by default

This is a temporary patch to enable HTTP/0.9 on non-standard ports by
default.  After merging it with Chrome 56, it will be reverted, in favor
of specifically adding detection of Shoutcast servers on HTTP ports
other than port 80.

BUG=669800

Review-Url: https://codereview.chromium.org/2651753002
Cr-Commit-Position: refs/heads/master@{#445743}
Committed:
https://chromium.googlesource.com/chromium/src/+/2617c4a136497244376e1bcf11aa...
==========

[commit-bot: I haz the power, 2017-01-24 16:54:38]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/2617c4a136497244376e1bcf11aa...

[mmenke, 2017-01-27 17:22:13]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2656373002/ by mmenke@chromium.org.

The reason for reverting is: Removing to apply the CL with Shoutcast detection
in its place.