Revert "Unrevert 119770 - "Event pages: remember events that the page registered for,"

chrome/browser/extensions/extension_process_manager.cc

 // Copyright (c) 2012 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 "base/bind.h"
 #include "base/command_line.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 
 #include "chrome/browser/extensions/extension_event_router.h"
 #include "chrome/browser/ui/browser_window.h"
@@ skipping 17 matching lines @@
 #include "content/public/browser/render_view_host_delegate.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/renderer_preferences.h"
 
 using content::BrowserThread;
 using content::OpenURLParams;
 using content::Referrer;
 using content::SiteInstance;
 
-namespace events {
-const char kOnInstalled[] = "experimental.extension.onInstalled";
-};  // namespace events
-
 namespace {
 
 // Incognito profiles use this process manager. It is mostly a shim that decides
 // whether to fall back on the original profile's ExtensionProcessManager based
 // on whether a given extension uses "split" or "spanning" incognito behavior.
 class IncognitoExtensionProcessManager : public ExtensionProcessManager {
  public:
   explicit IncognitoExtensionProcessManager(Profile* profile);
   virtual ~IncognitoExtensionProcessManager() {}
   virtual ExtensionHost* CreateViewHost(
@@ skipping 12 matching lines @@
                        const content::NotificationDetails& details);
 
   // Returns true if the extension is allowed to run in incognito mode.
   bool IsIncognitoEnabled(const Extension* extension);
 
   ExtensionProcessManager* original_manager_;
 };
 
 static void CreateBackgroundHostForExtensionLoad(
     ExtensionProcessManager* manager, const Extension* extension) {
-  if (extension->has_background_page()) {
-    if (extension->background_page_persists()) {
-      manager->CreateBackgroundHost(extension, extension->GetBackgroundURL());
-    } else {
-      // TODO(mpcomplete): Only call this on install once we persist event
-      // registration. Also call this for regular background pages.
-      manager->DispatchExtensionInstalledEvent(extension);
-    }
+  if (extension->has_background_page() &&
+      extension->background_page_persists()) {
+    manager->CreateBackgroundHost(extension, extension->GetBackgroundURL());
   }
 }
 
 static void CreateBackgroundHostsForProfileStartup(
     ExtensionProcessManager* manager, const ExtensionSet* extensions) {
   for (ExtensionSet::const_iterator extension = extensions->begin();
        extension != extensions->end(); ++extension) {
     CreateBackgroundHostForExtensionLoad(manager, *extension);
   }
 }
@@ skipping 319 matching lines @@
 }
 
 void ExtensionProcessManager::CloseBackgroundHosts() {
   for (ExtensionHostSet::iterator iter = background_hosts_.begin();
        iter != background_hosts_.end(); ) {
     ExtensionHostSet::iterator current = iter++;
     delete *current;
   }
 }
 
-void ExtensionProcessManager::DispatchExtensionInstalledEvent(
-    const Extension* extension) {
-  ExtensionEventRouter* router = GetProfile()->GetExtensionEventRouter();
-  router->AddLazyEventListener(events::kOnInstalled, extension->id());
-  router->DispatchEventToExtension(
-      extension->id(), events::kOnInstalled, "[]", NULL, GURL());
-}
-
 //
 // IncognitoExtensionProcessManager
 //
 
 IncognitoExtensionProcessManager::IncognitoExtensionProcessManager(
     Profile* profile)
     : ExtensionProcessManager(profile),
       original_manager_(profile->GetOriginalProfile()->
                             GetExtensionProcessManager()) {
   DCHECK(profile->IsOffTheRecord());
@@ skipping 71 matching lines @@
         if (service && service->is_ready())
           CreateBackgroundHostsForProfileStartup(this, service->extensions());
       }
       break;
     }
     default:
       ExtensionProcessManager::Observe(type, source, details);
       break;
   }
 }