JSNative->JSFunction fixup should happen in DefinePropertyById, not DefineProperty

RESOLVED FIXED in mozilla14

Status

()

RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: bholley, Assigned: bholley)

Tracking

unspecified
mozilla14
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
We need this stuff in the dom bindings, but it seems silly to convert our JSID to a char* just to take advantage of this. Patch coming right up.
(Assignee)

Comment 1

7 years ago
Created attachment 610601 [details] [diff] [review]
patch. v1

Attaching a patch. Flagging luke for review.
Attachment #610601 - Flags: review?(luke)

Updated

7 years ago
Attachment #610601 - Flags: review?(luke) → review+
(Assignee)

Comment 3

7 years ago
Looks like the try oranges were the result of the patch underneath, which was backed out on inbound.

Pushed to m-i: http://hg.mozilla.org/integration/mozilla-inbound/rev/5ed5a3e3acc4
Target Milestone: --- → mozilla14

Comment 4

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