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

chrome/browser/extensions/extension_event_router.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 "chrome/browser/extensions/extension_event_router.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/extension_devtools_manager.h"
 #include "chrome/browser/extensions/extension_host.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 #include "chrome/browser/extensions/extension_processes_api.h"
 #include "chrome/browser/extensions/extension_processes_api_constants.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_tabs_module.h"
 #include "chrome/browser/extensions/api/webrequest/webrequest_api.h"
 #include "chrome/browser/extensions/process_map.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/chrome_view_type.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_messages.h"
 #include "chrome/common/extensions/api/extension_api.h"
+#include "content/browser/child_process_security_policy.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/render_process_host.h"
 
 using content::BrowserThread;
 using extensions::ExtensionAPI;
 
 namespace {
 
 const char kDispatchEvent[] = "Event.dispatchJSON";
 
@@ skipping 18 matching lines @@
   bool operator<(const EventListener& that) const {
     if (process < that.process)
       return true;
     if (process == that.process && extension_id < that.extension_id)
       return true;
     return false;
   }
 };
 
 struct ExtensionEventRouter::ExtensionEvent {
+  std::string extension_id;
   std::string event_name;
   std::string event_args;
   GURL event_url;
   Profile* restrict_to_profile;
   std::string cross_incognito_args;
 
-  ExtensionEvent(const std::string& event_name,
+  ExtensionEvent(const std::string& extension_id,
+                 const std::string& event_name,
                  const std::string& event_args,
                  const GURL& event_url,
                  Profile* restrict_to_profile,
                  const std::string& cross_incognito_args)
-    : event_name(event_name),
+    : extension_id(extension_id),
+      event_name(event_name),
       event_args(event_args),
       event_url(event_url),
       restrict_to_profile(restrict_to_profile),
       cross_incognito_args(cross_incognito_args) {}
 };
 
 // static
 void ExtensionEventRouter::DispatchEvent(IPC::Message::Sender* ipc_sender,
                                          const std::string& extension_id,
                                          const std::string& event_name,
@@ skipping 59 matching lines @@
   if (event_name.compare(extension_processes_api_constants::kOnUpdated) == 0)
     ExtensionProcessesEventRouter::GetInstance()->ListenerRemoved();
 
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(
           &NotifyEventListenerRemovedOnIOThread,
           profile_, listener.extension_id, event_name));
 }
 
-void ExtensionEventRouter::AddLazyEventListener(
-    const std::string& event_name,
-    const std::string& extension_id) {
-  EventListener lazy_listener(NULL, extension_id);
-  if (lazy_listeners_[event_name].count(lazy_listener) == 0)
-    lazy_listeners_[event_name].insert(lazy_listener);
-}
-
-void ExtensionEventRouter::RemoveLazyEventListener(
-    const std::string& event_name,
-    const std::string& extension_id) {
-  EventListener lazy_listener(NULL, extension_id);
-  lazy_listeners_[event_name].erase(lazy_listener);
-}
-
 bool ExtensionEventRouter::HasEventListener(const std::string& event_name) {
   return (listeners_.find(event_name) != listeners_.end() &&
           !listeners_[event_name].empty());
 }
 
 bool ExtensionEventRouter::ExtensionHasEventListener(
     const std::string& extension_id, const std::string& event_name) {
   ListenerMap::iterator it = listeners_.find(event_name);
   if (it == listeners_.end())
     return false;
 
   std::set<EventListener>& listeners = it->second;
   for (std::set<EventListener>::iterator listener = listeners.begin();
        listener != listeners.end(); ++listener) {
     if (listener->extension_id == extension_id)
       return true;
   }
   return false;
 }
 
 void ExtensionEventRouter::DispatchEventToRenderers(
     const std::string& event_name,
     const std::string& event_args,
     Profile* restrict_to_profile,
     const GURL& event_url) {
   linked_ptr<ExtensionEvent> event(
-      new ExtensionEvent(event_name, event_args, event_url,
+      new ExtensionEvent("", event_name, event_args, event_url,
                          restrict_to_profile, ""));
-  DispatchEventImpl("", event, false);
+  DispatchEventImpl(event, false);
 }
 
 void ExtensionEventRouter::DispatchEventToExtension(
     const std::string& extension_id,
     const std::string& event_name,
     const std::string& event_args,
     Profile* restrict_to_profile,
     const GURL& event_url) {
   DCHECK(!extension_id.empty());
   linked_ptr<ExtensionEvent> event(
-      new ExtensionEvent(event_name, event_args, event_url,
+      new ExtensionEvent(extension_id, event_name, event_args, event_url,
                          restrict_to_profile, ""));
-  DispatchEventImpl(extension_id, event, false);
+  DispatchEventImpl(event, false);
 }
 
 void ExtensionEventRouter::DispatchEventsToRenderersAcrossIncognito(
     const std::string& event_name,
     const std::string& event_args,
     Profile* restrict_to_profile,
     const std::string& cross_incognito_args,
     const GURL& event_url) {
   linked_ptr<ExtensionEvent> event(
-      new ExtensionEvent(event_name, event_args, event_url,
+      new ExtensionEvent("", event_name, event_args, event_url,
                          restrict_to_profile, cross_incognito_args));
-  DispatchEventImpl("", event, false);
+  DispatchEventImpl(event, false);
 }
 
-bool ExtensionEventRouter::CanDispatchEventNow(const Extension* extension) {
-  DCHECK(extension);
-  if (extension->has_background_page() &&
+bool ExtensionEventRouter::CanDispatchEventNow(
+    const std::string& extension_id) {
+  if (extension_id.empty())
+    // TODO(mpcomplete): We need to test this per-extension, rather than
+    // globally.
+    return true;
+
+  const Extension* extension = profile_->GetExtensionService()->
+      GetExtensionById(extension_id, false);
+  if (extension && extension->has_background_page() &&
       !extension->background_page_persists()) {
     ExtensionProcessManager* pm = profile_->GetExtensionProcessManager();
-
-    // TODO(mpcomplete): this is incorrect. We need to check whether the page
-    // has finished loading. If not, we can't dispatch the event (because the
-    // listener hasn't been set up yet).
-    if (!pm->GetBackgroundHostForExtension(extension->id())) {
+    if (!pm->GetBackgroundHostForExtension(extension_id)) {
       pm->CreateBackgroundHost(extension, extension->GetBackgroundURL());
       return false;
     }
   }
 
   return true;
 }
 
 void ExtensionEventRouter::DispatchEventImpl(
-    const std::string& extension_id,
-    const linked_ptr<ExtensionEvent>& event,
-    bool was_pending) {
-  // Ensure we are only dispatching pending events to a particular extension.
-  if (was_pending)
-    CHECK(!extension_id.empty());
-
+    const linked_ptr<ExtensionEvent>& event, bool was_pending) {
   if (!profile_)
     return;
 
   // We don't expect to get events from a completely different profile.
   DCHECK(!event->restrict_to_profile ||
          profile_->IsSameProfile(event->restrict_to_profile));
 
-  if (!was_pending)
-    LoadLazyBackgroundPagesForEvent(extension_id, event);
+  if (!CanDispatchEventNow(event->extension_id)) {
+    // Events should not be made pending twice. This may happen if the
+    // background page is shutdown before we finish dispatching pending events.
+    CHECK(!was_pending);
+    // TODO(tessamac): make sure Background Page notification doesn't
+    //                 happen before the event is added to the pending list.
+    AppendEvent(event);
+    return;
+  }
 
   ListenerMap::iterator it = listeners_.find(event->event_name);
   if (it == listeners_.end())
     return;
 
   std::set<EventListener>& listeners = it->second;
   ExtensionService* service = profile_->GetExtensionService();
 
   // Send the event only to renderers that are listening for it.
   for (std::set<EventListener>::iterator listener = listeners.begin();
        listener != listeners.end(); ++listener) {
-    if (!extension_id.empty() && extension_id != listener->extension_id)
+    if (!event->extension_id.empty() &&
+        event->extension_id != listener->extension_id)
       continue;
 
-    const Extension* extension = service->extensions()->GetByID(
-        listener->extension_id);
+    const Extension* extension = service->GetExtensionById(
+        listener->extension_id, false);
 
     // The extension could have been removed, but we do not unregister it until
     // the extension process is unloaded.
     if (!extension)
       continue;
 
     Profile* listener_profile = Profile::FromBrowserContext(
         listener->process->GetBrowserContext());
     extensions::ProcessMap* process_map =
         listener_profile->GetExtensionService()->process_map();
@@ skipping 19 matching lines @@
       }
       continue;
     }
 
     DispatchEvent(listener->process, listener->extension_id,
                   event->event_name, event->event_args, event->event_url);
     IncrementInFlightEvents(extension);
   }
 }
 
-void ExtensionEventRouter::LoadLazyBackgroundPagesForEvent(
-    const std::string& extension_id,
-    const linked_ptr<ExtensionEvent>& event) {
-  ExtensionService* service = profile_->GetExtensionService();
-
-  ListenerMap::iterator it = lazy_listeners_.find(event->event_name);
-  if (it == lazy_listeners_.end())
-    return;
-
-  std::set<EventListener>& listeners = it->second;
-  for (std::set<EventListener>::iterator listener = listeners.begin();
-       listener != listeners.end(); ++listener) {
-    if (!extension_id.empty() && extension_id != listener->extension_id)
-      continue;
-
-    const Extension* extension = service->extensions()->GetByID(
-        listener->extension_id);
-
-    if (extension && !CanDispatchEventNow(extension)) {
-      // TODO(mpcomplete): make sure Background Page notification doesn't
-      //                   happen before the event is added to the pending list.
-      AppendEvent(extension->id(), event);
-    }
-  }
-}
-
 void ExtensionEventRouter::IncrementInFlightEvents(const Extension* extension) {
   if (!extension->background_page_persists())
     in_flight_events_[extension->id()]++;
 }
 
 void ExtensionEventRouter::OnExtensionEventAck(
     const std::string& extension_id) {
   CHECK(in_flight_events_[extension_id] > 0);
   in_flight_events_[extension_id]--;
 }
 
 bool ExtensionEventRouter::HasInFlightEvents(const std::string& extension_id) {
   return in_flight_events_[extension_id] > 0;
 }
 
 void ExtensionEventRouter::AppendEvent(
-    const std::string& extension_id,
     const linked_ptr<ExtensionEvent>& event) {
   PendingEventsList* events_list = NULL;
-  PendingEventsPerExtMap::iterator it = pending_events_.find(extension_id);
+  PendingEventsPerExtMap::iterator it =
+      pending_events_.find(event->extension_id);
   if (it == pending_events_.end()) {
     events_list = new PendingEventsList();
-    pending_events_[extension_id] = linked_ptr<PendingEventsList>(events_list);
+    pending_events_[event->extension_id] =
+        linked_ptr<PendingEventsList>(events_list);
   } else {
     events_list = it->second.get();
   }
 
   events_list->push_back(event);
 }
 
 void ExtensionEventRouter::DispatchPendingEvents(
-    const std::string& extension_id) {
-  CHECK(!extension_id.empty());
+    const std::string &extension_id) {
+  // Find the list of pending events for this extension.
   PendingEventsPerExtMap::const_iterator map_it =
       pending_events_.find(extension_id);
   if (map_it == pending_events_.end())
     return;
 
   PendingEventsList* events_list = map_it->second.get();
   for (PendingEventsList::const_iterator it = events_list->begin();
        it != events_list->end(); ++it)
-    DispatchEventImpl(extension_id, *it, true);
+    DispatchEventImpl(*it, true);
 
+  // Delete list.
   events_list->clear();
   pending_events_.erase(extension_id);
-
-  // Check if the extension is idle, which may be the case if no events were
-  // successfully dispatched.
-  profile_->GetExtensionProcessManager()->OnExtensionIdle(extension_id);
 }
 
 void ExtensionEventRouter::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   switch (type) {
     case content::NOTIFICATION_RENDERER_PROCESS_TERMINATED:
     case content::NOTIFICATION_RENDERER_PROCESS_CLOSED: {
       content::RenderProcessHost* renderer =
           content::Source<content::RenderProcessHost>(source).ptr();
       // Remove all event listeners associated with this renderer
       for (ListenerMap::iterator it = listeners_.begin();
            it != listeners_.end(); ) {
         ListenerMap::iterator current_it = it++;
         for (std::set<EventListener>::iterator jt = current_it->second.begin();
              jt != current_it->second.end(); ) {
           std::set<EventListener>::iterator current_jt = jt++;
           if (current_jt->process == renderer) {
             RemoveEventListener(current_it->first,
                                 current_jt->process,
                                 current_jt->extension_id);
           }
         }
       }
       break;
     }
     case chrome::NOTIFICATION_EXTENSION_HOST_DID_STOP_LOADING: {
-      // If an on-demand background page finished loading, dispatch queued up
-      // events for it.
+      // TODO: dispatch events in queue.  ExtensionHost is in the details.
       ExtensionHost* eh = content::Details<ExtensionHost>(details).ptr();
-      if (eh->extension_host_type() ==
-              chrome::VIEW_TYPE_EXTENSION_BACKGROUND_PAGE &&
-          !eh->extension()->background_page_persists()) {
-        DispatchPendingEvents(eh->extension_id());
-      }
+      DispatchPendingEvents(eh->extension_id());
       break;
     }
     // TODO(tessamac): if background page crashed/failed clear queue.
     default:
       NOTREACHED();
       return;
   }
 }