Closed Bug 1288071 Opened 8 years ago Closed 8 years ago

Replace ownerDocument.defaultView with ownerGlobal in devtools/

Categories

(DevTools :: General, defect)

defect
Not set
normal

Tracking

(firefox50 affected)

RESOLVED INVALID
Tracking Status
firefox50 --- affected

People

(Reporter: dao, Unassigned, Mentored)

Details

(Whiteboard: [good first bug][lang=js])

+++ This bug was initially created as a clone of Bug #1286854 +++

We should use the new and simpler ownerGlobal property instead of ownerDocument.defaultView.

Affected files:

https://dxr.mozilla.org/mozilla-central/search?q=ownerdocument.defaultview+path%3Adevtools%2F+-file%3Ajquery&redirect=false
I can't find any information about ownerGlobal in the DOM spec, and it seems available only on privileged pages (https://dxr.mozilla.org/mozilla-central/source/dom/webidl/EventTarget.webidl#61-67).

We're working towards running the devtools frontend in a content privileged window so we might actually end up having to remove the existing instances in favor of the more verbose ownerDocument.defaultView:

https://dxr.mozilla.org/mozilla-central/search?q=path%3Adevtool+sownerGlobal&redirect=false
OK
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.