Closed
Bug 1451532
Opened 7 years ago
Closed 6 years ago
Convert (or remove) remaining bootstrapped extensions used in automation
Categories
(Toolkit :: Add-ons Manager, enhancement, P2)
Toolkit
Add-ons Manager
Tracking
()
RESOLVED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox65 | --- | fixed |
People
(Reporter: aswan, Assigned: aswan)
References
(Depends on 1 open bug)
Details
Attachments
(5 files, 1 obsolete file)
This should probably eventually be broken down into multiple smaller bugs but ae have a bunch of bootstrapped extensions used in various places in automated tests that need to be converted to webextensions:
devtools/client/aboutdebugging/test/addons/unpacked
devtools/client/debugger/test/mochitest/addon-source/*
dom/plugins/test/testaddon
dom/workers/test/extensions/bootstrap
testing/mozbase/mozprofile/tests/addons/empty
toolkit/components/search/tests/xpcshell/data
toolkit/components/telemetry/tests/addons/*
plus of course a bunch of stuff in toolkit/mozapps/extensions/test/ some of which will obviously need to last as long as we support bootstrap
Updated•7 years ago
|
Priority: -- → P2
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → aswan
Assignee | ||
Comment 1•6 years ago
|
||
Julian, you mentioned in another bug that you'll be removing devtools support for debugging legacy extension. Is there a bug for that work and do you have a timeline for it?
Flags: needinfo?(jdescottes)
Comment 2•6 years ago
|
||
This is tracked in Bug 1497264. It is not prioritized for now, but we might clean this in 65.
It would simplify the ongoing fission work, since that would mean one less client to refactor. It would also help with the current effort around remote debugging, if we don't have to migrate the features dedicated to legacy extensions.
On top of that, for the devtools folders mentioned in the summary:
- devtools/client/debugger/test/mochitest/addon-source/* is getting removed in Bug 1314057 already
- current aboutdebugging tests are migrated to a new UI in Bug 1489226, so we can also get rid of devtools/client/aboutdebugging/test/addons/unpacked during the migration. Should happen in 65
Flags: needinfo?(jdescottes)
Assignee | ||
Comment 3•6 years ago
|
||
browser_doorhanger_installs.js is due for a more comprehensive pass to
weed out obsolete tests but for now this patch just gets rid of any
bootstrapped extensions it was using.
Updated•6 years ago
|
Attachment #9019511 -
Attachment is obsolete: true
Assignee | ||
Comment 4•6 years ago
|
||
Assignee | ||
Comment 5•6 years ago
|
||
Assignee | ||
Comment 6•6 years ago
|
||
Assignee | ||
Comment 7•6 years ago
|
||
This is a quick pass to weed out some obsolete tests and update others.
browser_doorhanger_installs.js still needs a more comprehensive pass to
get rid of obsolete test cases.
Assignee | ||
Comment 8•6 years ago
|
||
In most case this involves switching to webextensions. Some old or
redundant tests are removed, other tets are cleaned up and modernized
along the way.
Updated•6 years ago
|
Attachment #9024933 -
Attachment description: Bug 1451532 Remove unneeded bootstrapped extensions in addon manager tests r=kmag → Bug 1451532 Remove unneeded bootstrapped extensions in addon manager xpcshell tests r=kmag
Assignee | ||
Comment 9•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/c5751a5efe5beaa4dc82089bc5ede81ee35f3291
Bug 1451532 Remove unused test addon r=kmag
https://hg.mozilla.org/integration/mozilla-inbound/rev/f1a4ed8b794571fb9e8c3b7509b5aac5ec11282b
Bug 1451532 Remove/replace legacy extensions in telemetry tests r=gfritzsche
https://hg.mozilla.org/integration/mozilla-inbound/rev/27ace8afa15d5379b503aaab800251adf1e90a4f
Bug 1451532 Convert test dictionary to modern packaging r=mstriemer
https://hg.mozilla.org/integration/mozilla-inbound/rev/77e400a477280361e26b3b2d2c72416e0861eb95
Bug 1451532 Clean up bootstrapped extensions in addon manager mochitests r=kmag
https://hg.mozilla.org/integration/mozilla-inbound/rev/c1d4918380adbf58e0fb231fb8de6940fce5f7c3
Bug 1451532 Remove unneeded bootstrapped extensions in addon manager xpcshell tests r=kmag
Assignee | ||
Comment 10•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8cef663e7653bb1939fa81a0e5ca79ea959808d3
Bug 1451532 follow-up: fix eslint on CLOSED TREE r=me
Comment 11•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c5751a5efe5b
https://hg.mozilla.org/mozilla-central/rev/f1a4ed8b7945
https://hg.mozilla.org/mozilla-central/rev/27ace8afa15d
https://hg.mozilla.org/mozilla-central/rev/77e400a47728
https://hg.mozilla.org/mozilla-central/rev/c1d4918380ad
https://hg.mozilla.org/mozilla-central/rev/8cef663e7653
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in
before you can comment on or make changes to this bug.
Description
•