[Extensions Bindings] Move safeCallbackApply to exception handler

extensions/renderer/resources/send_request.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.
 
 var exceptionHandler = require('uncaught_exception_handler');
 var lastError = require('lastError');
 var logging = requireNative('logging');
 var natives = requireNative('sendRequest');
 var validate = require('schemaUtils').validate;
 
+var safeCallbackApply = exceptionHandler.safeCallbackApply;
+
 // All outstanding requests from sendRequest().
 var requests = { __proto__: null };
 
 // Used to prevent double Activity Logging for API calls that use both custom
 // bindings and ExtensionFunctions (via sendRequest).
 var calledSendRequest = false;
 
-// Runs a user-supplied callback safely.
-function safeCallbackApply(name, request, callback, args) {
-  try {
-    $Function.apply(callback, request, args);
-  } catch (e) {
-    exceptionHandler.handle('Error in response to ' + name, e, request.stack);
-  }
-}
-
 // Callback handling.
 function handleResponse(requestId, name, success, responseList, error) {
   // The chrome objects we will set lastError on. Really we should only be
   // setting this on the callback's chrome object, but set on ours too since
   // it's conceivable that something relies on that.
   var callerChrome = chrome;
 
   try {
     var request = requests[requestId];
     logging.DCHECK(request != null);
@@ skipping 99 matching lines @@
   return calledSendRequest;
 }
 
 function clearCalledSendRequest() {
   calledSendRequest = false;
 }
 
 exports.$set('sendRequest', sendRequest);
 exports.$set('getCalledSendRequest', getCalledSendRequest);
 exports.$set('clearCalledSendRequest', clearCalledSendRequest);
-exports.$set('safeCallbackApply', safeCallbackApply);
 
 // Called by C++.
 exports.$set('handleResponse', handleResponse);