Add Reflect.parse support to xpcshell

RESOLVED FIXED in mozilla9

Status

()

Core
XPConnect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: WeirdAl, Assigned: WeirdAl)

Tracking

Trunk
mozilla9
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
In FF Aurora, Firefox chrome code can access the Reflect API's.  The xpcshell application cannot.
(Assignee)

Comment 1

6 years ago
I think I found the right place to insert the needed code:
http://hg.mozilla.org/mozilla-central/annotate/b57d50c6c046/js/src/xpconnect/shell/xpcshell.cpp#l1971

We seem to be creating a few helpful top-level objects here.

Corresponding code in the JS shell:
http://hg.mozilla.org/mozilla-central/annotate/b57d50c6c046/js/src/shell/js.cpp#l5304
(Assignee)

Comment 2

6 years ago
Created attachment 559590 [details] [diff] [review]
patch, v1
Assignee: nobody → ajvincent
Status: NEW → ASSIGNED
Attachment #559590 - Flags: review?(jorendorff)
Attachment #559590 - Flags: review?(jorendorff) → review+
(Assignee)

Comment 3

6 years ago
I don't have commit privileges, and I'm not sure if this should go on mozilla-inbound, mozilla-central, or tracemonkey first.
Keywords: checkin-needed
(Assignee)

Updated

6 years ago
Attachment #559590 - Flags: review?(bzbarsky)
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
Comment on attachment 559590 [details] [diff] [review]
patch, v1

r=me
Attachment #559590 - Flags: review?(bzbarsky) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
There is no TraceMonkey, only Inbound.

http://hg.mozilla.org/integration/mozilla-inbound/rev/b384cfab8b1d
Keywords: checkin-needed
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/b384cfab8b1d
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
You need to log in before you can comment on or make changes to this bug.