987112 - Remove the parentObject argument to dictionary ToObject() methods

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Boris Zbarsky [:bzbarsky]]

In practice, this had better be the same as the current global on cx, since that last represents the compartment we'll put the actual dictionary object in.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Oh, and the reason this didn't use to be possible before is that we parented directly to the scope object unless a different native parent was provided.  But now we parent to the global of the scope object (as of bug 717637) for any case that doesn't provide a native parent of its own, so as long as we pass in something with the right global we're good to go.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: Remove the redundant and somewhat annoying parentObject argument to dictionary ToObject() methods.

I could have sworn I'd posted this patch....

Comment 3

[Boris Zbarsky [:bzbarsky]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e45db0a68b4f

Comment 4

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/e45db0a68b4f