diff options
| author | tony@chromium.org <tony@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-02 01:15:22 +0000 |
|---|---|---|
| committer | tony@chromium.org <tony@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-02 01:15:22 +0000 |
| commit | a513bce52e152a45fb78c671612c57c23a2efa28 (patch) | |
| tree | 2dee476e35dc54aa6bea8951479d3ed4e4a54a9c /chrome/browser/browser_theme_provider.cc | |
| parent | 5eca47da2ab69c70ed78cabd27bd3cbafaf6c6df (diff) | |
| download | chromium_src-a513bce52e152a45fb78c671612c57c23a2efa28.zip chromium_src-a513bce52e152a45fb78c671612c57c23a2efa28.tar.gz chromium_src-a513bce52e152a45fb78c671612c57c23a2efa28.tar.bz2 | |
Fix the opacity of titlebars on the NTP page. We were accidentally
setting the opacity to 1/255.
This bug was introduced in r32643.
BUG=29107
Review URL: http://codereview.chromium.org/452040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33526 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_theme_provider.cc')
| -rw-r--r-- | chrome/browser/browser_theme_provider.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/browser_theme_provider.cc b/chrome/browser/browser_theme_provider.cc index be81770..aceedf1 100644 --- a/chrome/browser/browser_theme_provider.cc +++ b/chrome/browser/browser_theme_provider.cc @@ -1112,9 +1112,9 @@ void BrowserThemeProvider::SetColorData(DictionaryValue* colors_value) { colors_[WideToUTF8(*iter)] = SkColorSetARGB(static_cast<int>(alpha * 255), r, g, b); } else if (color_list->GetInteger(3, &alpha_int) && - (alpha_int >= 0 && alpha_int <= 255)) { + (alpha_int == 0 || alpha_int == 1)) { colors_[WideToUTF8(*iter)] = - SkColorSetARGB(alpha_int, r, g, b); + SkColorSetARGB(alpha_int ? 255 : 0, r, g, b); } } else { colors_[WideToUTF8(*iter)] = SkColorSetRGB(r, g, b); |