Remove XPCOM interfaces that have WebIDL equivalents

NEW
Assigned to

Status

()

enhancement
P2
normal
2 years ago
3 months ago

People

(Reporter: qdot, Assigned: qdot)

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.
(Assignee)

Updated

2 years ago
Depends on: 1380413
Depends on: 1381615, 1380415
Note: the things appearing in kInterfaceShimMap may need telemetry to remove, so we should think about adding such telemetry...
(Assignee)

Updated

2 years ago
Depends on: 1387178
(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
(Assignee)

Updated

2 years ago
Blocks: 1387184
(Assignee)

Updated

2 years ago
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?
(Assignee)

Updated

2 years ago
Blocks: deCOM
(Assignee)

Updated

2 years ago
Depends on: 1389650
Depends on: 1390209
Priority: -- → P2
(Assignee)

Updated

2 years ago
Depends on: 1402102
(Assignee)

Updated

2 years ago
Depends on: 1402180
(Assignee)

Updated

2 years ago
Depends on: 1403795
(Assignee)

Updated

2 years ago
Depends on: 1403819
(Assignee)

Updated

2 years ago
Depends on: 1404079

Updated

2 years ago
Depends on: 792808
No longer depends on: 1404079
(Assignee)

Updated

2 years ago
Depends on: 1404079
Gonna assume that dependency removal was accidental?

Comment 5

2 years ago
Um, yes, sorry about that :S  I'm not sure how I managed to do that.
(Assignee)

Updated

2 years ago
Depends on: 1405162
(Assignee)

Updated

2 years ago
No longer depends on: 1405162
(Assignee)

Updated

2 years ago
Depends on: 1405792
(Assignee)

Updated

2 years ago
Depends on: 1406224
(Assignee)

Updated

2 years ago
Depends on: 1406525
(Assignee)

Updated

2 years ago
Depends on: 1407040
(Assignee)

Updated

2 years ago
Depends on: 1408169
(Assignee)

Updated

2 years ago
Depends on: 659610
(Assignee)

Updated

2 years ago
Depends on: 1408186
(Assignee)

Updated

2 years ago
Depends on: 1412414
(Assignee)

Updated

2 years ago
Depends on: 1412437
(Assignee)

Updated

2 years ago
Depends on: 1415716
(Assignee)

Updated

2 years ago
Depends on: 1415747
(Assignee)

Updated

2 years ago
Depends on: 1416069
(Assignee)

Updated

2 years ago
Depends on: 1418076
(Assignee)

Updated

2 years ago
Depends on: 1418077
(Assignee)

Updated

2 years ago
Blocks: 1418078
(Assignee)

Updated

2 years ago
Depends on: 1418085
Depends on: 1436864
Depends on: 1436865
Depends on: 1436869
Depends on: 1437135
Depends on: 830221
Depends on: 1438024
Depends on: 1438270
Depends on: 1438956
No longer blocks: 1418078
Depends on: 1418078
Depends on: 1440039
Depends on: 1440441
Depends on: 1387143
Depends on: 1445408
Depends on: 1446515
Depends on: 1377980
Depends on: 1447389

Updated

a year ago
Blocks: 1452353

Updated

a year ago
No longer blocks: 1452353
Depends on: 1452353
Depends on: 1460735

Updated

11 months ago
Depends on: 1462964

Updated

11 months ago
Depends on: 1463015

Updated

11 months ago
Depends on: 1463016
(Assignee)

Updated

3 months ago
Depends on: 1524683
You need to log in before you can comment on or make changes to this bug.