Plumb invalidations from Tango to the extensions code for the Push Messaging API.

chrome/browser/extensions/extension_service.cc

Index: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index 6425c251fc1699da0aa5feab1ac590a19d973280..f09b05e5ddd7f3b6b22cd692e6884a62bb25f29a 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -523,6 +523,10 @@ void ExtensionService::InitEventRouters() {
   event_routers_initialized_ = true;
 }
 
+void ExtensionService::Shutdown() {
+  push_messaging_event_router_->Shutdown();

[Munjal (Google), 2012/08/14 18:56:37]

Can't we clean up the event router in the destructor instead of adding an
explicit method call?

[dcheng, 2012/08/14 21:39:59]

We cannot. We have dependencies such that PSS depends on ExtensionSystem. That
means that PSS is destroyed before ExtensionSystem, so we need to drop our
references to PSS in Shutdown() before it is destroyed.

+}
+
 const Extension* ExtensionService::GetExtensionById(
     const std::string& id, bool include_disabled) const {
   return GetExtensionByIdInternal(id, true, include_disabled, false);