833015 - Update Suite Download Manager UI for Private Browsing changes

Status: RESOLVED FIXED

(SeaMonkey :: Download & File Handling, defect)

Description

[[e-mail]]

We need to update the suite download manager UI following some recent changes in the download manager UI interface.

Comment 1

[[e-mail]]

Attachment: Proposed patch

* Switched the 2nd parameter from an ID to an nsIDownload
* Added support for the aIsPrivate parameter - necessary to force progress
  dialogs because private downloads don't show in the download manager
* Disabled the "visible" attribute to force the backend to show downloads
  otherwise private downloads don't appear if the download manager is open
* Handled the "flash" pref after the "behaviour" pref - necessary now that
  the backend doesn't handle the "flash" pref any more
* Moved the controls in the pref pane around to reflect the new reality
* Renamed the "flash" entity

Code I used to test noninteractive download manager opening:

Components.classes['@mozilla.org/download-manager-ui;1']
          .getService(Components.interfaces.nsIDownloadManagerUI)
          .show(null, null, 1)

If you want to test actual private downloads, you'll need to build with per-window private browsing enabled and open a private browsing window:

window.openDialog(top.getBrowserURL(), '_blank',
                  'chrome,all,private,dialog=no', 'about:blank');

Comment 2

[[e-mail]]

Attachment: With test fix

I thought I might as well expose the existing recent window property. 
I also remembered to change the uuid. I also tweaked the indentation.

Comment 3

[[e-mail]]

Attachment: Includes bug 830190

I realised that cmd_properties was broken and I couldn't fix it without merging in the changes that you've already seen in bug 830190 except for the tweak to the showProperties function and caller that's needed to fix cmd_properties.

Comment 4

[Philip Chee]

Attachment: Screenshot of the download manager.

Sat Jan 26 2013 00:21:46
Error: TypeError: dl.dld is undefined
Source file: chrome://communicator/content/downloads/treeView.js
Line: 156

Comment 5

[Philip Chee]

Sat Jan 26 2013 00:33:40
Error: NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS: '[JavaScript Error: "dl.dld is undefined" {file: "chrome://communicator/content/downloads/treeView.js" line: 207}]' when calling method: [nsITreeView::cycleCell]
Source file: chrome://global/content/bindings/tree.xml
Line: 1014

Comment 6

[Philip Chee]

Sat Jan 26 2013 00:35:20
Error: NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS: '[JavaScript Error: "aDownload is undefined" {file: "chrome://communicator/content/downloads/downloadmanager.js" line: 141}]' when calling method: [nsITreeView::cycleCell]
Source file: chrome://global/content/bindings/tree.xml
Line: 1014

Comment 7

[[e-mail]]

Comment on attachment 706353 [details] [diff] [review]
Includes bug 830190

>   addDownload: function(aDownload) {
>     var attrs = {
>-      dlid: aDownload.id,
>+      guid: aDownload.guid,
>       file: aDownload.target.spec,
>       target: aDownload.displayName,
>       uri: aDownload.source.spec,
>       state: aDownload.state,
>       progress: aDownload.percentComplete,
>       progressMode: nsITreeView.PROGRESS_NONE,
>       resumable: aDownload.resumable,
>       startTime: Math.round(aDownload.startTime / 1000),
>       endTime: Date.now(),
>       referrer: null,
>       currBytes: aDownload.amountTransferred,
>       maxBytes: aDownload.size,
>       lastSec: Infinity, // For calculations of remaining time
>+      download: aDownload
Oops, this should be dld: aDownload - does that fix those errors?

Comment 8

[[e-mail]]

Attachment: Fix addDownload

A couple of fixes to addDownload that should address all the above issues.

Comment 9

[Philip Chee]

Comment on attachment 706423 [details] [diff] [review]
Fix addDownload

> A couple of fixes to addDownload that should address all the above issues.
Looks like things are working again f=me.

Comment 10

[[e-mail]]

Comment on attachment 706423 [details] [diff] [review]
Fix addDownload

(This subsumes bug 830190)

Comment 11

[Ian Neal]

Comment on attachment 706423 [details] [diff] [review]
Fix addDownload

When trying to compile I get the following error:
xpidl.IDLError: error: type 'nsIDownload' not found, /comm-central/suite/common/public/nsISuiteDownloadManagerUI.idl line 14:33
                   [optional] in nsIDownload aDownload,
                                 ^
make[6]: *** [_xpidlgen/nsISuiteDownloadManagerUI.h] Error 1

Comment 12

[[e-mail]]

(In reply to Ian Neal from comment #11)
> When trying to compile I get the following error:
> xpidl.IDLError: error: type 'nsIDownload' not found,
> /comm-central/suite/common/public/nsISuiteDownloadManagerUI.idl line 14:33
>                    [optional] in nsIDownload aDownload,
>                                  ^
> make[6]: *** [_xpidlgen/nsISuiteDownloadManagerUI.h] Error 1
Means your tree doesn't have bug 830271 applied yet.

Comment 14

[[e-mail]]

Pushed comm-central changeset dcdffce8cf46.