Issue metadata
Sign in to add a comment
|
Regression: Unnecessary disabled full screen button is seen for recorded videos in Camera app |
||||||||||||||||||||
Issue descriptionChrome Version:68.0.3245.0/10663.0.0 dev-channel Daisy,Candy,Reks OS:Chrome OS What steps will reproduce the problem? (1)Open Camera app >> Do a sample video recording (2)go to Gallery >>Double click on the recording to open and observe Actual:Unnecessary disabled full screen button is seen for recorded videos Expected: No such unnecessary disabled full screen button should be seen This is a Regression issue seen as same is working fine in 66.0.3359.158/10452.85.0 stable Note: 1.Issue is also seen in M-67 beta 2.Issue is not seen in Linux,Windows OS Attached the screencast for reference.. @Yuli: Please confirm the behavior
,
May 11 2018
shenghao@, it might be a regression from Chrome's video player?
,
May 11 2018
Seems that it's broken between 67 and 66. Conrad, could you route this bug to video player team?
,
May 11 2018
Pawel/Hiro, do you know who owns this?
,
May 11 2018
How do we do full-screen in the expected one? I think we can't. So looks just video player style change. If we would like to enable full-screen, there should be the flag to video-player to enable full-screen.
,
May 11 2018
Camera app explicitly disabled the fullscreen by 'video.setAttribute('controlsList', 'nodownload nofullscreen').'
However, 67 seems no longer respecting the 'nofullscreen' attribute settings.
,
Jul 6
fukino@: would you perhaps be able to triage this issue please? Thank you!
,
Aug 3
This bug has an owner, thus, it's been triaged. Changing status to "assigned".
,
Aug 9
According to comment #6, controlList="nofullscreen" does not hide the fullscreen button in video tag control. steimel@, is this a know issue?
,
Aug 9
This was an intentional change to never hide the fullscreen and only disable it. However, I think we made that change mostly for the preload="none" -> metadata loaded transition where the mute and fullscreen buttons would just appear, and we wanted to keep things from moving around. Since in the case of controlsList="nofullscreen" we'll never have a fullscreen button, it might be worth hiding. +amyroberts@ to make a UI decision on this. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by y...@chromium.org
, May 10 2018