Blink of white patch is seen at the bottom of the page when video is set to full screen.
Reported by
mni...@etouch.net,
Apr 13 2016
|
||||
Issue descriptionChrome Version : 52.0.2707.0 52d5c52fa34e5d312e2622b7799111bb63781403-refs/heads/master@{#386876} (32/64-bit) OS : Windows(Win 7 aero enabled),Linux. Url : https://www.duracell.com/en-us/program/duracell-star-wars-commercial-the-battle-for-christmas-morning/ What steps will reproduce the problem? 1. Launch chrome and navigate to above url. 2. Now click on 'play video' button,video will start. 3. Now click on 'Full screen' icon and observe at the bottom of the page Actual: Blink of white patch is seen at the bottom of the page when video is set to full screen. Expected: Blink of white patch should not be seen at the bottom of the page when video is set to full screen. This is a regression issue broken in 'M-51' and below is the manual regression and Narrow bisect info : Good Build: 51.0.2674.0 Bad Build: 51.0.2675.0 Narrow bisect: https://chromium.googlesource.com/chromium/src/+log/a2a4fab1693d6985dda4de9e91825fd667f07598..40e8aff900f1c7be05b3e6d68aae1504e116a8d8?pretty=fuller&n=100 Suspecting: r380739 from Narrow bisect @jbauman : Kindly help to re-assign, if your changes are not cause for this issue. Note : Issue is not seen on Mac OS.
,
Apr 13 2016
It looks similar to behavior on 51.0.2699 and 49. The page contents moves up when the chrome disappears, revealing unrendered parts of the background page before the video covers it. in 51 it appears the contents are lighter, hence more noticeable in this case. This appears to be related to subtle differences in handling the shade. It also varies in effect on earlier versions. Removing regression states.
,
Apr 13 2016
I can see a flash of content in 49.0.2623.112 (Official Build) (64-bit) (see grey block at bottom of the screen). Interestingly, if I have dev tools open (in a separate window) that is even a flash of white as has been observed in the bug. Seems like it is a question of timing whether or not you see a white flash but in either case we seem to reveal part of the page which is not yet available before the video goes fullscreen.
,
Apr 13 2016
Setting the background of the body to black causes black to fill, just as the CL would suggest.
,
May 12 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/41663d972e8f55b6f2876eb59329c41465157b7e commit 41663d972e8f55b6f2876eb59329c41465157b7e Author: jbauman <jbauman@chromium.org> Date: Thu May 12 00:13:06 2016 Use black for resize gutter in tab fullscreen mode. This avoids the possibility of flashing white in the gutter area, which is more obtrusive than flashing black. BUG= 602991 , 607415 Review-Url: https://codereview.chromium.org/1884043002 Cr-Commit-Position: refs/heads/master@{#393112} [modify] https://crrev.com/41663d972e8f55b6f2876eb59329c41465157b7e/content/browser/renderer_host/delegated_frame_host.cc [modify] https://crrev.com/41663d972e8f55b6f2876eb59329c41465157b7e/content/browser/renderer_host/delegated_frame_host.h [modify] https://crrev.com/41663d972e8f55b6f2876eb59329c41465157b7e/content/browser/renderer_host/render_widget_host_view_aura.cc [modify] https://crrev.com/41663d972e8f55b6f2876eb59329c41465157b7e/content/browser/renderer_host/render_widget_host_view_aura.h [modify] https://crrev.com/41663d972e8f55b6f2876eb59329c41465157b7e/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc [modify] https://crrev.com/41663d972e8f55b6f2876eb59329c41465157b7e/content/browser/renderer_host/render_widget_host_view_mac.h [modify] https://crrev.com/41663d972e8f55b6f2876eb59329c41465157b7e/content/browser/renderer_host/render_widget_host_view_mac.mm
,
May 16 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/181dffac401cb970900e3c41943ac1156a2974fa commit 181dffac401cb970900e3c41943ac1156a2974fa Author: John Bauman <jbauman@chromium.org> Date: Mon May 16 21:13:28 2016 Use black for resize gutter in tab fullscreen mode. This avoids the possibility of flashing white in the gutter area, which is more obtrusive than flashing black. BUG= 602991 , 607415 TBR=piman@chromium.org Review-Url: https://codereview.chromium.org/1884043002 Cr-Commit-Position: refs/heads/master@{#393112} (cherry picked from commit 41663d972e8f55b6f2876eb59329c41465157b7e) Review URL: https://codereview.chromium.org/1980163002 . Cr-Commit-Position: refs/branch-heads/2704@{#567} Cr-Branched-From: 6e53600def8f60d8c632fadc70d7c1939ccea347-refs/heads/master@{#386251} [modify] https://crrev.com/181dffac401cb970900e3c41943ac1156a2974fa/content/browser/renderer_host/delegated_frame_host.cc [modify] https://crrev.com/181dffac401cb970900e3c41943ac1156a2974fa/content/browser/renderer_host/delegated_frame_host.h [modify] https://crrev.com/181dffac401cb970900e3c41943ac1156a2974fa/content/browser/renderer_host/render_widget_host_view_aura.cc [modify] https://crrev.com/181dffac401cb970900e3c41943ac1156a2974fa/content/browser/renderer_host/render_widget_host_view_aura.h [modify] https://crrev.com/181dffac401cb970900e3c41943ac1156a2974fa/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc [modify] https://crrev.com/181dffac401cb970900e3c41943ac1156a2974fa/content/browser/renderer_host/render_widget_host_view_mac.h [modify] https://crrev.com/181dffac401cb970900e3c41943ac1156a2974fa/content/browser/renderer_host/render_widget_host_view_mac.mm
,
May 16 2016
,
May 18 2016
Retested the above issue on Windows & Linux with chrome version '51.0.2704.54' and working as intended and no white patch is seen while playing the video. Hence marking the same as TE-Verified-51.0.2704.54. Thank you! |
||||
►
Sign in to add a comment |
||||
Comment 1 by nyerramilli@chromium.org
, Apr 13 2016