578797 - dead code removal: browserOverlay.xul

Status: RESOLVED INCOMPLETE

(Other Applications :: DOM Inspector, defect)

Description

[Colby Russell :crussell (no longer Mozilla-ing)]

Attachment: remove browserOverlay.dtd and remove entities used only in that file from all localizations

We stopped shipping it in 2001, but it's been hanging around this whole time.

Comment 1

[[e-mail]]

What about the entities for the commented-out preference element to turn it on?

(It might be simpler to fix and enable the overlay!)

Comment 2

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) Remove browserOverlay.xul and venkmanOverlay.xul remnants

Bah, I just noticed this too ;->

(In reply to [e-mail] from comment #1)

> What about the entities for the commented-out preference element to turn it
> on?

This patch touches en-US only: I'll do the other locales before checkin.

> (It might be simpler to fix and enable the overlay!)

I think browserOverlay.xul was replaced by tasksOverlay-ff.xul.

Maybe venkmanOverlay.xul could "become" a tasksOverlay-jsd.xul, like tasksOverlay-cz.xul?
(Not sure if it's worth it.)

Comment 3

[[e-mail]]

(In reply to Serge Gautherie from comment #2)
> I think browserOverlay.xul was replaced by tasksOverlay-ff.xul.
The Navigator references make it quite clear that it is unrelated ;-)

Comment 4

[Colby Russell :crussell (no longer Mozilla-ing)]

What kind of feedback are you looking for?

Comment 5

[Colby Russell :crussell (no longer Mozilla-ing)]

Comment on attachment 609141 [details] [diff] [review]
(Bv1) Remove browserOverlay.xul and venkmanOverlay.xul remnants

Removing feedback=me.  See comment 4

Comment 6

[Firefox Bug Husbandry Bot]

Bulk close. This component is no longer supported or maintained.

https://bugzilla.mozilla.org/show_bug.cgi?id=1499023

Comment 7

[Firefox Bug Husbandry Bot]

Bulk close. This component is no longer supported or maintained.

https://bugzilla.mozilla.org/show_bug.cgi?id=1499023