Closed Bug 1451532 Opened 2 years ago Closed Last year

Convert (or remove) remaining bootstrapped extensions used in automation

Categories

(Toolkit :: Add-ons Manager, enhancement, P2)

enhancement

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
Priority: -- → P2
Assignee: nobody → aswan
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)
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)
See Also: → 1497264
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.
Attachment #9019511 - Attachment is obsolete: true
Depends on: 1502558
Depends on: 1505148
Depends on: 1500975
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.
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.
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
Depends on: 1508664
You need to log in before you can comment on or make changes to this bug.