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

Issue 706114 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

[Mac] [Regression?] PopUp Blocker icon in the Omnibox is blurred during the animation

Project Member Reported by meh...@chromium.org, Mar 28 2017

Issue description

Chrome Version: Version 59.0.3054.0 canary (64-bit)
OS: MacOS 10.12.3 / Macbook Air 11" NonRetina 

What steps will reproduce the problem?
(1) go to http://www.popuptest.com/popuptest1.html
(2) take a look at the PopUp Blocker icon during the animation
(3)

What is the expected result?
It should not be blurred.

What happens instead?
It is blurred.

Please use labels and text to provide additional information.
A screenshot is attached.

Not sure if this is a regression.
 
Bildschirmfoto 2017-03-28 um 21.33.02.png
29.1 KB View Download

Comment 1 by meh...@chromium.org, Mar 28 2017

Labels: Needs-Bisect

Comment 2 by ajha@chromium.org, Mar 29 2017

Labels: Needs-Triage-M59
Labels: -Pri-1 -Needs-Bisect -Type-Bug-Regression -Needs-Triage-M59 M-59 Pri-2 Type-Bug
Able to reproduce the issue on Mac-10.12.3 using chrome stable version 57.0.2987.110 and reported 59.0.3054.0 with the steps mentioned in comment#0.
This is Non regression issue observed from M53# 53.0.2744.0 and confirming this to get more inputs from dev team.
From M30 to M52 the PopUp Blocker icon looks different.
Please find the attached screenshot for reference.
Note:Issue is specific to Mac.

Thanks..
706114.png
10.6 KB View Download

Comment 4 by shrike@chromium.org, Mar 30 2017

Owner: shrike@chromium.org
Status: Started (was: Untriaged)

Comment 5 by shrike@chromium.org, Mar 31 2017

For some reason this info did not get attached to the bug.

commit	960575f6f8da2a8557c04adbeccc045cd51034b1
author	shrike <shrike@chromium.org>	Thu Mar 30 16:56:09 2017
committer	Commit bot <commit-bot@chromium.org>	Thu Mar 30 16:56:09 2017
tree	d355d555e96aa19a729bb99060560e24b342e1b3
parent	34b3bb86e0e8490c439cc81537461be312577fb8 [diff]
[Mac] Fix popup blocker animation's blurry end state.

The final state of the omnibox's popup blocker animation is drawn to a
non-integral point boundary, so the animated icon and text are blurry.
This change floors the incoming frame so that the animation is always
drawn to an integral point boundary.

R=avi@chromium.org
BUG= 706114 

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

Project Member

Comment 6 by bugdroid1@chromium.org, Apr 3 2017

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

commit e385704a77bfd6b5fd665bf860ae22beae237802
Author: shrike <shrike@chromium.org>
Date: Mon Apr 03 22:21:59 2017

[Mac] Fix jittery divider during omnibox's popup blocker animation.

A followup to crrev 2784103002 which fixed the popup blocker animation's
blurry end state. This cl addresses the jittery divider line that's
drawn to the right of the popup blocker, also caused by the non-integral
frame rect.

R=avi@chromium.org
BUG= 706114 

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

[modify] https://crrev.com/e385704a77bfd6b5fd665bf860ae22beae237802/chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm

Cc: manoranj...@chromium.org ligim...@chromium.org
Labels: TE-NeedsTriageFromMTV
Tested the issue on 13 inch Non-Retina Mac OS X 10.12.3 using Latest Dev# 59.0.3061.3 and found the animation is looking fine without any blurry font.
Could some one from MTV Team please look into the issue and check on 11" Non-Retina Mac OS X 10.12.3 and update the issue.
Thank You.
The PopUp Blocker Icon and the divider are no longer blurred in today's Version 59.0.3062.0 canary (64-bit). Tested on MacBook 11" Non-Retina. Thanks shrike@.

Status: Fixed (was: Started)
OK, good.

Sign in to add a comment