[Extensions] Pass EventFilteringInfo directly in DispatchEvent message

extensions/renderer/native_extension_bindings_system.cc

Index: extensions/renderer/native_extension_bindings_system.cc
diff --git a/extensions/renderer/native_extension_bindings_system.cc b/extensions/renderer/native_extension_bindings_system.cc
index 4c9ab439b96a78f2a11c6cf09cf015fdeec35c97..9431880163016ee5292fd263939d6d19712bab33 100644
--- a/extensions/renderer/native_extension_bindings_system.cc
+++ b/extensions/renderer/native_extension_bindings_system.cc
@@ -481,15 +481,13 @@ void NativeExtensionBindingsSystem::UpdateBindingsForContext(
 void NativeExtensionBindingsSystem::DispatchEventInContext(
     const std::string& event_name,
     const base::ListValue* event_args,
-    const base::DictionaryValue* filtering_info_dict,
+    const EventFilteringInfo* filtering_info,
     ScriptContext* context) {
   v8::HandleScope handle_scope(context->isolate());
   v8::Context::Scope context_scope(context->v8_context());
-  EventFilteringInfo filter;
-  if (filtering_info_dict)
-    filter = EventFilteringInfo(*filtering_info_dict);
-  api_system_.FireEventInContext(event_name, context->v8_context(), *event_args,
-                                 filter);
+  api_system_.FireEventInContext(
+      event_name, context->v8_context(), *event_args,
+      filtering_info ? *filtering_info : EventFilteringInfo());

[lazyboy, 2017/06/15 17:12:23]

Is there a reason FireEventInContext doesn't take EventFilteringInfo* (instead
of const&)? We could avoid creating empty EventFilteringInfo() in that case, and
isn't that the common scenario?

[Devlin, 2017/06/15 18:01:06]

Nope, it probably should.  I noticed that too, and was going to make a change
separately (it's goes a bit far down).  Does that SGTY?  Or would you prefer it
in this CL?

[lazyboy, 2017/06/15 18:06:43]

Separate CL sounds good! Thanks.

 }
 
 bool NativeExtensionBindingsSystem::HasEventListenerInContext(