Delete unused themes from disk on shutdown.

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/codec/png_codec.h"
 #include "app/gfx/skbitmap_operations.h"
 #include "base/file_util.h"
 #include "base/stl_util-inl.h"
 #include "base/string_util.h"
 #include "base/thread.h"
 #include "base/values.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browser_window.h"
+#include "chrome/browser/extensions/extensions_service.h"
 #include "chrome/browser/metrics/user_metrics.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/theme_resources_util.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/common/notification_type.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/pref_service.h"
 #include "grit/app_resources.h"
@@ skipping 265 matching lines @@
     resource_names_[IDR_THEME_NTP_BACKGROUND] = "theme_ntp_background";
     resource_names_[IDR_THEME_BUTTON_BACKGROUND] = "theme_button_background";
     resource_names_[IDR_THEME_NTP_ATTRIBUTION] = "theme_ntp_attribution";
     resource_names_[IDR_THEME_WINDOW_CONTROL_BACKGROUND] =
         "theme_window_control_background";
   }
 }
 
 BrowserThemeProvider::~BrowserThemeProvider() {
   ClearCaches();
+
+  RemoveUnusedThemes();
 }
 
 void BrowserThemeProvider::Init(Profile* profile) {
   DCHECK(CalledOnValidThread());
   profile_ = profile;
 
   image_dir_ = profile_->GetPath().Append(chrome::kThemeImagesDirname);
   if (!file_util::PathExists(image_dir_))
     file_util::CreateDirectory(image_dir_);
 
@@ skipping 190 matching lines @@
     AutoLock lock(themed_image_cache_lock_);
     GenerateFrameImages();
     GenerateTabImages();
   }
 
   WriteImagesToDisk();
   NotifyThemeChanged();
   UserMetrics::RecordAction(L"Themes_Installed", profile_);
 }
 
+void BrowserThemeProvider::RemoveUnusedThemes() {
+  if (!profile_)
+    return;
+  ExtensionsService* service = profile_->GetExtensionsService();
+  if (!service)
+    return;
+  std::string current_theme = GetThemeID();
+  std::vector<std::string> remove_list;
+  const ExtensionList* extensions = service->extensions();
+  for (ExtensionList::const_iterator it = extensions->begin();
+       it != extensions->end(); ++it) {
+    if ((*it)->IsTheme() && (*it)->id() != current_theme) {
+      remove_list.push_back((*it)->id());
+    }
+  }
+  for (size_t i = 0; i < remove_list.size(); ++i)
+    service->UninstallExtension(remove_list[i], false);
+}
+
 void BrowserThemeProvider::UseDefaultTheme() {
   ClearAllThemeData();
   NotifyThemeChanged();
   UserMetrics::RecordAction(L"Themes_Reset", profile_);
 }
 
 std::string BrowserThemeProvider::GetThemeID() const {
   std::wstring id = profile_->GetPrefs()->GetString(prefs::kCurrentThemeID);
   return WideToUTF8(id);
 }
@@ skipping 752 matching lines @@
 void BrowserThemeProvider::SaveCachedImageData() const {
   DictionaryValue* pref_images =
       profile_->GetPrefs()->GetMutableDictionary(prefs::kCurrentThemeImages);
 
   for (ImagesDiskCache::const_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_.find(it->second)->second;
     pref_images->SetString(UTF8ToWide(pref_name), WideToUTF8(disk_path));
   }
-  profile_->GetPrefs()->SavePersistentPrefs();
+  profile_->GetPrefs()->ScheduleSavePersistentPrefs();
 }
 
 void BrowserThemeProvider::SaveThemeID(const std::string& id) {
   profile_->GetPrefs()->SetString(prefs::kCurrentThemeID, UTF8ToWide(id));
 }
 
 void BrowserThemeProvider::ClearCaches() {
   FreePlatformCaches();
   STLDeleteValues(&image_cache_);
 
@@ skipping 13 matching lines @@
 }
 
 bool BrowserThemeProvider::ShouldTintFrames() const {
   return (HasCustomImage(IDR_THEME_FRAME) ||
       tints_.count(GetTintKey(TINT_BACKGROUND_TAB)) ||
       tints_.count(GetTintKey(TINT_FRAME)) ||
       tints_.count(GetTintKey(TINT_FRAME_INACTIVE)) ||
       tints_.count(GetTintKey(TINT_FRAME_INCOGNITO)) ||
       tints_.count(GetTintKey(TINT_FRAME_INCOGNITO_INACTIVE)));
 }