Various optimizations to reduce the number of temporary allocations.

chrome/browser/themes/theme_properties.cc

 // Copyright (c) 2013 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/themes/theme_properties.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "chrome/browser/themes/browser_theme_pack.h"
@@ skipping 183 matching lines @@
   if (tiling == REPEAT_X)
     return kTilingRepeatX;
   if (tiling == REPEAT_Y)
     return kTilingRepeatY;
   if (tiling == REPEAT)
     return kTilingRepeat;
   return kTilingNoRepeat;
 }
 
 // static
-bool ThemeProperties::IsThemeableImage(int id) {
-  // TODO(pkotwicz): Cache results to improve lookup speed.
-  std::set<int> themeable_idrs;
-  BrowserThemePack::GetThemeableImageIDRs(&themeable_idrs);
-  return themeable_idrs.find(id) != themeable_idrs.end();
-}
-
-// static
 const std::set<int>& ThemeProperties::GetTintableToolbarButtons() {
   CR_DEFINE_STATIC_LOCAL(std::set<int>, button_set, ());
   if (button_set.empty()) {
     button_set = std::set<int>(
         kToolbarButtonIDs,
         kToolbarButtonIDs + arraysize(kToolbarButtonIDs));
   }
 
   return button_set;
 }
@@ skipping 89 matching lines @@
     case NTP_BACKGROUND_ALIGNMENT:
       return kDefaultDisplayPropertyNTPAlignment;
     case NTP_BACKGROUND_TILING:
       return kDefaultDisplayPropertyNTPTiling;
     case NTP_LOGO_ALTERNATE:
       return kDefaultDisplayPropertyNTPAlternateLogo;
   }
 
   return -1;
 }