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

Issue 607415 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
not on Chrome anymore
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Weird white patch is observed on video fullscreen in play.google.com.

Reported by lpa...@etouch.net, Apr 28 2016

Issue description

Chrome Version: 52.0.2719.0 (Official Build) db7757b63f29696b9b465949f9704a0d9d149f2b-refs/heads/master@{#390251} 32/64 Bit.
OS: Windows 10
Pre-condition: Go to chrome://components >> Widevine Content Decryption Module >> click 'Check for update'  (IF REQUIRED)

Steps:
1. Launch chrome and navigate to https://play.google.com/store/movies/details/Joy?id=0-OIT3q21_8&hl=en
2. Play the video and click on full screen icon.
3. Observe the video when it goes in full screen mode.

Actual: Weird white patch is observed at the bottom of the video while it goes in fullscreen mode.

Expected: No such white patch should be seen.

This is a regression issue broken in M-51, will soon update the other info.
 

Comment 1 by tkent@chromium.org, Apr 28 2016

Components: -Blink>Fullscreen UI>Browser>FullScreen

Comment 2 by lpa...@etouch.net, Apr 28 2016

Cc: tkonch...@chromium.org
Labels: hasbisect OS-Linux
Owner: spqc...@chromium.org
Status: Assigned (was: Unconfirmed)
Manual Regression Range:
Good Build: 51.0.2674.0
Bad Build: 51.0.2675.0

Providing change log url as the issue is not reproducible in chromium builds:
https://chromium.googlesource.com/chromium/src/+log/51.0.2674.0..51.0.2675.0?pretty=fuller&n=10000

Suspecting: 380524

Issue is reproducible in win 10,8,8.1 and Linux (ubuntu 14.04 LTS)
Note: Issue is not seen in Mac OS.
 

Labels: ReleaseBlock-Stable
Adding RB label as this is a recent regression.
Owner: ----
Status: Untriaged (was: Assigned)
Unassigning self because R380524 only affects OSX and fullscreening videos inside popups

Comment 5 by lpa...@etouch.net, May 5 2016

Owner: jbau...@chromium.org
Status: Assigned (was: Untriaged)
Re-bisected on other machine and got the following narrow bisect:
https://chromium.googlesource.com/chromium/src/+log/a2a4fab1693d6985dda4de9e91825fd667f07598..40e8aff900f1c7be05b3e6d68aae1504e116a8d8?pretty=fuller&n=100

Suspecting: 380739 ?

Please help to re-assign if your change is not the cause of this issue. 
Yeah, this was caused by my change. I'm working on a fix that special-cases fullscreen transitions.
A friendly reminder that M51 Stable is launching soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch by May 17. All changes MUST be merged into the release branch by 5pm on May 20 to make into the desktop Stable final build cut. Thanks!
Just an update:

The issue is still reproducible on win8.1 chrome version canary 52.0.2730.0
Project Member

Comment 9 by bugdroid1@chromium.org, May 12 2016

Cc: rnimmagadda@chromium.org
Unable to repro this issue on Windows 7 for Google Chrome Canary Version - 52.0.2738.0 

Screen-recording is attached.

@lpanse: Could you please confirm the same and update the thread accordingly.

Thank you.
607415.mp4
1.9 MB Download
Labels: Needs-Feedback

Comment 12 Deleted

M51 Stable is launching very soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged ASAP. All changes MUST be merged into the release branch by 5pm on May 20 to make into the desktop Stable final build cut. Thank you!
Labels: Merge-Request-51

Comment 15 by tin...@google.com, May 16 2016

Labels: -Merge-Request-51 Merge-Approved-51 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M51 (branch: 2704)
Please merge your change to M51 branch 2704 before 5:00 PM PST today, Monday so we can take it for this week beta release. Thank you.
Project Member

Comment 17 by bugdroid1@chromium.org, May 16 2016

Labels: -merge-approved-51 merge-merged-2704
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

Status: Fixed (was: Assigned)

Comment 19 by ajha@chromium.org, May 18 2016

Labels: -Needs-Feedback TE-Verified-51.0.2704.54 TE-Verified-M51
Verified the merge on the latest M-51(51.0.2704.54) on Windows-10 and Linux Ubuntu 14.0. This is working as intended.

Sign in to add a comment