Add a way to know if an addon is disabled by user or because not supported
Categories
(GeckoView :: Extensions, enhancement, P1)
Tracking
(firefox74 fixed)
Tracking | Status | |
---|---|---|
firefox74 | --- | fixed |
People
(Reporter: agi, Assigned: agi)
References
Details
(Whiteboard: [geckoview:m74])
Attachments
(3 files)
We need to know why the addon was disabled (user disabled vs the addon is incompatible with the app) and for the app a way to set an addon as disabled because it's not compatible with the app.
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Comment 2•4 years ago
|
||
This allows embedding applications to disable addons independently of the user
disabled field.
Assignee | ||
Comment 3•4 years ago
|
||
Updated•4 years ago
|
Assignee | ||
Comment 4•4 years ago
|
||
Pushed by asferro@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/15bf31fa86b9 Use destructuring params in updateAddonDisabledState. r=mixedpuppy https://hg.mozilla.org/integration/autoland/rev/34eace2047b4 Add embedderDisabled field to addon. r=mixedpuppy https://hg.mozilla.org/integration/autoland/rev/a135f610a311 Implement disable(APP_DISABLED) for WebExtension. r=snorp
Comment 6•4 years ago
|
||
Backed out for perma fails gv-junit perma fails.
Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=284755660&resultStatus=testfailed%2Cbusted%2Cexception&revision=a135f610a311b8229bcdccd6b3a441603073b01a
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=284755660&repo=autoland&lineNumber=11308
Backout: https://hg.mozilla.org/integration/autoland/rev/d1406439c4614f7e84fab2cc2e2630018dd4b942
Pushed by asferro@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f05c8bbfa9a7 Use destructuring params in updateAddonDisabledState. r=mixedpuppy https://hg.mozilla.org/integration/autoland/rev/8efb33009288 Add embedderDisabled field to addon. r=mixedpuppy https://hg.mozilla.org/integration/autoland/rev/caa6177248f7 Implement disable(APP_DISABLED) for WebExtension. r=snorp
Assignee | ||
Comment 8•4 years ago
|
||
Thanks! Classic last minute change that breaks the test.
Comment 9•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f05c8bbfa9a7
https://hg.mozilla.org/mozilla-central/rev/8efb33009288
https://hg.mozilla.org/mozilla-central/rev/caa6177248f7
Description
•