New issue
Advanced search Search tips

Issue 817322 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Bug



Sign in to add a comment

Check for locked fullscreen permission on non-CrOS platforms

Project Member Reported by isandrk@chromium.org, Feb 28 2018

Issue description

Currently the check is done only on ChromeOS, so the behaviour is a bit weird on eg. Windows Chrome where if you request a chrome.windows.create({state: 'locked-fullscreen'}) you will get a fullscreen window instead of a locked-fullscreen (and no errors), which is a bit confusing. Make it also error out on non-CrOS platforms.

So the locked-fullscreen functionality is available only on CrOS, and on other platforms it silently degenerates into regular fullscreen behaviour (which is probably mildly confusing to users). 
 

Comment 1 by minz@google.com, Mar 1 2018

Cc: minz@google.com
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 2 2018

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

commit 89d107ff8326d5bf36df6bc99a7112c1dc219cf8
Author: Ivan Sandrk <isandrk@google.com>
Date: Fri Mar 02 08:29:56 2018

Enable locked fullscreen permission check on non-CrOS platforms

The locked-fullscreen functionality is available only on CrOS, but on other platforms it silently degenerates into regular fullscreen behaviour which is probably mildly confusing to users. Make the behaviour consistent across platforms, so also error out on non-CrOS platforms (same as CrOS behaviour).

Bug:  chromium:817322 
Change-Id: Idcbb80beeae896f981321264334dce30d8d3c845
Reviewed-on: https://chromium-review.googlesource.com/941125
Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Commit-Queue: Ivan Ĺ andrk <isandrk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#540473}
[modify] https://crrev.com/89d107ff8326d5bf36df6bc99a7112c1dc219cf8/chrome/browser/extensions/api/tabs/tabs_api.cc

Status: Fixed (was: Started)
Labels: OS-Linux OS-Mac OS-Windows
Labels: -OS-Chrome

Sign in to add a comment