Pump the http version to download system.

Pump the http version to download system.

We need to know the http version to determine if we can use parallel
download. So pump the http version from url request.

BUG=644352
Review-Url: https://codereview.chromium.org/2759843002
Cr-Commit-Position: refs/heads/master@{#458478}
Committed: https://chromium.googlesource.com/chromium/src/+/2a5a15495b82e6d4d5e9c5af354726417734226b

[xingliu, 2017-03-18 21:26:14]

xingliu@chromium.org changed reviewers:
+ asanka@chromium.org, dtrainor@chromium.org, qinmin@chromium.org

[xingliu, 2017-03-18 21:26:14]

Hi,PTAL.

[asanka, 2017-03-20 20:47:23]

LGTM

https://codereview.chromium.org/2759843002/diff/1/content/browser/download/pa...
File content/browser/download/parallel_download_utils.cc (right):

https://codereview.chromium.org/2759843002/diff/1/content/browser/download/pa...
content/browser/download/parallel_download_utils.cc:50: // 3. HTTP/1.1 protocol,
not QUIC or HTTP/2.
Nor HTTP 1.0.

[qinmin, 2017-03-21 05:34:23]

lgtm

[xingliu, 2017-03-21 17:15:07]

Thanks for the review.

https://codereview.chromium.org/2759843002/diff/1/content/browser/download/pa...
File content/browser/download/parallel_download_utils.cc (right):

https://codereview.chromium.org/2759843002/diff/1/content/browser/download/pa...
content/browser/download/parallel_download_utils.cc:50: // 3. HTTP/1.1 protocol,
not QUIC or HTTP/2.
On 2017/03/20 20:47:23, asanka wrote:
> Nor HTTP 1.0.

Done.

[xingliu, 2017-03-21 17:15:15]

The CQ bit was checked by xingliu@chromium.org

[xingliu, 2017-03-21 17:15:16]

The patchset sent to the CQ was uploaded after l-g-t-m from asanka@chromium.org,
qinmin@chromium.org
Link to the patchset: https://codereview.chromium.org/2759843002/#ps20001
(title: "Work on feedback.")

[commit-bot: I haz the power, 2017-03-21 17:16:31]

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

[David Trainor- moved to gerrit, 2017-03-21 17:28:02]

lgtm % nits

[xingliu, 2017-03-21 17:50:25]

The CQ bit was unchecked by xingliu@chromium.org

[xingliu, 2017-03-21 17:58:46]

The CQ bit was checked by xingliu@chromium.org

[commit-bot: I haz the power, 2017-03-21 17:59:32]

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-03-21 18:18:45]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490119126399800,
"parent_rev": "4e2655ac5428e02f7aeb2051b0b43f1c961cbe95", "commit_rev":
"bf9dea713c6efb584f415d2f76aeed068c3c2abe"}

[commit-bot: I haz the power, 2017-03-21 18:23:24]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490119126399800,
"parent_rev": "bfe64e9881235f7e355664acd97aef52a5a5d2c7", "commit_rev":
"2a5a15495b82e6d4d5e9c5af354726417734226b"}

[commit-bot: I haz the power, 2017-03-21 18:24:14]

Description was changed from

==========
Pump the http version to download system.

We need to know the http version to determine if we can use parallel
download. So pump the http version from url request.

BUG=644352
==========

to

==========
Pump the http version to download system.

We need to know the http version to determine if we can use parallel
download. So pump the http version from url request.

BUG=644352

Review-Url: https://codereview.chromium.org/2759843002
Cr-Commit-Position: refs/heads/master@{#458478}
Committed:
https://chromium.googlesource.com/chromium/src/+/2a5a15495b82e6d4d5e9c5af3547...
==========

[commit-bot: I haz the power, 2017-03-21 18:24:15]

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