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

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

Issue 196123: Make sure the text color is set correctly for incognito themes until we theme... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' 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/browser_window_controller.mm
===================================================================
--- chrome/browser/cocoa/browser_window_controller.mm (revision 26207)
+++ chrome/browser/cocoa/browser_window_controller.mm (working copy)
@@ -1281,12 +1281,25 @@
theme = [[GTMTheme alloc] init]; // "Leak" it in the cache.
cache[key] = theme;
+
+ // TODO(pinkerton): Need to be able to theme the entire incognito window
+ // http://crbug.com/18568 The hardcoding of the colors here will need to be
+ // removed when that bug is addressed, but are here in order for things to be
+ // usable in the meantime.
if (isOffTheRecord) {
NSColor* incognitoColor = [NSColor colorWithCalibratedRed:83/255.0
green:108.0/255.0
blue:140/255.0
alpha:1.0];
[theme setBackgroundColor:incognitoColor];
+ [theme setValue:[NSColor blackColor]
+ forAttribute:@"textColor"
+ style:GTMThemeStyleToolBar
+ state:GTMThemeStateActiveWindow];
+ [theme setValue:[NSColor blackColor]
+ forAttribute:@"textColor"
+ style:GTMThemeStyleTabBarDeselected
+ state:GTMThemeStateActiveWindow];
return theme;
}
« 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