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

Issue 629995 link

Starred by 4 users

Issue metadata

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



Sign in to add a comment

Wrong colors on EV cert chip when dragging incognito tab

Reported by jleedev@gmail.com, Jul 20 2016

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.82 Safari/537.36

Steps to reproduce the problem:
1. Open incognito window
2. Visit EV cert page (github.com)
3. Begin dragging tab out of window

What is the expected behavior?
The dark theme should stay consistent.

What went wrong?
The background is dark grey and the lock is white but the text is green, which looks weird.

Did this work before? N/A 

Chrome version: 52.0.2743.82  Channel: beta
OS Version: OS X 10.11.6
Flash Version:
 
Screen Shot 2016-07-20 at 18.08.21.png
27.7 KB View Download

Comment 1 by rsesek@chromium.org, Jul 21 2016

Components: -UI UI>Browser>Omnibox>OriginChip
Labels: Proj-MaterialDesign-NativeUI

Comment 2 by shrike@chromium.org, Jul 21 2016

Cc: shrike@chromium.org
Owner: spqc...@chromium.org
spqchan@ - please take a look at this, but it's probably something we won't fix given the Omnibox changes that are in flight?

Status: Assigned (was: Unconfirmed)
Sure, I can look into this. I fixed a similar issue (the alert indicator being dragged) and this is something we should fix since the Omnibox changes won't solve it
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 15 2016

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

commit f54c0238952086f5cf5c3dce0f5bdbf88a6d4d72
Author: spqchan <spqchan@chromium.org>
Date: Mon Aug 15 18:33:27 2016

[Material][Mac] Fixed EV chip's dark theme colors

Use hasDarkTheme instead of inIncognitoModeWithSystemTheme so that the chip gets the correct value.

BUG= 629995 

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

[modify] https://crrev.com/f54c0238952086f5cf5c3dce0f5bdbf88a6d4d72/chrome/browser/ui/cocoa/location_bar/bubble_decoration.mm

Status: Fixed (was: Assigned)

Sign in to add a comment