New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 760690 link

Starred by 1 user
Status: Assigned
Owner:
qin...@chromium.org
Cc:
█ jming@chromium.org
shaktisahu@chromium.org
xingliu@chromium.org
dtrainor@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Feature



Sign in to add a comment

Download won't automatically resume after toggling wifi on/off after download is paused

Project Member Reported by qin...@chromium.org, Aug 30 2017 
See b/64779151

STEPS TO REPRODUCE:
1. Launch Chrome > download any file
2. Pause the download
3. Turn off and then turn on WiFi
4. Tap on Resume to resume the download and observe

Download won't resume afterwards.

The issue is that the original connection is closed in step3. So when resuming the download in step 4, the download will automatically get interrupted. 
Autoresumption didn't kick in because this is a network failure, so native code will not auto resume. And java side is waiting for network connection change signals, so it will not resume either.
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 7 2017 

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

commit 65299a7b1b2df9c35d93330e2e36bb3e03a636e4
Author: Min Qin <qinmin@chromium.org>
Date: Thu Sep 07 19:14:36 2017

Fix an issue that download is not resumed after toggling wifi

If a download is paused, Chrome keeps the original network connection.
When user toggles wifi, the original connection is broken.
As a result, resuming the download on the original connection won't work.
The java side should retry download resumption as long as active network is available.
This CL fixes a previous TODO.
The previous fix only works when user switched from mobile to wireless networks.
This CL extends that fix to all types of networks.

BUG=760690

Change-Id: Ic111f5bc734936e012a83db666cccffd491bce3d
Reviewed-on: https://chromium-review.googlesource.com/651727
Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Min Qin <qinmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500356}
[modify] https://crrev.com/65299a7b1b2df9c35d93330e2e36bb3e03a636e4/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java
►

Sign in to add a comment