Closed
Bug 1487323
Opened 7 years ago
Closed 3 years ago
Remove isLegacyTemporaryExtension check from aboutdebugging
Categories
(DevTools :: about:debugging, enhancement, P3)
DevTools
about:debugging
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: jdescottes, Unassigned)
References
Details
Attachments
(2 obsolete files)
With the release of Firefox 63, the ESR version will be bumped to Fx 60. The following check and warning in about debugging is only needed for older runtimes than Fx59, so we can remove this logic.
https://searchfox.org/mozilla-central/rev/2fe43133dbc774dda668d4597174d73e3969181a/devtools/client/aboutdebugging/components/addons/Target.js#126-140
Blocking on Bug 1487078 since I will be moving the module where the helper is defined in this bug.
|
Reporter | |
Comment 1•7 years ago
|
||
Depends on D4647
|
||
Updated•7 years ago
|
Attachment #9005131 -
Attachment is obsolete: true
|
|
Reporter | |
Comment 2•7 years ago
|
||
Actually only the check on actor.type can be removed. The legacy warning needs to stay as long as legacy extensions can still be developed.
|
|
Reporter | |
Comment 3•7 years ago
|
||
Depends on D4647
|
||
Comment 4•7 years ago
|
||
Comment on attachment 9005133 [details]
Bug 1487323 - Remove deprecated backward-compat test in isLegacyTemporaryExtension;r=daisuke
Daisuke Akatsuka (:daisuke) has approved the revision.
Attachment #9005133 -
Flags: review+
|
|
||
Updated•7 years ago
|
Attachment #9005133 -
Attachment is obsolete: true
|
|
Reporter | |
Updated•3 years ago
|
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•