• Max Morin's avatar
    Revert "Reland "Cancel parallel download request if server doesn't send partial response"" · e127247e
    Max Morin authored
    This reverts commit 05267e5e.
    
    Reason for revert: <INSERT REASONING HERE>
    
    Original change's description:
    > Reland "Cancel parallel download request if server doesn't send partial response"
    > 
    > This reverts commit 13077433.
    > 
    > Reason for revert: <INSERT REASONING HERE>
    > 
    > Original change's description:
    > > Revert "Cancel parallel download request if server doesn't send partial response"
    > > 
    > > This reverts commit 189d1033.
    > > 
    > > Reason for revert: "ParallelDownloadTest.NoPartialResponse" is flaky. See https://crbug.com/841666.
    > > 
    > > Original change's description:
    > > > Cancel parallel download request if server doesn't send partial response
    > > > 
    > > > If the server doesn't support partial responses, Chrome should cancel
    > > > parallel download requests.
    > > > These requests result in the wrong write offset and makes the file larger
    > > > than the original
    > > > 
    > > > BUG=838627
    > > > 
    > > > Change-Id: I1fa878854579a1572e6e2f90f3bb932b59b333ba
    > > > Reviewed-on: https://chromium-review.googlesource.com/1048130
    > > > Commit-Queue: Min Qin <qinmin@chromium.org>
    > > > Reviewed-by: Xing Liu <xingliu@chromium.org>
    > > > Cr-Commit-Position: refs/heads/master@{#557252}
    > > 
    > > TBR=qinmin@chromium.org,xingliu@chromium.org
    > > 
    > > Change-Id: I995c3c48fe39c535275a353130b10204009c5e30
    > > No-Presubmit: true
    > > No-Tree-Checks: true
    > > No-Try: true
    > > Bug: 838627, 841666
    > > Reviewed-on: https://chromium-review.googlesource.com/1053470
    > > Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
    > > Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#557489}
    > 
    > TBR=yhirano@chromium.org,qinmin@chromium.org,xingliu@chromium.org
    > 
    > Change-Id: I47a2d4131efa8b863d9868eeca74e99d7beeaa8e
    > No-Presubmit: true
    > No-Tree-Checks: true
    > No-Try: true
    > Bug: 838627, 841666
    > Reviewed-on: https://chromium-review.googlesource.com/1054887
    > Reviewed-by: Min Qin <qinmin@chromium.org>
    > Commit-Queue: Min Qin <qinmin@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#557734}
    
    TBR=yhirano@chromium.org,qinmin@chromium.org,xingliu@chromium.org
    
    Change-Id: I7a618b9533229ea1bb97ff65470e9c60fe580a49
    No-Try: true
    Bug: 838627, 841666
    Reviewed-on: https://chromium-review.googlesource.com/1053729Reviewed-by: default avatarMax Morin <maxmorin@chromium.org>
    Commit-Queue: Max Morin <maxmorin@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#557834}
    e127247e
download_file_unittest.cc 39.7 KB