Closed Bug 1542003 Opened 5 years ago Closed 2 years ago

Drop methods from xptinfo for classes that don't inherit from nsISupports

Categories

(Core :: XPCOM, enhancement, P3)

enhancement

Tracking

()

RESOLVED DUPLICATE of bug 1611173

People

(Reporter: mccr8, Unassigned)

Details

XPCNativeInterface::NewInstance() contains this comment:

// If the interface does not have nsISupports in its inheritance chain
// then we know we can't reflect its methods. However, some interfaces that
// are used just to reflect constants are declared this way. We need to
// go ahead and build the thing. But, we'll ignore whatever methods it may
// have.

I'm a little surprised that we might have interfaces exposed to JS that don't inherit from nsISupports. If this is really the case, we should remove all method information for them, to save some space. If it isn't the case, then we could simplify the code that calls nsXPConnect::IsISupportsDescendant() a tiny bit.

Type: defect → enhancement
Priority: -- → P3

Support for non-nsISupports XPIDL interfaces was removed in bug 1611173.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.