Closed Bug 1288071 Opened 9 years ago Closed 9 years ago

Replace ownerDocument.defaultView with ownerGlobal in devtools/

Categories

(DevTools :: General, defect)

Product:
DevTools
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox50 affected)

Status:
RESOLVED INVALID
Tracking Flags:
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: 9 years ago
Resolution: --- → INVALID
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.