Closed
Bug 1886216
Opened 2 years ago
Closed 2 years ago
Remove unused tabExtrasActor.jsm
Categories
(Firefox for Android :: General, task)
Tracking
()
RESOLVED
FIXED
126 Branch
| Tracking | Status | |
|---|---|---|
| firefox126 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
(Whiteboard: [esmification-timeline])
Attachments
(1 file)
We have been working on moving mozilla-central away from using JSMs and have got down to a few remaining (6). With the merge of android to mozilla-central, this has now increased by 1 due to the addition of tabExtrasActor.jsm.
Thomas/Dennis, do you know if we can do a straight conversion, or are there other considerations we need to take in to account?
Flags: needinfo?(twisniewski)
Flags: needinfo?(dschubert)
|
||
Comment 1•2 years ago
|
||
Actually, we can remove that script entirely, it's no longer used or needed after our recent Report Broken Site updates, we just missed it in the related cleanup (we now query the built-in ReportBrokenSite actor to do the same thing). So just removing that jsm and the related line in the esmify map should suffice, based on a quick build and test on my end.
Flags: needinfo?(twisniewski)
Flags: needinfo?(dschubert)
|
Assignee | |
Comment 2•2 years ago
|
||
|
||
Updated•2 years ago
|
Assignee: nobody → standard8
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•2 years ago
|
Summary: Convert tabExtrasActor.jsm to an ES module → Remove unused tabExtrasActor.jsm
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/23309344058e
Remove unused tabExtrasActor.jsm. r=twisniewski
|
||
Comment 4•2 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox126:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 126 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•