Mark create/update/clear callbacks of notification API as optional

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;
 
@@ skipping 60 matching lines @@
         validate(responseList, request.callbackSchema.parameters);
       }
       safeCallbackApply(name, request, request.callback, responseList);
     }
 
     if (error &&
         !lastError.hasAccessed(chrome) &&
         !lastError.hasAccessed(callerChrome)) {
       // The native call caused an error, but the developer didn't check
       // runtime.lastError.
-      // Notify the developer of the error via the (error) console.
-      console.error("Unchecked runtime.lastError while running " +
-          (name || "unknown") + ": " + error +
-          (request.stack ? "\n" + request.stack : ""));
+      lastError.reportUncheckedLastError(name, error, request.stack);
     }
   } finally {
     delete requests[requestId];
     lastError.clear(chrome);
     if (callerChrome !== chrome)
       lastError.clear(callerChrome);
   }
 }
 
 function prepareRequest(args, argSchemas) {
@@ skipping 18 matching lines @@
 
 // Send an API request and optionally register a callback.
 // |optArgs| is an object with optional parameters as follows:
 // - customCallback: a callback that should be called instead of the standard
 //   callback.
 // - nativeFunction: the v8 native function to handle the request, or
 //   StartRequest if missing.
 // - forIOThread: true if this function should be handled on the browser IO
 //   thread.
 // - preserveNullInObjects: true if it is safe for null to be in objects.
+// - stack: An optional string that contains the stack trace, to be displayed
+//   to the user if an error occurs.
 function sendRequest(functionName, args, argSchemas, optArgs) {
   calledSendRequest = true;
   if (!optArgs)
     optArgs = {};
   var request = prepareRequest(args, argSchemas);
-  request.stack = exceptionHandler.getExtensionStackTrace();
+  request.stack = optArgs.stack || exceptionHandler.getExtensionStackTrace();
   if (optArgs.customCallback) {
     request.customCallback = optArgs.customCallback;
   }
 
   var nativeFunction = optArgs.nativeFunction || natives.StartRequest;
 
   var requestId = natives.GetNextRequestId();
   request.id = requestId;
   requests[requestId] = request;
 
@@ skipping 14 matching lines @@
   calledSendRequest = false;
 }
 
 exports.sendRequest = sendRequest;
 exports.getCalledSendRequest = getCalledSendRequest;
 exports.clearCalledSendRequest = clearCalledSendRequest;
 exports.safeCallbackApply = safeCallbackApply;
 
 // Called by C++.
 exports.handleResponse = handleResponse;