1223272 - Remove XUL directory viewer

Status: NEW

(Core :: XUL, defect)

Description

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Attachment: Remove code

directory.xul has been removed from the tree, so this code doesn't work any more.

Comment 1

[Masatoshi Kimura [:emk]]

SeaMonkey still uses Gecko-Content-Viewers. (Yes, I know you don't care about comm-central as a good MoCo employee :) )

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

It's not that I don't care about comm-central. But given that this is a seamonkey specific feature, it should live in the seamonkey tree. Especially given that this is quite a lot of code.

Looks like this code is using enough XPCOM that seamonkey can override the relevant contracts/categories and copy this code into there.

Comment 3

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Additionally, as far as I can see, this feature is disabled by default even in SeaMonkey with no UI to turn it back on.

Comment 4

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Review ping?

Comment 5

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 8685257 [details] [diff] [review]
Remove code

Review of attachment 8685257 [details] [diff] [review]:
-----------------------------------------------------------------

Trying an XPToolkit reviewer since none of the other XPFE reviewers have logged in to bugzilla the last 60 days and so bugzilla won't let me flag them for review.

Comment 6

[Jan Varga [:janv]]

Comment on attachment 8685257 [details] [diff] [review]
Remove code

Review of attachment 8685257 [details] [diff] [review]:
-----------------------------------------------------------------

Ok, just double check if nsDirectoryViewerFactory is used at all, if it's not, then just remove it too. Thanks.

Comment 7

[Firefox Bug Husbandry Bot]

Moving to Core:XUL per https://bugzilla.mozilla.org/show_bug.cgi?id=1455336

Comment 8

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.