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

Issue 609033 link

Starred by 4 users
Status: Fixed
Owner:
chrishtr@chromium.org
Closed: May 2016
Cc:
nyerramilli@chromium.org
█ ashej...@chromium.org
mac-bugs-priority@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Weird flickering of the overlay is observed on opening devtools.

Reported by dmascare...@etouch.net, May 4 2016 
Chrome Version:52.0.2724.0 (Official Build) 76d17b826d6473ef7a4bb731aa8b8dc05aaa5ab6-refs/heads/master@{#391399} 64 bit
Os: MAC (10.10.5) (10.11.4)

Test url:
https://chrome.google.com/webstore/detail/google-calendar-by-google/gmbgaklkmjakoegficnlkhebmhkjfich?utm_source=chrome-ntp-icon

Pre-condition: 1.‘Google calendar’ extension should be added to chrome using above test url.
               2.‘Developer mode’ should be ’On’ from ‘chrome://extensions/'

What steps will reproduce the problem?
1. Launch chrome and right click on extension icon such that context menu appears.
2. Select ‘Inspect popup’ and observe.

Actual: Weird flickering is observed on opening devtools.
Expected: Flickering should not be seen.(Devtools should open without flickering)

This is regression issue, broken in ‘M 52’ and will soon update the bisect info.

Good build:52.0.2722.0
Bad build: 52.0.2723.0

Note : Above issue is not reproducible on Windows and Linux OS.
Actual_cal.mov
1.6 MB Download
Exp_cal.mov
1.2 MB Download

Comment 1 by dmascare...@etouch.net, May 4 2016

Labels: hasbisect
Owner: chrishtr@chromium.org
Status: Assigned (was: Unconfirmed)
Correction in step: 
3. Repeat the above step again.

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

Suspecting: r390865 ?

Note: Issue is reproducible on second instant.

Comment 2 by dmascare...@etouch.net, May 4 2016 

Note: Above issue is also reproducible on Mac Retina (10.11.4)

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