[Extensions] Pass EventFilteringInfo directly in DispatchEvent message

extensions/renderer/event_bindings.cc

 // Copyright 2014 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 "extensions/renderer/event_bindings.h"
 
 #include <stdint.h>
 
 #include <map>
 #include <memory>
@@ skipping 110 matching lines @@
   if (counts->second->Remove(*filter)) {
     if (counts->second->is_empty())
       all_counts.erase(counts);  // Clean up if there are no more filters.
     return true;
   }
   return false;
 }
 
 // Returns a v8::Array containing the ids of the listeners that match the given
 // |event_filter_dict| in the given |script_context|.
-v8::Local<v8::Array> GetMatchingListeners(
-    ScriptContext* script_context,
-    const std::string& event_name,
-    const base::DictionaryValue& event_filter_dict) {
+v8::Local<v8::Array> GetMatchingListeners(ScriptContext* script_context,
+                                          const std::string& event_name,
+                                          const EventFilteringInfo& info) {
   const EventFilter& event_filter = g_event_filter.Get();
-  EventFilteringInfo info(event_filter_dict);
   v8::Isolate* isolate = script_context->isolate();
   v8::Local<v8::Context> context = script_context->v8_context();
 
   // Only match events routed to this context's RenderFrame or ones that don't
   // have a routingId in their filter.
   std::set<EventFilter::MatcherID> matched_event_filters =
       event_filter.MatchEvent(event_name, info,
                               script_context->GetRenderFrame()->GetRoutingID());
   v8::Local<v8::Array> array(
       v8::Array::New(isolate, matched_event_filters.size()));
@@ skipping 53 matching lines @@
     }
   }
 
   return false;
 }
 
 // static
 void EventBindings::DispatchEventInContext(
     const std::string& event_name,
     const base::ListValue* event_args,
-    const base::DictionaryValue* filtering_info,
+    const EventFilteringInfo* filtering_info,
     ScriptContext* context) {
   v8::HandleScope handle_scope(context->isolate());
   v8::Context::Scope context_scope(context->v8_context());
 
   v8::Local<v8::Array> listener_ids;
-  if (filtering_info && !filtering_info->empty())
+  if (filtering_info && !filtering_info->is_empty())
     listener_ids = GetMatchingListeners(context, event_name, *filtering_info);
   else
     listener_ids = v8::Array::New(context->isolate());
 
   v8::Local<v8::Value> v8_args[] = {
       gin::StringToSymbol(context->isolate(), event_name),
       content::V8ValueConverter::Create()->ToV8Value(event_args,
                                                      context->v8_context()),
       listener_ids,
   };
@@ skipping 202 matching lines @@
   for (int matcher_id : attached_matcher_ids_safe) {
     DetachFilteredEvent(matcher_id, false /* is_manual */);
   }
   DCHECK(attached_matcher_ids_.empty())
       << "Filtered events cannot be attached during invalidation";
 
   g_unmanaged_listeners.Get().erase(context());
 }
 
 }  // namespace extensions