Closed Bug 276679 Opened 21 years ago Closed 21 years ago

nsHistoryLoadListener.cpp in source tree, but unused

Categories

(Core Graveyard :: History: Global, defect)

x86
Linux
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: crispin, Unassigned)

References

Details

While trying to work out if nsIBrowserHistory::HidePage was ever actually used, I noticed that both: http://lxr.mozilla.org/seamonkey/source/xpfe/components/history/src/nsHistoryLoadListener.cpp http://lxr.mozilla.org/seamonkey/source/toolkit/components/history/src/nsHistoryLoadListener.cpp are not used, they are not compiled (missing from the Makefile.in) and in fact haven't been updated to the new nsIBrowserHistory API (nsIURI rather than strings). Possibly they should be removed, and perhaps the nsIBrowserHistory::HidePage() function as well.
r=me for removing the files. IIRC there were external embedders who were using hidePage, but I don't remember why.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Removing xpfe/components/history/src/nsHistoryLoadListener.cpp; /cvsroot/mozilla/xpfe/components/history/src/nsHistoryLoadListener.cpp,v <-- nsHistoryLoadListener.cpp new revision: delete; previous revision: 1.8 done Removing toolkit/components/history/src/nsHistoryLoadListener.cpp; /cvsroot/mozilla/toolkit/components/history/src/nsHistoryLoadListener.cpp,v <-- nsHistoryLoadListener.cpp new revision: delete; previous revision: 1.3 done
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.