GTK: Refactor some things so GTK doesn't touch the on disk image cache.

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 "app/gfx/skbitmap_operations.h"
 #include "base/file_util.h"
 #include "base/gfx/png_decoder.h"
 #include "base/gfx/png_encoder.h"
@@ skipping 314 matching lines @@
   // generate and store.
   ImageCache::const_iterator found = image_cache_.find(id);
   if (found != image_cache_.end())
     return found->second;
 
   scoped_ptr<SkBitmap> result;
 
   // Try to load the image from the extension.
   result.reset(LoadThemeBitmap(id));
 
-  // If the extension doesn't provide the requested image, but has provided
-  // a custom frame, then we may be able to generate the image required.
-  if (!result.get())
-    result.reset(GenerateTabBackgroundBitmap(id));
-
   // If we still don't have an image, load it from resourcebundle.
   if (!result.get())
     result.reset(new SkBitmap(*rb_.GetBitmapNamed(id)));
 
   if (result.get()) {
     // If the requested image is part of the toolbar button set, and we have
     // a provided tint for that set, tint it appropriately.
     if (button_images_.count(id) && tints_.count(kTintButtons)) {
       SkBitmap* tinted =
           new SkBitmap(TintBitmap(*result.release(), TINT_BUTTONS));
@@ skipping 715 matching lines @@
     // The requested image is a background tab. Get a frame to create the
     // tab against. As themes don't use the glass frame, we don't have to
     // worry about compositing them together, as our default theme provides
     // the necessary bitmaps.
     if (!process_images_) {
       scoped_ptr<SkBitmap> frame;
       frame.reset(LoadThemeBitmap(id));
       if (frame.get())
         themed_image_cache_[id] = new SkBitmap(*frame.get());
     } else {
-      int base_id;
-      std::string resource_name;
-      if (id == IDR_THEME_TAB_BACKGROUND) {
-        base_id = IDR_THEME_FRAME;
-        resource_name = "theme_tab_background";
-      } else {
-        base_id = IDR_THEME_FRAME_INCOGNITO;
-        resource_name = "theme_tab_background_incognito";
-      }
-      std::map<int, SkBitmap*>::iterator it = themed_image_cache_.find(base_id);
-      if (it != themed_image_cache_.end()) {
-        SkBitmap bg_tint = TintBitmap(*(it->second), TINT_BACKGROUND_TAB);
-        int vertical_offset = HasCustomImage(id) ?
-            kRestoredTabVerticalOffset : 0;
-        SkBitmap* bg_tab = new SkBitmap(SkBitmapOperations::CreateTiledBitmap(
-            bg_tint, 0, vertical_offset, bg_tint.width(), bg_tint.height()));
+      SkBitmap* bg_tab = GenerateTabBackgroundBitmapImpl(id);
 
-        // If they've provided a custom image, overlay it.
-        if (HasCustomImage(id)) {
-          SkBitmap* overlay = LoadThemeBitmap(id);
-          if (overlay) {
-            SkCanvas canvas(*bg_tab);
-            for (int x = 0; x < bg_tab->width(); x += overlay->width())
-              canvas.drawBitmap(*overlay, static_cast<SkScalar>(x), 0, NULL);
-          }
+      if (bg_tab) {
+        std::string resource_name;
+        if (id == IDR_THEME_TAB_BACKGROUND) {
+          resource_name = "theme_tab_background";
+        } else {
+          resource_name = "theme_tab_background_incognito";
         }
 
         themed_image_cache_[id] = bg_tab;
         SaveThemeBitmap(resource_name, id);
         return bg_tab;
       }
     }
   }
   return NULL;
 }
 
+SkBitmap* BrowserThemeProvider::GenerateTabBackgroundBitmapImpl(int id) {
+  int base_id;
+  if (id == IDR_THEME_TAB_BACKGROUND) {
+    base_id = IDR_THEME_FRAME;
+  } else {
+    base_id = IDR_THEME_FRAME_INCOGNITO;
+  }
+  std::map<int, SkBitmap*>::iterator it = themed_image_cache_.find(base_id);
+  if (it != themed_image_cache_.end()) {
+    SkBitmap bg_tint = TintBitmap(*(it->second), TINT_BACKGROUND_TAB);
+    int vertical_offset = HasCustomImage(id) ?
+                          kRestoredTabVerticalOffset : 0;
+    SkBitmap* bg_tab = new SkBitmap(SkBitmapOperations::CreateTiledBitmap(
+        bg_tint, 0, vertical_offset, bg_tint.width(), bg_tint.height()));
+
+    // If they've provided a custom image, overlay it.
+    if (HasCustomImage(id)) {
+      SkBitmap* overlay = LoadThemeBitmap(id);
+      if (overlay) {
+        SkCanvas canvas(*bg_tab);
+        for (int x = 0; x < bg_tab->width(); x += overlay->width())
+          canvas.drawBitmap(*overlay, static_cast<SkScalar>(x), 0, NULL);
+      }
+    }
+
+    return bg_tab;
+  }
+
+  return NULL;
+}
+
 void BrowserThemeProvider::SaveCachedImageData() {
   DictionaryValue* pref_images =
       profile_->GetPrefs()->GetMutableDictionary(prefs::kCurrentThemeImages);
 
   for (ImagesDiskCache::iterator it = images_disk_cache_.begin();
       it != images_disk_cache_.end(); it++) {
     std::wstring disk_path = it->first.ToWStringHack();
     std::string pref_name = resource_names_[it->second];
     pref_images->SetString(UTF8ToWide(pref_name),
                            WideToUTF8(disk_path));
@@ skipping 169 matching lines @@
   }
   image_cache_.clear();
   images_disk_cache_.clear();
 }
 
 #if defined(OS_WIN)
 void BrowserThemeProvider::FreePlatformCaches() {
   // Views (Skia) has no platform image cache to clear.
 }
 #endif