Closed Bug 868765 Opened 7 years ago Closed 7 years ago

Make the nsDependentJSString constructor take a Handle<jsid>

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: Ms2ger, Assigned: bzbarsky)

References

Details

Attachments

(2 files)

No description provided.
Attachment #745604 - Flags: review?(Ms2ger)
Assignee: nobody → bzbarsky
Whiteboard: [need review]
Comment on attachment 745604 [details] [diff] [review]
Hey, this compiles

I'd hope that an empty patch compiles!
Attachment #745604 - Flags: review?(Ms2ger)
Comment on attachment 745632 [details] [diff] [review]
Make the jsid constructor for nsDependentJSString take a Handle.

Review of attachment 745632 [details] [diff] [review]:
-----------------------------------------------------------------

If it does indeed compile, excellent :)
Attachment #745632 - Flags: review?(Ms2ger) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/79ce2f354769
Flags: in-testsuite-
Whiteboard: [need review]
Target Milestone: --- → mozilla23
https://hg.mozilla.org/mozilla-central/rev/79ce2f354769
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.