248011 - Cleanup <docbook.css>

Status: RESOLVED WONTFIX

(Core :: XML, 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)]

This file is used there:
{{ <http://lxr.mozilla.org/mozilla/search?string=docbook.css>
docbook.css

/content/xml/tests/docbooktest.xml, line 2 -- <?xml-stylesheet
href="docbook.css" type="text/css"?>
/content/xml/tests/script.xml, line 2 -- <?xml-stylesheet href="docbook.css"
type="text/css"?>
/content/xml/tests/simple.xml, line 2 -- <?xml-stylesheet href="docbook.css"
type="text/css"?>
}}

Comment 2

[Serge Gautherie (:sgautherie)]

Attachment: (Av1-r) <docbook.css> [Checkin: Comment 5]

If this one is good to be removed;
I'll post another patch inlucing the others that can be removed too.

Comment 3

[Peter Van der Beken [:peterv]]

Comment on attachment 151400 [details] [diff] [review]
(Av1-r) <docbook.css>
[Checkin: Comment 5]

Whatever. Since this is a test stylesheet I personally see no reason to touch
that.

Comment 4

[Anne (:annevk)]

Shouldn't this patch be checked in?

Comment 5

[Tyler Downer [He/Him]]

wasn't it?
http://hg.mozilla.org/mozilla-central/rev/a30604a3de20

Comment 6

[Serge Gautherie (:sgautherie)]

Comment on attachment 151400 [details] [diff] [review]
(Av1-r) <docbook.css>
[Checkin: Comment 5]


http://hg.mozilla.org/mozilla-central/rev/a30604a3de20

Comment 7

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) all the obviously unused

I might have left a few non-trivial unused rules,
but this patch should be enough for this bug.

(In reply to comment #3)
> I personally see no reason to touch that.

First patch was to clean up mxr results.
This one is just a continuation patch ... which removes more than 50% of the file :->

Comment 8

[Peter Van der Beken [:peterv]]

Comment on attachment 397008 [details] [diff] [review]
(Bv1) all the obviously unused

Either remove the files or leave them as is.