Closed Bug 842017 Opened 11 years ago Closed 11 years ago

Remove broken support for displaying window.status / window.defaultStatus

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 21

People

(Reporter: dao, Assigned: dao)

References

Details

(Keywords: user-doc-needed)

Attachments

(1 file)

Attached patch patchSplinter Review
dom.disable_window_open_feature.status has been true in Firefox for ages and support for displaying status text is broken (bug 343847).
Attachment #714780 - Flags: review?(felipc)
Shouldn't we then remove dom.disable_window_open_feature.status together with this? Or are you leaving it there for other applications like SeaMonkey?
Other application may still want it, but this doesn't mean it needs to be supported forever. That's however a separate decision that can happen in a different bug.
Comment on attachment 714780 [details] [diff] [review]
patch

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

Ok. It'd be nice if this pref didn't show up in about:config in Firefox as it'll be unsupported. But I guess the effort for that is not really worth it, it's probably much better use of time on dropping support for it altogether, as this is generally a bad feature and which has been unused for a long time.
Attachment #714780 - Flags: review?(felipc) → review+
https://hg.mozilla.org/mozilla-central/rev/69cc1bc86785
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 21
Should have been documented (see bug 862540).
Keywords: dev-doc-needed
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: