Let theme values that previous had to be real be specified as ints....

chrome/browser/browser_theme_provider.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browser_theme_provider.h"
 
 #include "base/gfx/png_decoder.h"
 #include "base/string_util.h"
 #include "base/values.h"
 #include "chrome/browser/browser_list.h"
@@ skipping 21 matching lines @@
 
 // Strings used by themes to identify colors for different parts of our UI.
 const char* BrowserThemeProvider::kColorFrame = "frame";
 const char* BrowserThemeProvider::kColorFrameInactive = "frame_inactive";
 const char* BrowserThemeProvider::kColorFrameIncognito = "frame_incognito";
 const char* BrowserThemeProvider::kColorFrameIncognitoInactive =
     "frame_incognito_inactive";
 const char* BrowserThemeProvider::kColorToolbar = "toolbar";
 const char* BrowserThemeProvider::kColorTabText = "tab_text";
 const char* BrowserThemeProvider::kColorBackgroundTabText =
-    "background_tab_text";
+    "tab_background_text";
 const char* BrowserThemeProvider::kColorBookmarkText = "bookmark_text";
 const char* BrowserThemeProvider::kColorNTPBackground = "ntp_background";
 const char* BrowserThemeProvider::kColorNTPText = "ntp_text";
 const char* BrowserThemeProvider::kColorNTPLink = "ntp_link";
 const char* BrowserThemeProvider::kColorNTPSection = "ntp_section";
 const char* BrowserThemeProvider::kColorNTPSectionText = "ntp_section_text";
 const char* BrowserThemeProvider::kColorNTPSectionLink = "ntp_section_link";
 const char* BrowserThemeProvider::kColorControlBackground =
     "control_background";
 const char* BrowserThemeProvider::kColorButtonBackground = "button_background";
@@ skipping 476 matching lines @@
   while (iter != colors_value->end_keys()) {
     ListValue* color_list;
     if (colors_value->GetList(*iter, &color_list) &&
         (color_list->GetSize() == 3 || color_list->GetSize() == 4)) {
       int r, g, b;
       color_list->GetInteger(0, &r);
       color_list->GetInteger(1, &g);
       color_list->GetInteger(2, &b);
       if (color_list->GetSize() == 4) {
         double alpha;
-        color_list->GetReal(3, &alpha);
-        colors_[WideToUTF8(*iter)] = SkColorSetARGB(
-            static_cast<int>(alpha * 255), r, g, b);
+        int alpha_int;
+        if (color_list->GetReal(3, &alpha)) {
+          colors_[WideToUTF8(*iter)] = SkColorSetARGB(
+              static_cast<int>(alpha * 255), r, g, b);
+        } else if (color_list->GetInteger(3, &alpha_int)) {
+          colors_[WideToUTF8(*iter)] = SkColorSetARGB(
+              alpha_int * 255, r, g, b);
+        }
       } else {
         colors_[WideToUTF8(*iter)] = SkColorSetRGB(r, g, b);
       }
     }
     ++iter;
   }
 }
 
 void BrowserThemeProvider::SetTintData(DictionaryValue* tints_value) {
   tints_.clear();
 
   if (!tints_value)
     return;
 
   DictionaryValue::key_iterator iter = tints_value->begin_keys();
   while (iter != tints_value->end_keys()) {
     ListValue* tint_list;
     if (tints_value->GetList(*iter, &tint_list) &&
         tint_list->GetSize() == 3) {
       skia::HSL hsl = { -1, -1, -1 };
-      // TODO(glen): Make this work with integer values.
-      tint_list->GetReal(0, &hsl.h);
-      tint_list->GetReal(1, &hsl.s);
-      tint_list->GetReal(2, &hsl.l);
+      int value = 0;
+      if (!tint_list->GetReal(0, &hsl.h) && tint_list->GetInteger(0, &value))
+        hsl.h = value;
+      if (!tint_list->GetReal(1, &hsl.s) &&  tint_list->GetInteger(1, &value))
+        hsl.s = value;
+      if (!tint_list->GetReal(2, &hsl.l) && tint_list->GetInteger(2, &value))
+        hsl.l = value;
+
       tints_[WideToUTF8(*iter)] = hsl;
     }
     ++iter;
   }
 }
 
 void BrowserThemeProvider::SetDisplayPropertyData(
     DictionaryValue* display_properties_value) {
   display_properties_.clear();
 
@@ skipping 361 matching lines @@
     delete i->second;
   }
   image_cache_.clear();
 }
 
 #if defined(TOOLKIT_VIEWS)
 void BrowserThemeProvider::FreePlatformCaches() {
   // Views (Skia) has no platform image cache to clear.
 }
 #endif