1114614 - Remove nsIDownloadManagerUI from Firefox for Desktop

Status: RESOLVED FIXED

(Firefox :: Downloads Panel, defect)

Description

[:Paolo Amadini]

Attachment: The patch

It turns out that we have quite a lot of dead code since we removed support for the nsIDownloadManager back-end in Firefox for Desktop.

This patch deals with the simplification of nsIDownloadManagerUI. The interface is not required anymore because it is only ever called by the browser front-end itself.

Comment 1

[Marco Bonardo [:mak]]

Comment on attachment 8540171 [details] [diff] [review]
The patch

Review of attachment 8540171 [details] [diff] [review]:
-----------------------------------------------------------------

Awesome!

::: browser/base/content/browser.js
@@ +3782,5 @@
> +                      "chrome,toolbar=yes,dialog=no,resizable", "Downloads");
> +  } else {
> +    organizer.PlacesOrganizer.selectLeftPaneQuery("Downloads");
> +    organizer.focus();
> +  }

I think instead of all of this, you can use  PlacesCommandHook.showPlacesOrganizer("Downloads");

and basically convert this to a simple if/else since then you have 2 oneliners

::: browser/components/downloads/DownloadsStartup.js
@@ +50,5 @@
>    {
>      if (aTopic != "profile-after-change") {
>        Cu.reportError("Unexpected observer notification.");
>        return;
>      }

this check is pointless, we can remove it.

Comment 2

[:Paolo Amadini]

(In reply to Marco Bonardo [::mak] (needinfo? me) from comment #1)
> I think instead of all of this, you can use 
> PlacesCommandHook.showPlacesOrganizer("Downloads");

Nice! Thanks!

Tryserver build before pushing to mozilla-central:

https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=275255915253

Comment 3

[:Paolo Amadini]

https://hg.mozilla.org/integration/fx-team/rev/a4335e10398f

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/a4335e10398f