New issue
Advanced search Search tips

Issue 758785 link

Starred by 1 user
Status: Fixed
Owner:
karandeepb@chromium.org
Closed: Aug 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Reflect the fact that some extensions can't be enabled as part of Management API.

Project Member Reported by karandeepb@chromium.org, Aug 25 2017 
https://chromium-review.googlesource.com/c/chromium/src/+/611662 changed how ManagementPolicy::UserMayLoad is handled. As a result,
installed extensions blacklisted by management policy are now disabled (instead
of remaining silently installed). These extensions now also show up as part of
chrome.management.getAll(). Therefore, we need to add some info to the ExtensionInfo returned as part of the Management API, to allow clients to identify this case.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 25 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e795bd9fa191bf9b563c5f17fa6aabcb762dc933

commit e795bd9fa191bf9b563c5f17fa6aabcb762dc933
Author: Karan Bhatia <karandeepb@chromium.org>
Date: Fri Aug 25 03:18:33 2017

Management API: Add a "mayEnable" field to ExtensionInfo.

r496521 changed how ManagementPolicy::UserMayLoad is handled. As a result,
installed extensions blacklisted by management policy are now disabled (instead
of remaining silently installed). These extensions now also show up as part of
chrome.management.getAll(). 

To help distinguish this new case, this CL adds a new field "mayEnable" to the
ExtensionInfo returned as part of the Management API. This is only returned for
disabled extensions.

BUG= 758785 

Change-Id: I15fc82d1cfb70252c63900214f4a973f90359a52
Reviewed-on: https://chromium-review.googlesource.com/631556
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Karan Bhatia <karandeepb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#497311}
[modify] https://crrev.com/e795bd9fa191bf9b563c5f17fa6aabcb762dc933/chrome/browser/extensions/api/management/management_api_unittest.cc
[modify] https://crrev.com/e795bd9fa191bf9b563c5f17fa6aabcb762dc933/extensions/browser/api/management/management_api.cc
[modify] https://crrev.com/e795bd9fa191bf9b563c5f17fa6aabcb762dc933/extensions/common/api/management.json

Comment 2 by karandeepb@chromium.org, Aug 25 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment