Open Bug 1451959 Opened 7 years ago Updated 2 years ago

Investigate shimming less interfaces

Categories

(Core :: DOM: Core & HTML, enhancement, P3)

enhancement

Tracking

()

People

(Reporter: mccr8, Unassigned)

References

Details

While looking at the patches in bug 1444745, I wonder if we can drop some shims from ShimInterfaceInfo.cpp now that we don't need to support addons. I don't see any uses of nsISelection in .js or .jsm files, for instance. I do see nsIDOMElement. (Note that bug 1444745 is changing how shimming is done, but it will be easy enough to update the patch if that is done.)
We can do that, probably. I haven't been worried about it too much because it wasn't really hurting anything and we were getting rid of stuff from here by removing the interfaces anyway, and in particular this list was serving as a sort of guide for interfaces to try to remove. But if there are gains to be had from removing things from this list, I'm all for it; we should just file bugs on removing the relevant interfaces if we do that, and link them all up to bug 1387169.
Priority: -- → P3
Bug 1444745 removes some shims already, so I'll mark it as a dependence.
Depends on: 1444745
Component: DOM → DOM: Core & HTML
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.