Download progress dialog can break in Private Browsing mode

RESOLVED FIXED in seamonkey2.21

Status

SeaMonkey
Download & File Handling
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: rsx11m, Assigned: neil@parkwaycc.co.uk)

Tracking

(Blocks: 1 bug)

Trunk
seamonkey2.21

SeaMonkey Tracking Flags

(seamonkey2.19 fixed, seamonkey2.20 fixed, seamonkey2.21 fixed)

Details

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
Steps to reproduce:

 1. Go into Edit > Preferences > Browser > Downloads
 2. Select "Open a progress dialog"
 3. Open a Private Browsing window
 4. Open the Download Manager (seems to help triggering the issue)
 5. Download some file and watch the progress bar
 6. Repeat until the progress bar dialog breaks (screenshot follows)
    (closing the private window and opening a new one may help as well)
 7. File is downloaded but the dialog is completely useless

Following entries in the Error Console:

Error: TypeError: dldata is undefined
Source File: chrome://communicator/content/downloads/progressDialog.js
Line: 36

Error: NS_ERROR_FAILURE: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIControllers.removeController]
Source File: chrome://communicator/content/downloads/progressDialog.js
Line: 102
(Reporter)

Comment 1

4 years ago
Created attachment 750871 [details]
Screenshot on Windows 7
(Reporter)

Comment 2

4 years ago
Step #2 is implicit for Private Browsing, thus no need to select it. Apparently the issue needs a prior download from a non-private window to be triggered. And, the Download Manager has to be open when downloading from a private window.

Thus, here the more direct list of steps to reproduce:

 1. Start SeaMonkey with a new profile (just to be sure that nothing interferes)
 2. Download some image or file from a site with the right-click menu
 3. The Download Manager opens by default, leave it open
 4. Now open a new private window, leave the initial non-private window open
 5. Download another image or file from a web site
 6. Progress Dialog comes up and stays broken, file is downloaded
(Reporter)

Comment 3

4 years ago
To clarify the timing of the messages in the Error Console:

> Error: TypeError: dldata is undefined
> Source File: chrome://communicator/content/downloads/progressDialog.js
> Line: 36

The first error occurs upon opening of the progress dialog.

> Error: NS_ERROR_FAILURE: Component returned failure code: 0x80004005
> (NS_ERROR_FAILURE) [nsIControllers.removeController]
> Source File: chrome://communicator/content/downloads/progressDialog.js
> Line: 102

The second error is logged when closing the progress dialog.
OS: Windows 7 → All
Hardware: x86_64 → All
(Assignee)

Comment 4

4 years ago
Ah yes, I know what's going on here; the progress dialog assumes that if the download manager is open then it will be showing its download. This is of course false for private downloads.
(Assignee)

Comment 5

4 years ago
Created attachment 751686 [details] [diff] [review]
Proposed patch
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #751686 - Flags: feedback?(rsx11m.pub)
(Reporter)

Comment 6

4 years ago
Comment on attachment 751686 [details] [diff] [review]
Proposed patch

This fixes it for me, with or without the Download Manager open.
Attachment #751686 - Flags: feedback?(rsx11m.pub) → feedback+
(Assignee)

Updated

4 years ago
Attachment #751686 - Flags: review?(iann_bugzilla)

Updated

4 years ago
Attachment #751686 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 7

4 years ago
Pushed comm-central changeset 6c6d2a00b240.
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.21
(Assignee)

Comment 8

4 years ago
Comment on attachment 751686 [details] [diff] [review]
Proposed patch

[Approval Request Comment]
User impact if declined: Unable to track progress of a private download
Risk to taking this patch (and alternatives if risky): Workaround is to close the download manager before starting a private download
String changes made by this patch: None
Attachment #751686 - Flags: approval-comm-beta?
Attachment #751686 - Flags: approval-comm-aurora?
Attachment #751686 - Flags: approval-comm-beta?
Attachment #751686 - Flags: approval-comm-beta+
Attachment #751686 - Flags: approval-comm-aurora?
Attachment #751686 - Flags: approval-comm-aurora+
(Assignee)

Comment 9

4 years ago
https://hg.mozilla.org/releases/comm-aurora/rev/e84bb9894e03
http://hg.mozilla.org/releases/comm-beta/rev/a70e094d8b4d
status-seamonkey2.19: --- → fixed
status-seamonkey2.20: --- → fixed
status-seamonkey2.21: --- → fixed
You need to log in before you can comment on or make changes to this bug.