commit c1abb3237596e83cc12639d5b9e7607b67ae77f2
author kalman@chromium.org <kalman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Jul 30 18:28:39 2014
committer kalman@chromium.org <kalman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Jul 30 18:28:39 2014
tree 4696ac8ad2d52243324f89fd2f238669785541e2
parent 056fb8baa877888f0996eb5283721f1dbf2b5fb4

Allow restricting WebUI-enabled extension APIs to URL patterns.

BUG=391944
R=rockot@chromium.org, jschuh@chromium.org

Review URL: https://codereview.chromium.org/422433005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286564 0039d316-1c4b-4281-b951-d872f2087c98

extensions/browser/event_router.h

diff --git a/extensions/browser/event_router.h b/extensions/browser/event_router.h
index a69d0422..c8c5226 100644
--- a/extensions/browser/event_router.h
+++ b/extensions/browser/event_router.h
@@ -13,7 +13,6 @@
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/containers/hash_tables.h"
-#include "base/gtest_prod_util.h"
 #include "base/memory/linked_ptr.h"
 #include "base/memory/ref_counted.h"
 #include "base/values.h"
@@ -93,7 +92,8 @@
               ExtensionPrefs* extension_prefs);
   virtual ~EventRouter();
 
-  // Add or remove the process/extension pair as a listener for |event_name|.
+  // Add or remove an extension as an event listener for |event_name|.
+  //
   // Note that multiple extensions can share a process due to process
   // collapsing. Also, a single extension can have 2 processes if it is a split
   // mode extension.
@@ -104,6 +104,14 @@
                            content::RenderProcessHost* process,
                            const std::string& extension_id);
 
+  // Add or remove a URL as an event listener for |event_name|.
+  void AddEventListenerForURL(const std::string& event_name,
+                              content::RenderProcessHost* process,
+                              const GURL& listener_url);
+  void RemoveEventListenerForURL(const std::string& event_name,
+                                 content::RenderProcessHost* process,
+                                 const GURL& listener_url);
+
   EventListenerMap& listeners() { return listeners_; }
 
   // Registers an observer to be notified when an event listener for
@@ -171,7 +179,7 @@
                   const std::string& extension_id);
 
  private:
-  FRIEND_TEST_ALL_PREFIXES(EventRouterTest, EventRouterObserver);
+  friend class EventRouterTest;
 
   // The extension and process that contains the event listener for a given
   // event.
@@ -221,8 +229,10 @@
                          const linked_ptr<Event>& event,
                          std::set<EventDispatchIdentifier>* already_dispatched);
 
-  // Dispatches the event to the specified extension running in |process|.
+  // Dispatches the event to the specified extension or URL running in
+  // |process|.
   void DispatchEventToProcess(const std::string& extension_id,
+                              const GURL& listener_url,
                               content::RenderProcessHost* process,
                               const linked_ptr<Event>& event);
 
@@ -357,12 +367,14 @@
 struct EventListenerInfo {
   EventListenerInfo(const std::string& event_name,
                     const std::string& extension_id,
+                    const GURL& listener_url,
                     content::BrowserContext* browser_context);
   // The event name including any sub-event, e.g. "runtime.onStartup" or
   // "webRequest.onCompleted/123".
   const std::string event_name;
 
   const std::string extension_id;
+  const GURL listener_url;
   content::BrowserContext* browser_context;
 };