986940 - Remove Cu.createArrayIn and Cu.createDateIn

Status: RESOLVED FIXED

(Core :: XPConnect, defect)

Description

[Bobby Holley (:bholley)]

In bug 933681, I set things up so that we resolve standard classes on Xrayed globals. This means that you can now safely do:

new contentWindow.Date();
new contentWindow.Array();
new contentWindow.Object();

(Note that I also implemented eval, so you can do contentWindow.eval(foo) as well).

This means that Cu.createObjectIn, Cu.createArrayIn, and Cu.createDateIn are no longer very useful. The latter two are not widely used, and I think we can remove them. createObjectIn is more widely used, and has a few extra features, so I'll leave it for now.

Cu.createDataIn is not used in addons. Cu.createArrayIn is used by the b2g simulator, so I'm CCing myk to make sure he knows this is coming.

Comment 1

[Bobby Holley (:bholley)]

(also note that Cu.cloneInto is a more powerful API that also subsumes all of these).

Comment 2

[Bobby Holley (:bholley)]

https://tbpl.mozilla.org/?tree=Try&rev=df3b5d13a402

Comment 3

[Bobby Holley (:bholley)]

Attachment: Remove Cu.createDateIn. v1

When we switch ObjectWrapper.jsm to use Cu.cloneInto, this became unused.

Comment 4

[Bobby Holley (:bholley)]

Attachment: Remove Cu.createArrayIn. v1

Comment 5

[Gabor Krizsanits (INACTIVE)]

(In reply to Bobby Holley (:bholley) from comment #0)
> In bug 933681, I set things up so that we resolve standard classes on Xrayed
> globals. This means that you can now safely do:
> 
> new contentWindow.Date();
> new contentWindow.Array();
> new contentWindow.Object();
> 
> (Note that I also implemented eval, so you can do contentWindow.eval(foo) as
> well).
> 
> This means that Cu.createObjectIn, Cu.createArrayIn, and Cu.createDateIn are
> no longer very useful. The latter two are not widely used, and I think we
> can remove them. createObjectIn is more widely used, and has a few extra
> features, so I'll leave it for now.
> 
> Cu.createDataIn is not used in addons. Cu.createArrayIn is used by the b2g
> simulator, so I'm CCing myk to make sure he knows this is coming.

Would be great to see a one liner summary of this under Components.utils.createObjectIn on MDN... And then we could slowly deprecate createObjectIn... and note Cu.cloneInto ofc, it's a very handy function.

Comment 6

[Bobby Holley (:bholley)]

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/0f8fbc309505
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/936b03713f6c


(In reply to Gabor Krizsanits [:krizsa :gabor] from comment #5)
> Would be great to see a one liner summary of this under
> Components.utils.createObjectIn on MDN... And then we could slowly deprecate
> createObjectIn... and note Cu.cloneInto ofc, it's a very handy function.

Will seems to be the owner of these docs.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/0f8fbc309505
https://hg.mozilla.org/mozilla-central/rev/936b03713f6c

Comment 8

[Will Bamberg [:wbamberg]]

Sorry this took so long. I've added a note in https://developer.mozilla.org/en-US/docs/Components.utils.createObjectIn, let me know if it works for you.