Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(420)

Issue 2656693003: Merge 2617c4a136497244376e1bcf11aa356afb2a1e06 (Closed)

Created:
3 years, 11 months ago by mmenke
Modified:
3 years, 11 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

Merge 2617c4a136497244376e1bcf11aa356afb2a1e06 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. TBR=eroman@chromium.org NOTRY=TRUE NOPRESUBMIT=TRUE BUG=669800 Review-Url: https://codereview.chromium.org/2651753002 Cr-Original-Commit-Position: refs/heads/master@{#445743} Review-Url: https://codereview.chromium.org/2656693003 Cr-Commit-Position: refs/branch-heads/2924@{#856} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} Committed: https://chromium.googlesource.com/chromium/src/+/431a5e87e4eb425d0c65cf12070b4bd3a3d0ea22

Patch Set 1 #

Patch Set 2 : fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -5 lines) Patch
M chrome/browser/io_thread.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/net/errorpage_browsertest.cc View 1 chunk +5 lines, -3 lines 0 comments Download
M net/http/http_network_session.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 11 (7 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2656693003/20001
3 years, 11 months ago (2017-01-24 21:00:29 UTC) #3
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full ...
3 years, 11 months ago (2017-01-24 21:00:30 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2656693003/20001
3 years, 11 months ago (2017-01-24 21:02:36 UTC) #8
commit-bot: I haz the power
3 years, 11 months ago (2017-01-24 21:07:21 UTC) #11
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/431a5e87e4eb425d0c65cf12070b...

Powered by Google App Engine
This is Rietveld 408576698