562193 - Implement Components.utils.getGlobalForObject(obj)

Status: RESOLVED FIXED

(Core :: XPConnect, enhancement)

Description

[Jeff Walden [:Waldo]]

With __parent__ going away soon, we should provide a better, simpler alternative to support this reasonable use of it.

Comment 1

[Jeff Walden [:Waldo]]

Attachment: Patch

Manual test fodder used to check outerization correctness (success condition: not asserting about inner windows in equality hooks), as typed into the JavaScript console:

var wm = Components.classes['@mozilla.org/appshell/window-mediator;1'].getService(Components.interfaces.nsIWindowMediator); var enumerator = wm.getEnumerator("navigator:browser"); var win = enumerator.getNext(); var doc = win.gBrowser.contentDocument; doc.defaultView === Components.utils.getGlobalForObject(doc.documentElement); var dv = doc.defaultView; var ggfo = Components.utils.getGlobalForObject(doc.documentElement); Components.utils.getGlobalForObject(dv.eval("(function foo(){})")) === ggfo

Comment 2

[Jeff Walden [:Waldo]]

Attachment: qref ftw :-\

Comment 3

[Jeff Walden [:Waldo]]

http://hg.mozilla.org/tracemonkey/rev/16f1064de17e

I have a brief blog post queued up on the new method, which I'll make live once this migrates over to m-c.  I'll try to add a writeup to MDC in a bit when the site's less flaky and unreachable.

Comment 4

[Jeff Walden [:Waldo]]

https://developer.mozilla.org/en/Components.utils.getGlobalForObject

Linked from the Components and Components.utils pages, needs to be added to what's-new pages of whatever sort so leaving the d-d-n keyword...

Comment 5

[Robert Sayre]

http://hg.mozilla.org/mozilla-central/rev/16f1064de17e

Comment 6

[Eric Shepherd [:sheppy]]

Minor tweaks made to Jeff's doc, and it's been listed where appropriate.

https://developer.mozilla.org/en/Components.utils.getGlobalForObject