Closed
Bug 1354205
Opened 8 years ago
Closed 7 years ago
Remove FUEL
Categories
(Toolkit :: General, enhancement)
Toolkit
General
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: mossop, Assigned: dao)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
This is only used by old-style add-ons and so once they have gone it can be removed.
Reporter | ||
Updated•8 years ago
|
Blocks: post-57-api-changes
Is this already done by bug 1090880?
Di you mean extIApplication here?
Reporter | ||
Comment 2•8 years ago
|
||
(In reply to :aceman from comment #1)
> Is this already done by bug 1090880?
> Di you mean extIApplication here?
That and https://dxr.mozilla.org/mozilla-central/source/toolkit/mozapps/extensions/test/xpcshell/test_fuel.js
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•7 years ago
|
||
Wait, I thought extApplication.js was not supposed to be removed. Or was that just meant that it will be removed much later than FUEL itself, so e.g. now?
If so, we use it in TB and need to remove STEEL that uses extApplication.js. We have the patch ready in bug 1278067.
And of course Seamonkey with SMILE.
Assignee | ||
Comment 6•7 years ago
|
||
(In reply to :aceman from comment #5)
> Wait, I thought extApplication.js was not supposed to be removed.
Why not?
> If so, we use it in TB and need to remove STEEL that uses extApplication.js.
> We have the patch ready in bug 1278067.
> And of course Seamonkey with SMILE.
Right, it seems that you should just get rid of those too. Or if there's some reason against doing that, pull extApplication.js into Thunderbird/Seamonkey.
It may be that it just wasn't yet sure what the plan is 2 years ago (bug 1090880 comment 3).
We do not use STEEL anymore in TB core, we've done preparations this time. Just some addons still use it. Then again many addons still use the long removed FUEL :)
Reporter | ||
Comment 8•7 years ago
|
||
mozreview-review |
Comment on attachment 8893391 [details]
Bug 1354205 - Remove FUEL remnants.
https://reviewboard.mozilla.org/r/164510/#review169976
Looks good.
Attachment #8893391 -
Flags: review?(dtownsend) → review+
Reporter | ||
Comment 9•7 years ago
|
||
(In reply to :aceman from comment #7)
> It may be that it just wasn't yet sure what the plan is 2 years ago (bug
> 1090880 comment 3).
> We do not use STEEL anymore in TB core, we've done preparations this time.
> Just some addons still use it. Then again many addons still use the long
> removed FUEL :)
Do you want us to defer landing this for a day or two to import this file to comm-central?
Flags: needinfo?(acelists)
Comment 10•7 years ago
|
||
Yes, please. We are coming to an agreement in bug 1278067.
Please land this on Monday if possible. Thanks.
Flags: needinfo?(acelists)
Comment 11•7 years ago
|
||
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1d7af093298b
Remove FUEL remnants. r=mossop
Comment 12•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Comment 13•7 years ago
|
||
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/838073194362
Port bug 1354205 to TB/IB/SM: remove exthelper.xpt from package manifests. rs=bustage-fix
You need to log in
before you can comment on or make changes to this bug.
Description
•