Remove JS_ConstructObject* APIs

RESOLVED FIXED in mozilla16

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Waldo, Assigned: Waldo)

Tracking

({dev-doc-complete})

unspecified
mozilla16
dev-doc-complete
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [js:t])

Attachments

(1 attachment, 1 obsolete attachment)

These methods don't act like anything people would be familiar with (new Ctor(...), more specifically).  They're only used by E4X internals in all of Mozilla.  We should remove them.

I sent a query out to the newgroup to see if anyone's using these, and if so how to address any use case they might have (however awkwardly) been fulfilling:

https://groups.google.com/forum/?fromgroups#!topic/mozilla.dev.tech.js-engine/5xqxa2IAOoQ
Whiteboard: [js:t]
Created attachment 627066 [details] [diff] [review]
Patch

Some people are using these APIs, and I'm still working through trying to migrate them to something better (probably JS_New for the most part).  Just posting this to get it in the database for now...
Created attachment 628783 [details] [diff] [review]
Unbitrotted

One response in the newsgroups we determined could be worked through with JS_New.  The other response mentioned they used the methods but never responded to my response asking if JS_New wouldn't work to address them.  So going by that, one embedder has a reasonable workaround, and another hasn't said he doesn't.

At some point you have to move forward, so let's put this up for review now.  (I sent one last newsgroup ping noting I was putting the patch up for review, for anyone who missed the earlier bits of the thread.)
Attachment #627066 - Attachment is obsolete: true
Attachment #628783 - Flags: review?(dmandelin)
Attachment #628783 - Flags: review?(dmandelin) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/9fa952692a8c

I marked these methods as obsolete:

https://developer.mozilla.org/en/SpiderMonkey/JSAPI_Reference/JS_ConstructObject
https://developer.mozilla.org/en/SpiderMonkey/JSAPI_Reference#section_8

I also noted these methods' removal in release notes:

https://developer.mozilla.org/en/SpiderMonkey/1.8.8
Keywords: dev-doc-complete
Target Milestone: --- → mozilla16

Comment 4

5 years ago
https://hg.mozilla.org/mozilla-central/rev/9fa952692a8c
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.