Use utils.forEach everywhere rather than Array.prototype.forEach to guard

chrome/renderer/resources/extensions/send_request.js

 // Copyright (c) 2012 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 chromeHidden = requireNative('chrome_hidden').GetChromeHidden();
 var DCHECK = requireNative('logging').DCHECK;
+var forEach = require('utils').forEach;
 var json = require('json');
 var lastError = require('lastError');
 var natives = requireNative('sendRequest');
 var validate = require('schemaUtils').validate;
 
 // All outstanding requests from sendRequest().
 var requests = {};
 
 // Callback handling.
 chromeHidden.handleResponse = function(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 chromesForLastError = [chrome];
 
   try {
     var request = requests[requestId];
     DCHECK(request != null);
 
     // lastError needs to be set on the caller's chrome object no matter what,
     // though chances are it's the same as ours (it will be different when
     // calling API methods on other contexts).
     if (request.callback) {
       var chromeForCallback = natives.GetGlobal(request.callback).chrome;
       if (chromeForCallback != chrome)
         chromesForLastError.push(chromeForCallback);
     }
 
-    chromesForLastError.forEach(function(c) {lastError.clear(c)});
+    forEach(chromesForLastError, function(i, c) {lastError.clear(c)});
     if (!success) {
       if (!error)
         error = "Unknown error.";
-      chromesForLastError.forEach(function(c) {lastError.set(error, c)});
+      forEach(chromesForLastError, function(i, c) {lastError.set(error, c)});
     }
 
     if (request.customCallback) {
       var customCallbackArgs = [name, request].concat(responseList);
       request.customCallback.apply(request, customCallbackArgs);
     }
 
     if (request.callback) {
       // Validate callback in debug only -- and only when the
       // caller has provided a callback. Implementations of api
       // calls may not return data if they observe the caller
       // has not provided a callback.
       if (chromeHidden.validateCallbacks && !error) {
         try {
           if (!request.callbackSchema.parameters) {
             throw new Error("No callback schemas defined");
           }
 
           validate(responseList, request.callbackSchema.parameters);
         } catch (exception) {
           return "Callback validation error during " + name + " -- " +
                  exception.stack;
         }
       }
 
       request.callback.apply(request, responseList);
     }
   } finally {
     delete requests[requestId];
-    chromesForLastError.forEach(function(c) {lastError.clear(c)});
+    forEach(chromesForLastError, function(i, c) {lastError.clear(c)});
   }
 };
 
 function prepareRequest(args, argSchemas) {
   var request = {};
   var argCount = args.length;
 
   // Look for callback param.
   if (argSchemas.length > 0 &&
       argSchemas[argSchemas.length - 1].type == "function") {
@@ skipping 46 matching lines @@
   var hasCallback = request.callback || optArgs.customCallback;
   return nativeFunction(functionName,
                         requestArgs,
                         requestId,
                         hasCallback,
                         optArgs.forIOThread,
                         optArgs.preserveNullInObjects);
 }
 
 exports.sendRequest = sendRequest;