diff options
| author | tony@chromium.org <tony@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-02 01:41:06 +0000 |
|---|---|---|
| committer | tony@chromium.org <tony@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-02 01:41:06 +0000 |
| commit | 9829d8d83608c54937e5187512f98a81b29f80bb (patch) | |
| tree | 29db8d5ea5f6e7ad1ec67a84f8b251d513c97dc7 /chrome/browser/browser_theme_provider_unittest.cc | |
| parent | 6d7342d33a13358f7acfb78cee3c7a3ebd67c226 (diff) | |
| download | chromium_src-9829d8d83608c54937e5187512f98a81b29f80bb.zip chromium_src-9829d8d83608c54937e5187512f98a81b29f80bb.tar.gz chromium_src-9829d8d83608c54937e5187512f98a81b29f80bb.tar.bz2 | |
Take 2 at: Fix the opacity of titlebars on the NTP page.
We were accidentally setting the opacity to 1/255.
This bug was introduced in r32643. Fix the unittest too.
BUG=29107
Review URL: http://codereview.chromium.org/461003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33529 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_theme_provider_unittest.cc')
| -rw-r--r-- | chrome/browser/browser_theme_provider_unittest.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/chrome/browser/browser_theme_provider_unittest.cc b/chrome/browser/browser_theme_provider_unittest.cc index 503c9e4..6556a56 100644 --- a/chrome/browser/browser_theme_provider_unittest.cc +++ b/chrome/browser/browser_theme_provider_unittest.cc @@ -141,12 +141,12 @@ TEST_F(BrowserThemeProviderTest, ColorSanityCheck) { TEST_F(BrowserThemeProviderTest, DeriveUnderlineLinkColor) { // If we specify a link color, but don't specify the underline color, the // theme provider should create one. - std::string color_json = "{ \"ntp_link\": [128, 128, 128, 126]," - " \"ntp_section_link\": [128, 128, 128, 126] }"; + std::string color_json = "{ \"ntp_link\": [128, 128, 128, 1]," + " \"ntp_section_link\": [128, 128, 128, 1] }"; LoadColorJSON(color_json); std::map<int, SkColor> colors = GetDefaultColorMap(); - SkColor link_color = SkColorSetARGB(126, 128, 128, 128); + SkColor link_color = SkColorSetARGB(255, 128, 128, 128); colors[BrowserThemeProvider::COLOR_NTP_LINK] = link_color; colors[BrowserThemeProvider::COLOR_NTP_LINK_UNDERLINE] = BuildThirdOpacity(link_color); |