Closed
Bug 276679
Opened 21 years ago
Closed 21 years ago
nsHistoryLoadListener.cpp in source tree, but unused
Categories
(Core Graveyard :: History: Global, defect)
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.
Comment 1•21 years ago
|
||
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
Comment 2•21 years ago
|
||
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
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•