593108 - Download notification removed when tapped while download in progress

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Ayan Shah[:ashah]]

Build Id:
Mozilla/5.0 (Android; Linux armv71; rv:2.0b6pre)
Gecko/20100901 Namoroka/4.0b6pre Fennec/2.0b1pre

Steps to Reproduce:
1. Start downloading a file
2. Open the status bar notification pane on your android device
3. Tap on the "Downloading" notification entry

Actual Result:
Tapping on the notification(while the file is still downloading) brings
me back to the webpage and the notification entry is removed. But when the file
has completed downloading, the notification entry comes back to the system bar. 

Expected Result:
Tapping on the notification should take me to the download manager.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

dupe of bug 593092

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

why is this a dupe? are you planning on re-posting the notification when the callback is called?

It may be dependent on bug 592088 depending on how that's implemented.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Dupe because:

> Expected Result:
> Tapping on the notification should take me to the download manager.

Comment 4

[Brad Lassey [:blassey] (use needinfo?)]

(In reply to comment #3)
> Dupe because:
> 
> > Expected Result:
> > Tapping on the notification should take me to the download manager.


Ehh.. let's edit that to be:

Tapping on the notification should only take me to the download manager and the notification should persist

Perhaps that's the current behavior we want for the moment, but when we have progress bars in the notifications I think we want the notification to persist until the download finishes.

Comment 5

[Brad Lassey [:blassey] (use needinfo?)]

I believe the patch on bug 592088 will fix this. Please re-test this after that lands.

Comment 6

[Alex Pakhotin (:alexp)]

Bug 593113 is the fix for this.

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

fixed by bug 593113

Comment 8

[Ayan Shah[:ashah]]

Verified on :

Build: Mozilla /5.0 (Android;Linux armv7l;rv:2.0b8pre) Gecko/20101012
Firefox/4.0b8pre Fennec /4.0b2pre

Comment 9

[Ayan Shah[:ashah]]

Litmus case added - https://litmus.mozilla.org/show_test.cgi?id=13659