New issue
Advanced search Search tips

Issue 651071 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Public Session whitelisting - adjust several permissions

Project Member Reported by isandrk@chromium.org, Sep 28 2016

Issue description

The status quo for the following permissions is enable, but they need to be adjusted as per [1]:
- app.window.fullscreen
- app.window.fullscreen.overrideEsc
- fullscreen
- overrideEscFullscreen


[1]: https://docs.google.com/spreadsheets/d/1ahozeNp9dAde6q2euiFVOnFfosnp9UeizBOq5Wjd6z4/edit#gid=610952148
 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 5 2016

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

commit 454aea3b9268d20b6a8aba76fc8f49fcbb114f9b
Author: isandrk <isandrk@chromium.org>
Date: Wed Oct 05 13:30:37 2016

Public Session whitelisting - updated comments for app.window.* and enabled "app.window.alwaysOnTop" since it's safe.

BUG= 651071 

Review-Url: https://codereview.chromium.org/2391123003
Cr-Commit-Position: refs/heads/master@{#423141}

[modify] https://crrev.com/454aea3b9268d20b6a8aba76fc8f49fcbb114f9b/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc

Status: Fixed (was: Assigned)

Comment 3 by dchan@chromium.org, Oct 7 2016

Labels: VerifyIn-55
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 27 2016

Labels: merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/454aea3b9268d20b6a8aba76fc8f49fcbb114f9b

commit 454aea3b9268d20b6a8aba76fc8f49fcbb114f9b
Author: isandrk <isandrk@chromium.org>
Date: Wed Oct 05 13:30:37 2016

Public Session whitelisting - updated comments for app.window.* and enabled "app.window.alwaysOnTop" since it's safe.

BUG= 651071 

Review-Url: https://codereview.chromium.org/2391123003
Cr-Commit-Position: refs/heads/master@{#423141}

[modify] https://crrev.com/454aea3b9268d20b6a8aba76fc8f49fcbb114f9b/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc

Comment 5 by dimu@google.com, Nov 4 2016

Labels: -merge-merged-2840
[Automated comment] removing mislabelled merge-merged-2840

Comment 6 by dchan@google.com, Nov 19 2016

Labels: VerifyIn-56

Comment 7 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 8 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 9 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 10 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 12 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment