diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-13 20:22:49 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-13 20:22:49 +0000 |
| commit | 83a4075694df07811f995653615c8685bacef3ce (patch) | |
| tree | 5085ecf85717710ae1970282dfeb168dcaf0d54b /chrome/browser/browser_theme_provider.cc | |
| parent | ebcb20c7ead50898319724aa42efb0e31acf1877 (diff) | |
| download | chromium_src-83a4075694df07811f995653615c8685bacef3ce.zip chromium_src-83a4075694df07811f995653615c8685bacef3ce.tar.gz chromium_src-83a4075694df07811f995653615c8685bacef3ce.tar.bz2 | |
Use predefined color names where possible for clarity. Also use "SkColorSetARGB(0, 0, 0, 0)" in place of "NULL" since it's not obvious what the latter means as a color.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/273042
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28874 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_theme_provider.cc')
| -rw-r--r-- | chrome/browser/browser_theme_provider.cc | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/chrome/browser/browser_theme_provider.cc b/chrome/browser/browser_theme_provider.cc index 4e1562e..85fcd15 100644 --- a/chrome/browser/browser_theme_provider.cc +++ b/chrome/browser/browser_theme_provider.cc @@ -102,28 +102,26 @@ const SkColor BrowserThemeProvider::kDefaultColorFrameIncognitoInactive = SkColorSetRGB(126, 139, 156); const SkColor BrowserThemeProvider::kDefaultColorToolbar = SkColorSetRGB(210, 225, 246); -const SkColor BrowserThemeProvider::kDefaultColorTabText = - SkColorSetRGB(0, 0, 0); +const SkColor BrowserThemeProvider::kDefaultColorTabText = SK_ColorBLACK; const SkColor BrowserThemeProvider::kDefaultColorBackgroundTabText = SkColorSetRGB(64, 64, 64); const SkColor BrowserThemeProvider::kDefaultColorBookmarkText = SkColorSetRGB(18, 50, 114); -const SkColor BrowserThemeProvider::kDefaultColorNTPBackground = - SkColorSetRGB(255, 255, 255); -const SkColor BrowserThemeProvider::kDefaultColorNTPText = - SkColorSetRGB(0, 0, 0); +const SkColor BrowserThemeProvider::kDefaultColorNTPBackground = SK_ColorWHITE; +const SkColor BrowserThemeProvider::kDefaultColorNTPText = SK_ColorBLACK; const SkColor BrowserThemeProvider::kDefaultColorNTPLink = SkColorSetRGB(6, 55, 116); const SkColor BrowserThemeProvider::kDefaultColorNTPHeader = SkColorSetRGB(75, 140, 220); const SkColor BrowserThemeProvider::kDefaultColorNTPSection = SkColorSetRGB(229, 239, 254); -const SkColor BrowserThemeProvider::kDefaultColorNTPSectionText = - SkColorSetRGB(0, 0, 0); +const SkColor BrowserThemeProvider::kDefaultColorNTPSectionText = SK_ColorBLACK; const SkColor BrowserThemeProvider::kDefaultColorNTPSectionLink = SkColorSetRGB(6, 55, 116); -const SkColor BrowserThemeProvider::kDefaultColorControlBackground = NULL; -const SkColor BrowserThemeProvider::kDefaultColorButtonBackground = NULL; +const SkColor BrowserThemeProvider::kDefaultColorControlBackground = + SkColorSetARGB(0, 0, 0, 0); +const SkColor BrowserThemeProvider::kDefaultColorButtonBackground = + SkColorSetARGB(0, 0, 0, 0); // Default tints. const color_utils::HSL BrowserThemeProvider::kDefaultTintButtons = |