New issue
Advanced search Search tips

Issue 915419 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android
Pri: 2
Type: Bug



Sign in to add a comment

Download infobar : Infobar doesn't update from 1 item pending although download has resumed

Project Member Reported by shaktisahu@chromium.org, Dec 15

Issue description

Steps to repro :

1- Start download
2- Do a bunch of wifi disconnect and reconnect so that it says pending
3- In pending state, hit pause and then hit resume
4- Notice that even though the download is in progress, infobar doesn't update
 
Status: Assigned (was: Unconfirmed)
This issue has an owner, a component and a priority, but is still listed as untriaged or unconfirmed. By definition, this bug is triaged. Changing status to "assigned". Please reach out to me if you disagree with how I've done this.
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 18 (4 days ago)

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

commit 35a381d7f5f22c793e6a095960a3a93b574c97b9
Author: Shakti Sahu <shaktisahu@chromium.org>
Date: Fri Jan 18 15:24:23 2019

Download InfoBar : Handling paused and cancelled state correctly

Fixed a few bugs :
1 - If an item is cancelled, the infobar should close
2 - If an item moves to paused from pending state, the infobar should close

Bug: 915419
Change-Id: Ia3f23e7d213062f7853510cfce2b7cbf7ea43e5e
Reviewed-on: https://chromium-review.googlesource.com/c/1418401
Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Shakti Sahu <shaktisahu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#624129}
[modify] https://crrev.com/35a381d7f5f22c793e6a095960a3a93b574c97b9/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadInfoBarController.java
[modify] https://crrev.com/35a381d7f5f22c793e6a095960a3a93b574c97b9/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadInfoBarControllerTest.java

Sign in to add a comment