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

Issue 2864533004: Fix an issue that download is not automatically resumed (Closed)

Created:
3 years, 7 months ago by qinmin
Modified:
3 years, 7 months ago
CC:
chromium-reviews, David Trainor- moved to gerrit, agrieve+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Fix an issue that download is not automatically resumed An interrupted download will automatically resume after network reconnection. However, if the interruption is caused by switching active networks, no reconnection event will happen. For example, download may get interrupted if it is started on LTE and then OS decides to switch to WIFI. In this case, we should immediately resume the download rather than waiting for reconnection. BUG=707753 Review-Url: https://codereview.chromium.org/2864533004 Cr-Commit-Position: refs/heads/master@{#471004} Committed: https://chromium.googlesource.com/chromium/src/+/43c5f35105c185d6d48b580d7a2ad98ca3be21eb

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -13 lines) Patch
M chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java View 3 chunks +39 lines, -13 lines 0 comments Download

Messages

Total messages: 12 (6 generated)
qinmin
PTAL
3 years, 7 months ago (2017-05-05 23:35:24 UTC) #2
David Trainor- moved to gerrit
lgtm
3 years, 7 months ago (2017-05-09 05:38:34 UTC) #3
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/2864533004/1
3 years, 7 months ago (2017-05-10 19:43:26 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: linux_android_rel_ng on master.tryserver.chromium.android (JOB_TIMED_OUT, https://build.chromium.org/p/tryserver.chromium.android/builders/linux_android_rel_ng/builds/290781)
3 years, 7 months ago (2017-05-11 01:46:49 UTC) #7
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/2864533004/1
3 years, 7 months ago (2017-05-11 17:15:53 UTC) #9
commit-bot: I haz the power
3 years, 7 months ago (2017-05-11 17:56:29 UTC) #12
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/43c5f35105c185d6d48b580d7a2a...

Powered by Google App Engine
This is Rietveld 408576698