Fix memory leak caused by Rules Registries

chrome/browser/extensions/extension_system.cc

Index: chrome/browser/extensions/extension_system.cc
diff --git a/chrome/browser/extensions/extension_system.cc b/chrome/browser/extensions/extension_system.cc
index 1240080341c2c477e6d526883dbed0b32d3e359e..557e9bb8bb2dc275b7f87754cc43cf71ac18c6bf 100644
--- a/chrome/browser/extensions/extension_system.cc
+++ b/chrome/browser/extensions/extension_system.cc
@@ -63,6 +63,7 @@ ExtensionSystemImpl::Shared::Shared(Profile* profile)
 }
 
 ExtensionSystemImpl::Shared::~Shared() {
+  rules_registry_service_->Shutdown();

[Matt Perry, 2012/04/12 21:46:19]

When is this called? If it's during shutdown, then the IO thread might already
be gone, or in the process of shutting down, so that the posted tasks don't have
a chance to run.

[battre, 2012/04/12 22:20:52]

ExtensionSystemImpl::Shared is ref counted so it is deleted when the last
ExtensionSystemImpl is destroyed. This is triggered by
ProfileImpl::~ProfileImpl(). The Profile is owned by the ProfileManager which is
destroyed in BrowserProcessImpl before the IO thread is killed.

 }
 
 void ExtensionSystemImpl::Shared::InitPrefs() {