Remove (but leave references to) all of the builtins that we don't need saved

chrome/renderer/extensions/safe_builtins.cc

 // Copyright 2013 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.
 
 #include "chrome/renderer/extensions/safe_builtins.h"
 
 #include "base/logging.h"
 #include "base/stl_util.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/renderer/extensions/chrome_v8_context.h"
@@ skipping 51 matching lines @@
     "  makeCallable(builtin, safe, true);\n"
     "  if (includePrototype)\n"
     "    makeCallable(builtin.prototype, safe, false);\n"
     "  return safe;\n"
     "}\n"
     "\n"
     "// Built-in types taken from the ECMAScript spec. The spec may change,\n"
     "// though. It would be nice to generate this somehow.\n"
     "// Note: no JSON, it needs to handle toJSON being overriden so is\n"
     "//       implemented by hand.\n"
-    "var builtinTypes = [Object, Function, Array, String, Boolean, Number,\n"
-    "                    /*Math,*/ Date, RegExp, /*JSON,*/ Error, EvalError,\n"
-    "                    ReferenceError, SyntaxError, TypeError, URIError];\n"
+    "// Note: no Math, it's static (we also don't need it yet).\n"
+    "// Note: the other things that are commented out are ones that aren't\n"
+    "//       needed to make the clobber tests pass, because running this\n"
+    "//       file is a bit slow.\n"
+    "var builtinTypes = [\n"
+    "    Object, Function, Array, String, /*Boolean, Number,*/\n"
+    "    /*Math, Date,*/ RegExp, /*JSON, Error, EvalError,\n"
+    "    ReferenceError, SyntaxError, TypeError, URIError*/];\n"
     "builtinTypes.forEach(function(builtin) {\n"
     "  Save(builtin.name, getSafeBuiltin(builtin, true));\n"
     "});\n"
-    "Save('Math', getSafeBuiltin(Math, false));\n"
+    "//Save('Math', getSafeBuiltin(Math, false));\n"
     "// Save JSON. This is trickier because extensions can override toJSON in\n"
     "// incompatible ways, and we need to prevent that.\n"
     "var builtinToJSONs = builtinTypes.map(function(t) {\n"
     "  return t.toJSON;\n"
     "});\n"
     "var builtinArray = Array;\n"
     "var builtinJSONStringify = JSON.stringify;\n"
     "Save('JSON', {\n"
     "  parse: JSON.parse,\n"
     "  stringify: function(obj) {\n"
@@ skipping 131 matching lines @@
 
 v8::Local<v8::Object> SafeBuiltins::GetRegExp() const {
   return Load("RegExp", context_->v8_context());
 }
 
 v8::Local<v8::Object> SafeBuiltins::GetString() const {
   return Load("String", context_->v8_context());
 }
 
 } //  namespace extensions