When we null out the callback RegisterCallback(base::Closure()) of the ByteStreamReader in DownloadFileImpl, we need to figure out if the TCPSocket is disconnected evetuanlly.

https://cs.chromium.org/chromium/src/net/socket/tcp_socket.h?dr=C&ssfr=1&l=29

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b444a98187571cf9ec627ac52fee7e81ffcecb93

commit b444a98187571cf9ec627ac52fee7e81ffcecb93
Author: xingliu <xingliu@chromium.org>
Date: Tue Apr 18 18:09:49 2017

Fix an issue that we didn't clean url request properly.

The ByteStreamReader we used in DownloadFile does not support Close()
api, so we need to close the url request on UI thread if
"ByteStreamReader::Close" should be called.

Before we implement parallel download, we null out the callback on
ByteStreamReader during download interruption, and on UI thread, we
cancel the url request. But for parallel download, at some point, we
won't interrupt the download but need to cancel a particular request.

There is another CL that implements a browser test for parallel download and it will test the code here.
https://codereview.chromium.org/2819483002/

BUG= 710576 , 644352

Review-Url: https://codereview.chromium.org/2811293004
Cr-Commit-Position: refs/heads/master@{#465289}

[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_browsertest.cc
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_file.h
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_file_impl.cc
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_file_impl.h
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_file_unittest.cc
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_item_impl.cc
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_item_impl.h
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_item_impl_unittest.cc
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_job.cc
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/download_job.h
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/mock_download_file.cc
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/mock_download_file.h
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/parallel_download_job.cc
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/browser/download/parallel_download_job.h
[modify] https://crrev.com/b444a98187571cf9ec627ac52fee7e81ffcecb93/content/public/test/test_file_error_injector.cc

This is needed to fix a parallel download issue, also another CL needs this to be merged first.

The link to the other CL that needs this to be merged. 

https://codereview.chromium.org/2823273004/

Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b

commit 8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b
Author: Xing Liu <xingliu@chromium.org>
Date: Wed Apr 19 18:38:18 2017

Fix an issue that we didn't clean url request properly.

The ByteStreamReader we used in DownloadFile does not support Close()
api, so we need to close the url request on UI thread if
"ByteStreamReader::Close" should be called.

Before we implement parallel download, we null out the callback on
ByteStreamReader during download interruption, and on UI thread, we
cancel the url request. But for parallel download, at some point, we
won't interrupt the download but need to cancel a particular request.

There is another CL that implements a browser test for parallel download and it will test the code here.
https://codereview.chromium.org/2819483002/

BUG= 710576 , 644352

Review-Url: https://codereview.chromium.org/2811293004
Cr-Commit-Position: refs/heads/master@{#465289}
(cherry picked from commit b444a98187571cf9ec627ac52fee7e81ffcecb93)

Review-Url: https://codereview.chromium.org/2828073002 .
Cr-Commit-Position: refs/branch-heads/3071@{#60}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}

[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_browsertest.cc
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_file.h
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_file_impl.cc
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_file_impl.h
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_file_unittest.cc
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_item_impl.cc
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_item_impl.h
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_item_impl_unittest.cc
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_job.cc
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/download_job.h
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/mock_download_file.cc
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/mock_download_file.h
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/parallel_download_job.cc
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/browser/download/parallel_download_job.h
[modify] https://crrev.com/8cb70eaa5ea6568193561a95fc04bdcd4cd8fe4b/content/public/test/test_file_error_injector.cc

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8487c6bd015949404b45ad174f6b8fc806f95f2c

commit 8487c6bd015949404b45ad174f6b8fc806f95f2c
Author: xingliu <xingliu@chromium.org>
Date: Thu Apr 20 23:14:33 2017

Parallel download browser test.

This CL sets up a browser test fixture for parallel download.

BUG= 710576 , 644352

Review-Url: https://codereview.chromium.org/2819483002
Cr-Commit-Position: refs/heads/master@{#466181}

[modify] https://crrev.com/8487c6bd015949404b45ad174f6b8fc806f95f2c/content/browser/download/download_browsertest.cc
[modify] https://crrev.com/8487c6bd015949404b45ad174f6b8fc806f95f2c/content/browser/download/download_job_factory.cc
[modify] https://crrev.com/8487c6bd015949404b45ad174f6b8fc806f95f2c/content/browser/download/parallel_download_utils.cc
[modify] https://crrev.com/8487c6bd015949404b45ad174f6b8fc806f95f2c/content/browser/download/parallel_download_utils.h
[modify] https://crrev.com/8487c6bd015949404b45ad174f6b8fc806f95f2c/content/public/test/test_download_request_handler.cc
[modify] https://crrev.com/8487c6bd015949404b45ad174f6b8fc806f95f2c/content/public/test/test_download_request_handler.h