Issue metadata
Sign in to add a comment
|
Leaving fullscreen breaks chrome home url bar |
||||||||||||||||||||||
Issue descriptionChrome Version: 61.0.3118.0 OS: Android 7 What steps will reproduce the problem? (1) Enable chrome home by going to about:flags and making "Chrome Home" enabled. Restart browser. (2) Visit https://www.reddit.com/r/videos/comments/5iyjwk/fans_at_a_hockey_game_go_crazy_for_a_little_kid/ (3) Play the video, and click the fullscreen button (4) Try to exit fullscreen (notice the buttons don't work), hit the back button and notice the url bar is permanently gone. This affects all fullscreen videos for me. What is the expected result? Entering and exiting fullscreen should not break the url bar.
,
Jun 6 2017
Bisect Information: Good build: 61.0.3117.0 Bad build: 61.0.3118.0 Regression range: https://chromium.googlesource.com/chromium/src/+log/61.0.3117.0..61.0.3118.0?pretty=fuller&n=10000pretty=fuller&n=10000 Good commit: 476231 Bad commit: 476232 Culprit CL: 476232 https://chromium.googlesource.com/chromium/src/+/1da41371886c46b3e47fefcc20b40b2c569fe460"
,
Jun 6 2017
,
Jun 6 2017
,
Jun 7 2017
This also affects top controls. In the attached screenshot, the toolbar is not mid-scroll, that is the default position. The bottom of the page is also cut off. Also worth noting is that the previous tab I had open is now missing.
,
Jun 9 2017
Now this bug only applies when the FullscreenActivity feature is manually enabled.
,
Jun 14 2017
,
Jun 20 2017
,
Jul 11 2017
Is PiP shipping in M61 (or below)? If so, can we please get this fixed for M61 to unblock more Chrome Home experimentation?
,
Jul 12 2017
twellington@, PIP is shipping but the FullscreenActivity isn't (see comment #6).
,
Jul 14 2017
OK great - does this mean we can punt any fixes to M62?
,
Jul 18 2017
,
Aug 10 2017
,
Aug 22 2017
It'd be great to get a fix in for this for M62 (assuming it still reproduces)
,
Aug 22 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by candr...@chromium.org
, Jun 6 2017Labels: triage-te