Open
Bug 1387169
Opened 6 years ago
Updated 1 year ago
Remove XPCOM interfaces that have WebIDL equivalents
Categories
(Core :: DOM: Core & HTML, enhancement, P2)
Core
DOM: Core & HTML
Tracking
()
NEW
People
(Reporter: qdot, Unassigned)
References
(Depends on 3 open bugs, Blocks 2 open bugs)
Details
With the deprecation of XPCOM based extensions happens in Firefox 57, we can remove XPCOM components that have equivalent WebIDL interfaces/implementations. This cleanup will happen in 2 steps, based on timing. - During the period before 57 moves to beta, we can gut XPIDL interfaces, which will remove a good bit of code both generated and hand-written. This can be done as cleanup while working on other bugs that may touch code near the interfaces. - After 57 moves to beta (2017-09-20 according to https://wiki.mozilla.org/RapidRelease/Calendar), we can be sure that no extensions will be using the interfaces, and can remove them completely from chrome js and kComponentsInterfaceShimMap.
Updated•6 years ago
|
![]() |
||
Comment 1•6 years ago
|
||
Note: the things appearing in kInterfaceShimMap may need telemetry to remove, so we should think about adding such telemetry...
Reporter | ||
Comment 2•6 years ago
|
||
(In reply to Boris Zbarsky [:bz] from comment #1) > Note: the things appearing in kInterfaceShimMap may need telemetry to > remove, so we should think about adding such telemetry... Filed as bug 1387178
Reporter | ||
Updated•6 years ago
|
Comment 3•6 years ago
|
||
(In reply to Kyle Machulis [:qdot] [:kmachulis] (if a patch has no decent commit message, automatic r-) from comment #0) > - After 57 moves to beta (2017-09-20 according to > https://wiki.mozilla.org/RapidRelease/Calendar), we can be sure that no > extensions will be using the interfaces, and can remove them completely from > chrome js and kComponentsInterfaceShimMap. Why do we have to wait until 57 moves to beta?
Updated•6 years ago
|
Priority: -- → P2
Updated•6 years ago
|
Reporter | ||
Comment 4•6 years ago
|
||
Gonna assume that dependency removal was accidental?
Comment 5•6 years ago
|
||
Um, yes, sorry about that :S I'm not sure how I managed to do that.
Updated•6 years ago
|
Updated•6 years ago
|
Reporter | ||
Updated•4 years ago
|
Assignee: kyle → nobody
Updated•1 year ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•