Closed Bug 1425999 Opened 2 years ago Closed 2 years ago
.disable _window _status _change is obsolete
Nika, can you please take a look?
Priority: -- → P3
Consulting the HTML Living Standard and the W3C HTML5 changelog (https://www.w3.org/TR/2012/WD-html5-diff-20121025/) as well as bug 863339 I believe this (non)behavior is intentional and according to spec. Accordingly, the preference should be removed and (most importantly!) the appropriate documentation updated.
Summary: dom.disable_window_status_change=false has no effect → dom.disable_window_status_change is obsolete
It looks like we never got around to implementing this API in e10s. The object which should implement `setStatus` in e10s doesn't have any `setStatus` implementation: (https://searchfox.org/mozilla-central/rev/22c55eb7b7e6494a8615a7af3b613ff899d2cdba/browser/base/content/tab-content.js#697-735). This is fine, because as comment 2 notes, this is a deprecated API and we don't have it enabled by default. I imagine we can probably remove the pref & restrict callers to chrome code in the parent process.
Attachment #8938522 - Flags: review?(kyle)
Comment on attachment 8938522 [details] [diff] [review] Remove the dom.disable_window_status_change pref, as it no longer can be disabled Review of attachment 8938522 [details] [diff] [review]: ----------------------------------------------------------------- OH COME ON I'M ON VACATION. *goes back to sitting here writing code for own projects*
Attachment #8938522 - Flags: review?(kyle) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b86850493c20 Remove the dom.disable_window_status_change pref, as it no longer can be disabled, r=qdot
You need to log in before you can comment on or make changes to this bug.