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

extensions/renderer/resources/app_runtime_custom_bindings.js

 // 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.
 
 // Custom binding for the chrome.app.runtime API.
 
 var binding = apiBridge || require('binding').Binding.create('app.runtime');
 
 var AppViewGuestInternal =
     getInternalApi ?
         getInternalApi('appViewGuestInternal') :
         require('binding').Binding.create('appViewGuestInternal').generate();
-var eventBindings = require('event_bindings');
+var registerArgumentMassager = bindingUtil ?
+    bindingUtil.registerEventArgumentMassager.bind(bindingUtil) :

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

Do we need to use $Function.bind here?

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

Done.

+    require('event_bindings').registerArgumentMassager;
 var fileSystemHelpers = requireNative('file_system_natives');
 var GetIsolatedFileSystem = fileSystemHelpers.GetIsolatedFileSystem;
 var entryIdManager = require('entryIdManager');
 
-eventBindings.registerArgumentMassager('app.runtime.onEmbedRequested',
-    function(args, dispatch) {
+registerArgumentMassager('app.runtime.onEmbedRequested',
+                         function(args, dispatch) {
   var appEmbeddingRequest = args[0];
   var id = appEmbeddingRequest.guestInstanceId;
   delete appEmbeddingRequest.guestInstanceId;
   appEmbeddingRequest.allow = function(url) {
     AppViewGuestInternal.attachFrame(url, id);
   };
 
   appEmbeddingRequest.deny = function() {
     AppViewGuestInternal.denyRequest(id);
   };
 
   dispatch([appEmbeddingRequest]);
 });
 
-eventBindings.registerArgumentMassager('app.runtime.onLaunched',
-    function(args, dispatch) {
+registerArgumentMassager('app.runtime.onLaunched', function(args, dispatch) {
   var launchData = args[0];
   if (launchData.items) {
     // An onLaunched corresponding to file_handlers in the app's manifest.
     var items = [];
     var numItems = launchData.items.length;
     var itemLoaded = function(err, item) {
       if (err) {
         console.error('Error getting fileEntry, code: ' + err.code);
       } else {
         $Array.push(items, item);
@@ skipping 32 matching lines @@
     });
   } else {
     // Default case. This currently covers an onLaunched corresponding to
     // url_handlers in the app's manifest.
     dispatch([launchData]);
   }
 });
 
 if (!apiBridge)
   exports.$set('binding', binding.generate());