New issue
Advanced search Search tips

Issue 602223 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug-Regression



Sign in to add a comment

Regression: Unnecessary patch is observe while closing permission dialogue box on ‘Chrome Apps Extensions Developer Tool’

Reported by dmascare...@etouch.net, Apr 11 2016

Issue description

Chrome Version :Version:52.0.2705.0 (Official Build) fefaf92686d3786bd74fbda74d7dff3ca2f49e3e-refs/heads/master@{#386318} 
OS: MAC

Pre-condition:
https://chrome.google.com/webstore/detail/chrome-apps-extensions-de/ohmmkhmmmpcnpikjeljgnaoabkaalbgc/related?utm_source=chrome-ntp-launcher

What steps will reproduce the problem?
1. Launch chrome and dd the app using above url.
2. Launch the app and click on ‘Permission’ link of any app
3. Click on ‘Close’ button and observe the overlay/dialogue box while closing.

Actual: Unnecessary patch is observed after step 3.
Expected: Patch should not be observed.

This is regression issue,broken in ‘M 47’ and below is narrow bisect:
https://chromium.googlesource.com/chromium/src/+log/2087f15fb276483dcc40e97c7d1a4813f689f7a2..0984b53f543ea04db60b324d491a129594329c3e?pretty=fuller&n=100

Suspecting: r350706

Good build:47.0.2518.0
Bad build:47.0.2519.0

Could you please help to reassign if your change is not the cause for this change?

Note: Issue is not seen on Windows and Linux OS.

 
Actual_patch.mov
1.9 MB Download
Expected_patch.mov
690 KB Download
Components: -Blink>Animation UI>Aura
This bug does not involve Blink animations.

Comment 2 by tapted@chromium.org, Apr 12 2016

Cc: erikc...@chromium.org
Components: -UI>Aura
Labels: -Pri-1 Pri-3
Heh, this isn't an aura animation either.

For some reason Cocoa isn't fading out the NSScrollView in step with the window. In r350706 that picked up an explicit layer to fix subpixel AA and not break other things. If this only affects the Chrome Apps & Extensions Developer Tool, I don't think it's worth fixing.

"eventually" this dialog will be replaced with a toolkit-views one.

There's also Issue 567494 which is kinda related to this.
Project Member

Comment 3 by sheriffbot@chromium.org, Jun 1 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 4 by sheriffbot@chromium.org, Jul 14 2016

Labels: -M-53 MovedFrom-53
This issue has been moved once and is lower than Pri-1. Removing the milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment