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

chrome/renderer/resources/extensions/platform_app.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 forEach = require('utils').forEach;
+
 /**
  * Returns a function that throws a 'not available' exception when called.
  *
  * @param {string} messagePrefix text to prepend to the exception message.
  */
 function generateDisabledMethodStub(messagePrefix, opt_messageSuffix) {
   return function() {
     var message = messagePrefix + ' is not available in packaged apps.';
     if (opt_messageSuffix) message = message + ' ' + opt_messageSuffix;
     throw message;
   };
 }
 
 /**
  * Replaces the given methods of the passed in object with stubs that throw
  * 'not available' exceptions when called.
  *
  * @param {Object} object The object with methods to disable. The prototype is
  *     preferred.
  * @param {string} objectName The display name to use in the error message
  *     thrown by the stub (this is the name that the object is commonly referred
  *     to by web developers, e.g. "document" instead of "HTMLDocument").
  * @param {Array.<string>} methodNames names of methods to disable.
  */
 function disableMethods(object, objectName, methodNames) {
-  methodNames.forEach(function(methodName) {
+  forEach(methodNames, function(i, methodName) {
     object[methodName] =
         generateDisabledMethodStub(objectName + '.' + methodName + '()');
   });
 }
 
 /**
  * Replaces the given properties of the passed in object with stubs that throw
  * 'not available' exceptions when gotten.  If a property's setter is later
  * invoked, the getter and setter are restored to default behaviors.
  *
  * @param {Object} object The object with properties to disable. The prototype
  *     is preferred.
  * @param {string} objectName The display name to use in the error message
  *     thrown by the getter stub (this is the name that the object is commonly
  *     referred to by web developers, e.g. "document" instead of
  *     "HTMLDocument").
  * @param {Array.<string>} propertyNames names of properties to disable.
  */
 function disableGetters(object, objectName, propertyNames, opt_messageSuffix) {
-  propertyNames.forEach(function(propertyName) {
+  forEach(propertyNames, function(i, propertyName) {
     var stub = generateDisabledMethodStub(objectName + '.' + propertyName,
                                           opt_messageSuffix);
     stub._is_platform_app_disabled_getter = true;
     object.__defineGetter__(propertyName, stub);
 
     object.__defineSetter__(propertyName, function(value) {
       var getter = this.__lookupGetter__(propertyName);
       if (!getter || getter._is_platform_app_disabled_getter) {
         // The stub getter is still defined.  Blow-away the property to restore
         // default getter/setter behaviors and re-create it with the given
@@ skipping 60 matching lines @@
     'onbeforeunload', generateDisabledMethodStub('onbeforeunload'));
 Window.prototype.__defineSetter__(
     'onunload', generateDisabledMethodStub('onunload'));
 var windowAddEventListener = Window.prototype.addEventListener;
 Window.prototype.addEventListener = function(type) {
   if (type === 'unload' || type === 'beforeunload')
     generateDisabledMethodStub(type)();
   else
     return windowAddEventListener.apply(window, arguments);
 };