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

Issue 608623 link

Starred by 3 users
Status: Fixed
Owner:
chrishtr@chromium.org
Closed: May 2016
Cc:
nyerramilli@chromium.org
█ ashej...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Glimpse of scroll bar and black patch is seen in browser sample app.

Reported by vku...@etouch.net, May 3 2016 
Chrome Version:52.0.2723.0 (Official Build)Revision 5e216d9b62275101c30f4a5cb2174bb68970ef3c-refs/heads/master@{#391139}  (64-bit)
OS:Mac OSX(10.10.5, 10.11.4) 

Test Url: https://chrome.google.com/webstore/detail/browser-sample/edggnmnajhcbhlnpjnogkjpghaikidaa

What steps will reproduce the problem?
1.Launch chrome and add 'browser sample' from above mentioned url.
2.Click on app and observe the page, click on 'simulate crash' button such page crash
3.Now click on reload button and observe.

Actual:
1.Glimpse of scroll bar is seen while opening browser sample app.(i.e after step 2)
2.Black patch is observe after clicking on reload button.

Expected:
1.Glimpse of scroll bar should not be seen while opening browser sample app.
2.Black patch should not appear after clicking on reload button.


This is a regression issue broken in 'M52' and will soon update other info.

Comment 1 by vku...@etouch.net, May 3 2016

Labels: hasbisect OS-Linux OS-Windows
Owner: chrishtr@chromium.org
Status: Assigned (was: Unconfirmed)
Manual regression range:
Good Build: 52.0.2722.0
Bad Build:  52.0.2723.0

Narrow bisect:
https://chromium.googlesource.com/chromium/src/+log/337bc794f9d8282a047150ac6270d93c70ed5d3a..eb72314ba51288a9e0b8b351286a375d6065763f?pretty=fuller&n=10

Suspecting: 390865 ?
Kindly help to re-assign, if your changes are not cause for this issue.

Note: Issue is seen on Windows (7, 8, 10) & Linux (ubuntu 14.04 LTS)
Actual_Apps.mov
874 KB Download
Expected_Apps.mov
1.2 MB Download

Comment 2 by chrishtr@chromium.org, May 4 2016 

Confirmed that my CL caused this issue to exhibit.

I'm not 100% sure what the root cause is, but I think this is likely a problem with
the implementation of <webview>, where it doesn't deal with this situation properly.

Comment 3 by ashej...@chromium.org, May 4 2016

Labels: ReleaseBlock-Stable
Marking the above issue as RB-Stable as this is a recent regression.

Feel free to change accordingly.

Thank you!
Project Member

Comment 4 by bugdroid1@chromium.org, May 4 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d5766b3e7b9476d7427f0f5034352ff0c874abfd

commit d5766b3e7b9476d7427f0f5034352ff0c874abfd
Author: chrishtr <chrishtr@chromium.org>
Date: Wed May 04 18:14:27 2016

Revert of Start out deferring commits in WebViewImpl. (patchset #2 id:20001 of https://codereview.chromium.org/1934513004/ )

Reason for revert:
Caused 608623 and 609033

Original issue's description:
> Start out deferring commits in WebViewImpl.
>
> This is analogous to the call in WebViewImpl::setRootGraphicsLayer when deleting
> the root graphics layer (as a proxy for detecting page navigation), but for when
> starting up a new WebViewImpl.
>
> BUG= 470669 
>
> Committed: https://crrev.com/eb72314ba51288a9e0b8b351286a375d6065763f
> Cr-Commit-Position: refs/heads/master@{#390865}

TBR=wangxianzhu@chromium.org,enne@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 470669 , 608623 , 609033 

Review-Url: https://codereview.chromium.org/1952673002
Cr-Commit-Position: refs/heads/master@{#391563}

[modify] https://crrev.com/d5766b3e7b9476d7427f0f5034352ff0c874abfd/third_party/WebKit/Source/web/WebViewImpl.cpp

Comment 5 by chrishtr@chromium.org, May 4 2016

Status: Fixed (was: Assigned)
►

Sign in to add a comment