<webview>: Don't attempt to dispatch events to all app windows.

This CL associates a routingId with each event listener. This ensures that when dispatching events from the browser process, only event listeners associated with the current RenderView are fired.

BUG=166165
Test=manually by opening a two Chrome app windows in the same app with webviews
Verify that there is no error spew in either console output.

Review URL: https://chromiumcodereview.appspot.com/18858005

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

1 files changed, 9 insertions, 2 deletions

diff --git a/extensions/common/event_matcher.cc b/extensions/common/event_matcher.cc
index 937c783..ad76c89 100644
--- a/extensions/common/event_matcher.cc
+++ b/extensions/common/event_matcher.cc
@@ -12,8 +12,10 @@ const char kUrlFiltersKey[] = "url";
 
 namespace extensions {
 
-EventMatcher::EventMatcher(scoped_ptr<base::DictionaryValue> filter)
-    : filter_(filter.Pass()) {
+EventMatcher::EventMatcher(scoped_ptr<base::DictionaryValue> filter,
+                           int routing_id)
+    : filter_(filter.Pass()),
+      routing_id_(routing_id) {
 }
 
 EventMatcher::~EventMatcher() {
@@ -23,6 +25,7 @@ bool EventMatcher::MatchNonURLCriteria(
     const EventFilteringInfo& event_info) const {
   if (!event_info.has_instance_id())
     return true;
+
   return event_info.instance_id() == GetInstanceID();
 }
 
@@ -51,4 +54,8 @@ int EventMatcher::GetInstanceID() const {
   return instance_id;
 }
 
+int EventMatcher::GetRoutingID() const {
+  return routing_id_;
+}
+
 }  // namespace extensions