Closed Bug 869928 Opened 7 years ago Closed 7 years ago

Remove Off-Main-Thread XPCWrappedJS refcounting from OpenSignedJARFileTask

Categories

(Core :: Security, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: jdm, Assigned: jdm)

References

Details

Attachments

(1 file)

Comment on attachment 746944 [details] [diff] [review]
Remove Off-Main-Thread XPCWrappedJS refcounting from OpenSignedJARFileTask.

Honza, if you want to race Brian for this review, that would please me too. Bobby's keen to get off-main-thread XPWrappedJS refcounting aborts enabled before the merge next week, and this is the only patch still blocking that.
Attachment #746944 - Flags: review?(honzab.moz)
Comment on attachment 746944 [details] [diff] [review]
Remove Off-Main-Thread XPCWrappedJS refcounting from OpenSignedJARFileTask.

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

I verified that this usage of the new API matches the documentation given in the new API's header file (from https://bugzilla.mozilla.org/page.cgi?id=splinter.html&bug=771074&attachment=639268).
Attachment #746944 - Flags: review?(honzab.moz)
Attachment #746944 - Flags: review?(bsmith)
Attachment #746944 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/7fa500e12a38
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.