Move themes garbage-collection out of shutdown code.

chrome/browser/extensions/extensions_service.cc

Index: chrome/browser/extensions/extensions_service.cc
diff --git a/chrome/browser/extensions/extensions_service.cc b/chrome/browser/extensions/extensions_service.cc
index 05c6768a9d6e3ac6ed15d2559bb06511fa6a36b3..6de568010647bc2aa692deb0761561b55cce1e15 100644
--- a/chrome/browser/extensions/extensions_service.cc
+++ b/chrome/browser/extensions/extensions_service.cc
@@ -21,6 +21,7 @@
 #include "base/values.h"
 #include "base/version.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/themes/browser_theme_provider.h"
 #include "chrome/browser/browser_thread.h"
 #include "chrome/browser/debugger/devtools_manager.h"
 #include "chrome/browser/dom_ui/shown_sections_handler.h"
@@ -1460,6 +1461,11 @@ void ExtensionsService::GarbageCollectExtensions() {
       NewRunnableFunction(
           &extension_file_util::GarbageCollectExtensions, install_directory_,
           extension_paths));
+
+  // Also garbage-collect themes.
+  if (profile_) {

[asargent_no_longer_on_chrome, 2010/11/16 23:13:36]

I think it's safe to assume the profile_ is valid at this point. 

[akalin, 2010/11/17 00:29:39]

True, but i want to be defensive given that this may be called on a timer in the
future.

Added a comment.

+    profile_->GetThemeProvider()->RemoveUnusedThemes();
+  }
 }
 
 void ExtensionsService::OnLoadedInstalledExtensions() {