The default bug view has changed. See this FAQ.

Remove window.defaultStatus

RESOLVED FIXED in mozilla23

Status

()

Core
DOM
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Gavin, Assigned: bz)

Tracking

({dev-doc-complete, site-compat})

Trunk
mozilla23
dev-doc-complete, site-compat
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Setting them has had no effect in Firefox since the aviary days (dom.disable_window_status_change has been true). Recently, the Firefox UI dropped support for enabling that pref (bug 842017).

These attributes aren't specified in HTML5, so we should just completely remove support for the feature from the DOM (and simplify the relevant nsIWebBrowserChrome2/nsIXULBrowserWindow APIs and client implementations).

Comment 1

4 years ago
I'd rather just make these no-ops. window.status has been mentioned in many docs, so removing would
probably lead to some obscure problem in some page.
What kind of compat problems do you foresee? Since it's just a simple property I'm not really sure what kind of issues removing it could cause. Unqualified references to "status"?
Oh, I was wrong about window.status not being specced:
http://www.whatwg.org/specs/web-apps/current-work/multipage/browsers.html#dom-window-status

That seems like a fine compromise.
Summary: drop support for window.status/window.defaultStatus → drop support for setting window.status, completely remove window.defaultStatus
Created attachment 738895 [details] [diff] [review]
Remove the non-standard window.defaultStatus and the propagation of window.status values to the UI.
Attachment #738895 - Flags: review?(gavin.sharp)
Attachment #738895 - Flags: review?(bugs)
Assignee: nobody → bzbarsky
Created attachment 738897 [details] [diff] [review]
Remove the non-standard window.defaultStatus.
Attachment #738897 - Flags: review?(gavin.sharp)
Attachment #738897 - Flags: review?(bugs)
Attachment #738895 - Attachment is obsolete: true
Attachment #738895 - Flags: review?(gavin.sharp)
Attachment #738895 - Flags: review?(bugs)
I didn't change the guts of .status because there are Gecko-based apps that in fact have UI for it...  The decision to drop it should probably have somewhat more public input than this bug.
Summary: drop support for setting window.status, completely remove window.defaultStatus → Remove window.defaultStatus
Whiteboard: [need review]

Comment 7

4 years ago
Comment on attachment 738897 [details] [diff] [review]
Remove the non-standard window.defaultStatus.

Tiny bit scary. I guess we can try and put back dummy .defaultStatus if 
there are problems.
Attachment #738897 - Flags: review?(bugs) → review+
Attachment #738897 - Flags: review?(gavin.sharp) → review+
(In reply to Boris Zbarsky (:bz) from comment #6)
> I didn't change the guts of .status because there are Gecko-based apps that
> in fact have UI for it...  The decision to drop it should probably have
> somewhat more public input than this bug.

What apps? We can CC a SeaMonkey representative to a bug, I don't see this needing much more input than that given the state of the feature.
Now I filed bug 863339 :) I'll propose a patch on top of these ones.
SeaMonkey was the obvious one, yes.
https://hg.mozilla.org/integration/mozilla-inbound/rev/fc6404ba7c74
Whiteboard: [need review]
Target Milestone: --- → mozilla23
https://hg.mozilla.org/mozilla-central/rev/fc6404ba7c74
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Updated

4 years ago
Blocks: 864256

Updated

4 years ago
Keywords: dev-doc-needed
I've added this bug to the compatibility doc. Please correct the info if wrong.
https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_23
https://developer.mozilla.org/en-US/docs/Web/API/window
https://developer.mozilla.org/en-US/docs/Web/API/window.defaultStatus
Keywords: dev-doc-needed → dev-doc-complete, site-compat
You need to log in before you can comment on or make changes to this bug.