New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 720304 link

Starred by 4 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Fullscreen (manifest) prevents fullscreen API from working

Project Member Reported by aerotwist@chromium.org, May 10 2017

Issue description

Chrome Version: 60.0.3094.0
OS: Android

What steps will reproduce the problem?
(1) Grab the attached zip, unzip and run.
(2) Add to Home Screen
(3) Run it and tap on the image

What is the expected result?
It should go full screen

What happens instead?
It doesn't. If it's not launched from HS with "display": "fullscreen", or the display is anything else, the FS API works.


 
fs.zip
122 KB Download
Components: UI>Browser>AppShortcuts
Status: Available (was: Untriaged)
This is deliberate. If you set display: fullscreen, we explicitly disable the HTML fullscreen API to avoid any clobbering between the two. There are questions of what the back stack should contain, what UI transitions there should be when you're in display: fullscreen and then you make something on the page fullscreen, etc.

The team doesn't really have cycles to explore whether or not we can resolve all of this right now unfortunately.
Cc: owe...@chromium.org
Does Opera's implementation behave the same? Was this properly documented to web developers? It sounds unfortunate that an API is completely disabled because of how the websites is being launched :(
Fullscreen video playback is a primary video use case. 

Is our recommendation that video sites not use "display" : "fullscreen"?
I don't think that's a viable solution in the long term: two use cases for 'display: fullscreen' are games and VR and they might want to play videos. They might make fake fullscreen by stretching the video to the size of the viewport but that wouldn't be a real future forward solution.
In the (sometime) term, we'll absolutely get the fullscreen API working properly with display: fullscreen. We're just a little stretched at the moment unfortunately. :(

I'm going to be OOO for a week, but I'll try and set aside some time to scope out how complicated this might be when I'm back.

Comment 6 by klo...@chromium.org, May 19 2017

Cc: klo...@chromium.org
Why clobber? Do we use the same fullscreen activity for content for fullscreen web app? Seems we should have a separate one as it is browser level, not content level.
Labels: -Pri-3 M-61 Pri-2
Owner: goog...@chromium.org
Status: Assigned (was: Available)
Over to googleo@ who is aiming to fix this for M61.
Components: UI>Browser>WebAppInstalls
Owner: dominickn@chromium.org
Owner: ----
Status: Available (was: Assigned)
Marking as available for now. We don't have eng resources to tackle this in SYD unfortunately.

Sign in to add a comment