XPCNativeWrapperMap appears to be completely unused

RESOLVED FIXED in mozilla14

Status

()

Core
XPConnect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

Trunk
mozilla14
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 611072 [details] [diff] [review]
v0: lol wut?

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
(Assignee)

Comment 1

5 years ago
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+
(Assignee)

Comment 3

5 years ago
Ah, excellent!  Thank you for the explanation.  I'll go ahead and get this in.
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/1ba6065d8202
https://hg.mozilla.org/mozilla-central/rev/1ba6065d8202
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
You need to log in before you can comment on or make changes to this bug.