Closed Bug 741005 Opened 8 years ago Closed 8 years ago

XPCNativeWrapperMap appears to be completely unused

Categories

(Core :: XPConnect, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla14

People

(Reporter: terrence, Assigned: terrence)

Details

Attachments

(1 file)

Attached patch v0: lol wut?Splinter Review
I am able to build locally (linux64) with this set of symbols removed and I don't see any other references either locally or on MXR.  Lets see what try thinks.

https://tbpl.mozilla.org/?tree=Try&rev=b17b4630e814
Comment on attachment 611072 [details] [diff] [review]
v0: lol wut?

Try seems fine with it, flagging review.
Attachment #611072 - Flags: review?(mrbkap)
Comment on attachment 611072 [details] [diff] [review]
v0: lol wut?

Yeah, we don't have XPCNativeWrappers anymore. They've been replaced with XrayWrappers, which are ProxyHandlers.
Attachment #611072 - Flags: review?(mrbkap) → review+
Ah, excellent!  Thank you for the explanation.  I'll go ahead and get this in.
https://hg.mozilla.org/mozilla-central/rev/1ba6065d8202
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
You need to log in before you can comment on or make changes to this bug.