1419704 - Remove nsIGlobalHistory2.idl & docshell implementation of nsDownloadHistory

Status: RESOLVED FIXED

(Core :: DOM: Navigation, enhancement, P3)

Description

[Samael Wang [:freesamael] (away for now)]

It seems we've deleted the only implementation - nsGlobalHistory - for many years. 
Event comm-central doesn't seem to have any implementation:
https://searchfox.org/comm-central/search?q=nsIGlobalHistory2&path=

We should remove the idl and clean up the dead code.

Comment 1

[Marco Bonardo [:mak]]

nsDownloadHistory.cpp that is the generic downloadhistory implementation (not used in firefox, but likely used in other software?) seems to use it yet.
Though, if there is no implementation of nsIGlobalHistory2, it's worthless to keep it around, so let's also remove nsDownloadHistory at this point. From a quick glance most callpoints are properly checking for success when instantiating an nsIDownloadHistory handler. Fwiw, the only consumer I am worried about is Android, but it should just do nothing, as before, since nsIGlobalHistory2 is not implemented.

Comment 2

[Marco Bonardo [:mak]]

Marco recently worked on excluding nsDownloadHistory from Firefox, maybe he could be interested into removing everything here, so cc-ing him.

Comment 3

[Mark Banner (:standard8)]

Attachment: Bug 1419704 - Remove nsIGlobalHistory2 and docshell's nsDownloadHistory as they are redundant.

nsIGlobalHistory2 is not implemented by any mozilla-central nor comm-central applications, and hence can be removed. As a result, nsDownloadHistory's implementation can also be removed.

MozReview-Commit-ID: 5bQ2OHsV1Ky

Comment 4

[Phabricator Automation]

Comment on attachment 8989524 [details]
Bug 1419704 - Remove nsIGlobalHistory2 and docshell's nsDownloadHistory as they are redundant.

Boris Zbarsky [:bz] (no decent commit message means r-) has approved the revision.

https://phabricator.services.mozilla.com/D1929

Comment 5

[Phabricator Automation]

Comment on attachment 8989524 [details]
Bug 1419704 - Remove nsIGlobalHistory2 and docshell's nsDownloadHistory as they are redundant.

Marco Bonardo [::mak] has approved the revision.

https://phabricator.services.mozilla.com/D1929

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2291d4737e17
Remove nsIGlobalHistory2 and docshell's nsDownloadHistory as they are redundant. r=mak,bzbarsky

Comment 7

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/2291d4737e17

Comment 8

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/2291d4737e17