[Extensions Bindings] Don't allow `event` module with native bindings

extensions/renderer/module_system.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/module_system.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/macros.h"
@@ skipping 827 matching lines @@
   };
   {
     v8::TryCatch try_catch(GetIsolate());
     try_catch.SetCaptureMessage(true);
     context_->SafeCallFunction(func, arraysize(args), args);
     if (try_catch.HasCaught()) {
       HandleException(try_catch);
       return v8::Undefined(GetIsolate());
     }
   }
+

[jbroman, 2017/06/12 15:09:48]

super-nit: accidental change?

[Devlin, 2017/06/12 20:30:14]

Yep!  Removed.

   return handle_scope.Escape(exports);
 }
 
 void ModuleSystem::OnDidAddPendingModule(
     const std::string& id,
     const std::vector<std::string>& dependencies) {
   bool module_system_managed = source_map_->Contains(id);
 
   gin::ModuleRegistry* registry =
       gin::ModuleRegistry::From(context_->v8_context());
@@ skipping 60 matching lines @@
       !value->IsFunction()) {
     Fatal(context_, module_name + "." + method_name + " is not a function");
     return function;
   }
 
   function = v8::Local<v8::Function>::Cast(value);
   return function;
 }
 
 }  // namespace extensions