Last Comment Bug 741005 - XPCNativeWrapperMap appears to be completely unused
: XPCNativeWrapperMap appears to be completely unused
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: XPConnect (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla14
Assigned To: Terrence Cole [:terrence]
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-30 16:16 PDT by Terrence Cole [:terrence]
Modified: 2012-04-03 02:04 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
v0: lol wut? (8.20 KB, patch)
2012-03-30 16:16 PDT, Terrence Cole [:terrence]
bobbyholley: review+
Details | Diff | Splinter Review

Description Terrence Cole [:terrence] 2012-03-30 16:16:21 PDT
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
Comment 1 Terrence Cole [:terrence] 2012-04-02 10:46:09 PDT
Comment on attachment 611072 [details] [diff] [review]
v0: lol wut?

Try seems fine with it, flagging review.
Comment 2 Bobby Holley (:bholley) (busy with Stylo) 2012-04-02 11:01:44 PDT
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.
Comment 3 Terrence Cole [:terrence] 2012-04-02 13:03:30 PDT
Ah, excellent!  Thank you for the explanation.  I'll go ahead and get this in.
Comment 4 Terrence Cole [:terrence] 2012-04-02 13:12:32 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/1ba6065d8202
Comment 5 Marco Bonardo [::mak] 2012-04-03 02:04:40 PDT
https://hg.mozilla.org/mozilla-central/rev/1ba6065d8202

Note You need to log in before you can comment on or make changes to this bug.