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

Issue 615359 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

Regression:Unnecessary white patch is seen on page after pressing 'Backspace'.

Reported by rk...@etouch.net, May 27 2016

Issue description

Chrome Version: Chrome Version: 53.0.2750.0 Revision c84885e83ecac137b93ed2b6c8b7323d8e91ffe1-refs/heads/master@{#396337}(32/64 bit)
OS: Windows(7,8,10), Linux (14.04 LTS)
    
What steps will reproduce the problem?
(1) Launch chrome, navigate to https://plus.google.com/explore
(2) Click on 'Share' icon of any image, select 'Copy link' option and press 'Backspace'
(3) Observe the white patch on top of the page

Unnecessary white patch is seen on page after pressing 'Backspace'.

No such white patch should seen on page after pressing 'Backspace'.

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

Good Build: 53.0.2749.0
Bad Build: 53.0.2750.0


 
Video_Actual.mp4
520 KB Download

Comment 1 by rk...@etouch.net, May 27 2016

Owner: mgiuca@chromium.org
Status: Assigned (was: Unconfirmed)
Summary: Regression:Unnecessary white patch is seen on page after pressing 'Backspace'. (was: Unnecessary white patch is seen on play.google.com page after pressing 'Backspace'.)
Narrow Bisect:
https://chromium.googlesource.com/chromium/src/+log/0063cf80ae18f025807016c41f17462d51bee872..5783d8c7eb60443d5d31d7e9bbb095d897caf3f0?pretty=fuller&n=50

Suspecting: r396138
Labels: ReleaseBlock-Beta
Marking the above issue as RB-Beta as this is a very recent regression and affecting Google Services. Feel free to lower if someone things otherwise.

Thank you!

Comment 3 by rk...@etouch.net, May 27 2016

Just to update:

Above issue is reproduce with following steps also:

1. Launch chrome, navigate to youtube.com
2. Play any video and click on 'Fullscreen' icon(Press(Esc)to exist fullscreen info bar is appear)
3. After dismissing infobar,Press 'Backspace' and observe.
Video_Expected.mp4
516 KB Download

Comment 4 by mgiuca@chromium.org, May 30 2016

Cc: mgiuca@chromium.org sky@chromium.org
Owner: pkasting@chromium.org
Status: Fixed (was: Assigned)
I'm guessing this was the same issue detected by pkasting and fixed in https://codereview.chromium.org/2016203002/. See  Issue 610039  comments #57 to #60. (pkasting: Please file a separate bug if you detect an issue after a CL lands so it doesn't get jumbled up in the original feature request bug.)

It's kind of messy so I'll give the full story:
1. I had my CL ready to land (in code review) and working fine.
2. sky landed r395980 which changed Widget::SetOpacity which I was using.
3. I landed r396138 hours later which was broken at the time of landing due to unanticipated interaction with sky's CL.
4. pkasting landed r396590 to fix this.

So I expect this would have been broken from r396138 to r396589, and fixed after that. Please verify on a new build.

More complexity: I have to merge r396138 into the M52 branch (which branched before all three of these CLs). Because sky's CL has not landed in the M52 branch, I should be able to merge it without pkasting's fix, but I'll need to test this carefully.

Sign in to add a comment