248010 - Remove unused |nsSHistory::PrintHistory()|

Status: RESOLVED INVALID

(Core :: DOM: Navigation, defect)

Description

[Serge Gautherie (:sgautherie)]

From bug 78672 comment 28 and bug 78672 comment 29:
{{ <http://lxr.mozilla.org/mozilla/search?string=PrintHistory>
PrintHistory

/content/xml/tests/docbook.css, line 342 -- PrintHistory {

/xpfe/components/shistory/src/nsSHistory.cpp, line 281 -- nsSHistory::PrintHistory()
/xpfe/components/shistory/src/nsSHistory.h, line 85 -- nsresult PrintHistory();
}}

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) <nsSHistory.*>

Comment 2

[Serge Gautherie (:sgautherie)]

Comment on attachment 151397 [details] [diff] [review]
(Av1) <nsSHistory.*>


I have no compiler: Could you compile (with DEBUG=ON !) / review this patch ?
Thanks.

I'm asking you directly on this one,
because it's a line removal only,
and I wonder if it could be a WontFix...

Comment 3

[[e-mail]]

Bug 78672 has a misleading title. You can actually build optimized, symbols and
debug independently. The default is optimized, no symbols, no debug; this
function will not get compiled in this case and therefore is not relevant.
Relevant .mozconfig/configure options:
--disable-optimize
--enable-debug (#define DEBUG plus also enables symbols by default)
--enable-debugger-info-modules=yes (enables symbols but not DEBUG)
--enable-debugger-info-modules=no (disables --enable-debug symbols)