Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1079)

Unified Diff: chrome/browser/cocoa/autocomplete_text_field_cell.mm

Issue 175055: [Mac] Fix Omnibox keyword-search color. (Closed)
Patch Set: Indeed, something HAD changed. Expect a poor merge, change it back. Created 11 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/cocoa/autocomplete_text_field_cell.mm
diff --git a/chrome/browser/cocoa/autocomplete_text_field_cell.mm b/chrome/browser/cocoa/autocomplete_text_field_cell.mm
index 7125774a0cbda6c3896d8f2bce03e9bd895ce257..e065c14ae9bf5b72c5e5d85d13b48e45dace262c 100644
--- a/chrome/browser/cocoa/autocomplete_text_field_cell.mm
+++ b/chrome/browser/cocoa/autocomplete_text_field_cell.mm
@@ -245,18 +245,17 @@ const NSInteger kBaselineOffset = 4;
NSMinX(textFrame) - x,
cellFrame.size.height - 2 * kKeywordYInset));
- // Draw a token rectangle with rounded corners.
+ // Draw the token rectangle with rounded corners.
NSRect frame(NSInsetRect(infoFrame, 0.5, 0.5));
NSBezierPath* path =
[NSBezierPath bezierPathWithRoundedRect:frame xRadius:4.0 yRadius:4.0];
- [[NSColor controlColor] set];
+ // Matches the color of the highlighted line in the popup.
+ [[NSColor selectedControlColor] set];
[path fill];
- GTMTheme *theme = [controlView gtm_theme];
- NSColor* stroke = [theme strokeColorForStyle:GTMThemeStyleToolBarButton
- state:YES];
- [stroke setStroke];
+ // Border around token rectangle, match focus ring's inner color.
+ [[[NSColor keyboardFocusIndicatorColor] colorWithAlphaComponent:0.5] set];
[path setLineWidth:1.0];
[path stroke];
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698