1966113 - Allow chrome.management.setEnabled for enterprise policy extensions

Status: RESOLVED FIXED

(WebExtensions :: General, enhancement)

Description

[Mike Kaply [:mkaply]]

Chrome documentation: https://developer.chrome.com/extensions/management#method-setEnabled

Creating a separate bug for just the enterprise implementation of setEnabled so we don't close the main bug.

Comment 1

[Mike Kaply [:mkaply]]

Attachment: Bug 1966113 - Allow setEnabled for policy installed extensions. r=robwu!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b91dfc46189f
Allow setEnabled for policy installed extensions. r=robwu

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a9e71301c96f
Revert "Bug 1966113 - Allow setEnabled for policy installed extensions. r=robwu" for xpcshell failures on test_ext_management_policy.js

Comment 4

[Narcis Beleuzu [:NarcisB]]

Backed out for xpcshell failures on test_ext_management_policy.js

Backout link: https://hg-edge.mozilla.org/integration/autoland/rev/a9e71301c96f2357970c718b44e4992265451df0
Log link: https://treeherder.mozilla.org/logviewer?job_id=508078907&repo=autoland&lineNumber=3811

Comment 5

[Mike Kaply [:mkaply]]

Needed to disable test for Android.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b2c3c158a36e
Allow setEnabled for policy installed extensions. r=robwu

Comment 7

[Ryan VanderMeulen [:RyanVM]]

I'm not going to track this as something that absolutely must ship in 139, but I'm happy to consider an uplift request if it's ready to go in time.

Comment 8

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/b2c3c158a36e

Comment 9

[Mike Kaply [:mkaply]]

Attachment: Bug 1966113 - Allow setEnabled for policy installed extensions. r=robwu!

Original Revision: https://phabricator.services.mozilla.com/D247771

Comment 10

[Mike Kaply [:mkaply]]

Attachment: Bug 1966113 - Allow setEnabled for policy installed extensions. r=robwu!

Original Revision: https://phabricator.services.mozilla.com/D247771

Comment 11

[Phabricator Automation]

firefox-beta Uplift Approval Request

• User impact if declined: Functionality unavailable to enterprise addons
• Code covered by automated testing: yes
• Fix verified in Nightly: yes
• Needs manual QE test: no
• Steps to reproduce for manual QE testing: N/A
• Risk associated with taking this patch: Low
• Explanation of risk level: Seldom used feature, policy only
• String changes made/needed: None
• Is Android affected?: no

Comment 12

[Phabricator Automation]

firefox-esr128 Uplift Approval Request

• User impact if declined: Functionality unavailable to enterprise addons
• Code covered by automated testing: yes
• Fix verified in Nightly: yes
• Needs manual QE test: no
• Steps to reproduce for manual QE testing: N/A
• Risk associated with taking this patch: Low
• Explanation of risk level: Seldom used feature, policy only
• String changes made/needed: No
• Is Android affected?: no

Comment 13

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/a1c6ce860f85

Comment 14

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-esr128/rev/6a9063738159