New issue
Advanced search Search tips

Issue 746017 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Hangouts doesn't let me share my screen when using ExtensionSettings policy

Project Member Reported by nrpeter@chromium.org, Jul 18 2017

Issue description

Chrome Version: (copy from chrome://version)
OS: Linux

What steps will reproduce the problem?
(1) Start Chrome with no ExtensionSettings policy
(2) Apply ExtensionSettings policy to block extensions with the "desktopCapture" permission
(3) Start a meeting with Google Hangouts and try to present my screen.


What is the expected result?
The screen selector dialog would popup

What happens instead?
The screen selector dialog never pops up and I can't share my screen.

Details:
The use of Hangouts for meetings relies on a Component extension that uses the "desktopCapture" permission. When a policy update of ExtensionSettings occurs, ExtensionService looks for all currently loaded extensions that declare a blacklisted permission. If an extension is using a blacklisted permission, it revokes that active permission. The ExtensionService fails to check whether the extension must remain enabled. This leads to the Hangouts component having its active permission disabled and unable to run. 
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 19 2017

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

commit 0203be4af060be6e50ced42d50590f2b2ad0aa39
Author: Nick Peterson <nrpeter@google.com>
Date: Wed Jul 19 00:25:14 2017

Prevent revocation of active permissions for components when ExtensionSettings policy updates.

Bug:  746017 
Change-Id: Ie5d523be0d14323d6fb0e55eaf19efd196b8e100
Reviewed-on: https://chromium-review.googlesource.com/576514
Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Commit-Queue: Nick Peterson <nrpeter@google.com>
Cr-Commit-Position: refs/heads/master@{#487686}
[modify] https://crrev.com/0203be4af060be6e50ced42d50590f2b2ad0aa39/chrome/browser/extensions/extension_service.cc
[modify] https://crrev.com/0203be4af060be6e50ced42d50590f2b2ad0aa39/chrome/browser/extensions/extension_service_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment