[Extensions Bindings] Move setHandleRequestWithPromise to util

extensions/renderer/resources/utils.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 nativeDeepCopy = requireNative('utils').deepCopy;
 var DCHECK = requireNative('logging').DCHECK;
+var WARNING = requireNative('logging').WARNING;
+var logActivity = requireNative('activityLogger');
+var exceptionHandler = require('uncaught_exception_handler');
+
+var runCallbackWithLastError;
+if (bindingUtil) {
+  runCallbackWithLastError = function(name, message, stack, callback, args) {
+    bindingUtil.runCallbackWithLastError(message, function() {
+      $Function.apply(callback, null, args);
+    });
+  }
+} else {
+  var lastError = require('lastError');
+  if (lastError)  // lastError can be undefined in unittests.
+    runCallbackWithLastError = lastError.run;
+}
 
 /**
  * An object forEach. Calls |f| with each (key, value) pair of |obj|, using
  * |self| as the target.
  * @param {Object} obj The object to iterate over.
  * @param {function} f The function to call in each iteration.
  * @param {Object} self The object to use as |this| in each function call.
  */
 function forEach(obj, f, self) {
   for (var key in obj) {
@@ skipping 182 matching lines @@
       }
       if (arguments.length <= 1)
         resolve(arguments[0]);
       else
         resolve($Array.slice(arguments));
     });
     $Function.apply(func, null, args);
   });
 }
 
+// DO NOT USE. This causes problems with safe builtins, and makes migration to
+// native bindings more difficult.
+function handleRequestWithPromiseDoNotUse(
+    binding, apiName, methodName, customizedFunction) {
+  var fullName = apiName + '.' + methodName;
+  var extensionId = requireNative('process').GetExtensionId();
+  binding.setHandleRequest(methodName, function() {
+    logActivity.LogAPICall(extensionId, fullName, $Array.slice(arguments));
+    var stack = exceptionHandler.getExtensionStackTrace();
+    var callback = arguments[arguments.length - 1];
+    var args = $Array.slice(arguments, 0, arguments.length - 1);
+    var keepAlivePromise = requireAsync('keep_alive').then(function(module) {
+      return module.createKeepAlive();
+    });
+    $Function.apply(customizedFunction, this, args).then(function(result) {
+      if (callback) {
+        exceptionHandler.safeCallbackApply(
+            fullName, {__proto__: null, stack: stack}, callback, [result]);
+      }
+    }).catch(function(error) {
+      if (callback) {
+        var message = exceptionHandler.safeErrorToString(error, true);
+        runCallbackWithLastError(fullName, message, stack, callback);
+      }
+    }).then(function() {
+      keepAlivePromise.then(function(keepAlive) {
+        keepAlive.close();
+      });
+    });
+  });
+};
+
 exports.$set('forEach', forEach);
 exports.$set('lookup', lookup);
 exports.$set('defineProperty', defineProperty);
 exports.$set('expose', expose);
 exports.$set('deepCopy', deepCopy);
 exports.$set('promise', promise);
+exports.$set('handleRequestWithPromiseDoNotUse',
+             handleRequestWithPromiseDoNotUse);