Add an option to deep-freeze the result of Cu.cloneInto

RESOLVED FIXED in Firefox 42

Status

()

Core
XPConnect
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Bobby Holley (parental leave - send mail for anything urgent), Assigned: Bobby Holley (parental leave - send mail for anything urgent))

Tracking

(Blocks: 1 bug, {dev-doc-needed})

unspecified
mozilla42
dev-doc-needed
Points:
---

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(1 attachment)

Tim ran into this in bug 1184439. You can't freeze an XrayWrapper, and we don't want people to have to waive and then iterate on the waiver freezing stuff.

We can solve this easily with an API on Cu.cloneInto.
Keywords: dev-doc-needed
Created attachment 8636820 [details] [diff] [review]
Add an API to deep-freeze the result of Cu.cloneInto. v1
Attachment #8636820 - Flags: review?(gkrizsanits)
Attachment #8636820 - Flags: feedback?(jimb)
https://treeherder.mozilla.org/#/jobs?repo=try&revision=3828587979a3
Tim: Assuming there are no major review nits, feel free to push this on my behalf while I'm on PTO.
Attachment #8636820 - Flags: review?(gkrizsanits) → review+
Keywords: checkin-needed

Comment 4

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/1cdbaf4b9807
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/1cdbaf4b9807
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox42: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42

Updated

2 years ago
Attachment #8636820 - Flags: feedback?(jimb) → feedback+
You need to log in before you can comment on or make changes to this bug.