Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(37)

Issue 599903002: Force extension media access to obey enterprise policy. (Closed)

Created:
6 years, 2 months ago by pastarmovj
Modified:
6 years, 2 months ago
CC:
chromium-reviews, posciak+watch_chromium.org, feature-media-reviews_chromium.org, wjia+watch_chromium.org, mcasas+watch_chromium.org
Base URL:
http://git.chromium.org/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Force extension media access to obey enterprise policy. BUG=415964 TEST=Install an extension like "Webcam" and make sure it doesn't work when policy is enabled. Committed: https://crrev.com/fdfc92d59fe0dc0ca50795773cf62f6e054c9837 Cr-Commit-Position: refs/heads/master@{#297383}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -4 lines) Patch
M chrome/browser/media/media_capture_devices_dispatcher.cc View 2 chunks +11 lines, -4 lines 0 comments Download

Messages

Total messages: 17 (6 generated)
pastarmovj
Hi Victoria, since I saw your name in a todo in the function I am ...
6 years, 2 months ago (2014-09-24 13:20:11 UTC) #2
perkj_chrome
This looks like something grunell would now about since he just did refactoring in this ...
6 years, 2 months ago (2014-09-26 14:10:54 UTC) #4
pastarmovj
On 2014/09/26 14:10:54, perkj wrote: > This looks like something grunell would now about since ...
6 years, 2 months ago (2014-09-29 11:23:51 UTC) #6
Henrik Grunell
lgtm
6 years, 2 months ago (2014-09-29 12:07:39 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/599903002/1
6 years, 2 months ago (2014-09-29 12:16:09 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/14070)
6 years, 2 months ago (2014-09-29 12:23:21 UTC) #11
pastarmovj
Seems like Henrik is not in the list of owners. Victoria can you please stamp ...
6 years, 2 months ago (2014-09-29 12:29:37 UTC) #12
no longer working on chromium
On 2014/09/29 12:29:37, pastarmovj wrote: > Seems like Henrik is not in the list of ...
6 years, 2 months ago (2014-09-30 08:31:48 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/599903002/1
6 years, 2 months ago (2014-09-30 08:33:12 UTC) #15
commit-bot: I haz the power
Committed patchset #1 (id:1) as c5880d5b53c0a5f200e18d3aa233c2784c52c705
6 years, 2 months ago (2014-09-30 08:37:58 UTC) #16
commit-bot: I haz the power
6 years, 2 months ago (2014-09-30 08:38:39 UTC) #17
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/fdfc92d59fe0dc0ca50795773cf62f6e054c9837
Cr-Commit-Position: refs/heads/master@{#297383}

Powered by Google App Engine
This is Rietveld 408576698