952872 - Make XPCWrappedNative a skippable class

Status: RESOLVED INCOMPLETE

(Core :: XPConnect, defect)

Description

[Andrew McCreight [:mccr8]]

In my regular CC browsing log, I see about 1500 XPCWNs without any children, which is almost half of the normal graph.  It shouldn't be that hard to remove these from the graph.

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Make XPCWrappedNative a skippable class. r=smaug

Comment 2

[Andrew McCreight [:mccr8]]

Though maybe you aren't really saving any time (here and in XPCWJS) because you have to do a virtual method call to save a virtual method call.  Maybe rather than being a CanSkip class, XPCWN and XPCWJS should do a non-virtual call to check if they can be skipped when adding them as roots.  Which is what XPCWJS did before.

Comment 3

[Andrew McCreight [:mccr8]]

XPCWN CCing is kind of a mess so there are probably better ways to spend time.