Remove XPCOM interfaces that have WebIDL equivalents

NEW
Unassigned

Status

()

enhancement
P2
normal
2 years ago
2 months ago

People

(Reporter: qdot, Unassigned)

Tracking

(Depends on 4 bugs, Blocks 2 bugs)

Firefox Tracking Flags

(Not tracked)

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.
Depends on: 1381615, 1380415
Note: the things appearing in kInterfaceShimMap may need telemetry to remove, so we should think about adding such telemetry...
(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
No longer blocks: 1387184
Depends on: 1387184
(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?
Depends on: 1390209
Priority: -- → P2
Depends on: 792808
No longer depends on: 1404079
Gonna assume that dependency removal was accidental?
Um, yes, sorry about that :S  I'm not sure how I managed to do that.
No longer depends on: 1405162
Depends on: 1436864
Depends on: 1436865
Depends on: 1436869
Depends on: 1437135
Depends on: 830221
No longer blocks: 1418078
Depends on: 1418078
Blocks: 1452353
No longer blocks: 1452353
Depends on: 1452353
Depends on: 1462964
Depends on: 1463015
Depends on: 1463016
Depends on: 1524683
Assignee: kyle → nobody
You need to log in before you can comment on or make changes to this bug.