Make functions on the built-in object non-writable.

src/v8globals.h

Index: src/v8globals.h
diff --git a/src/v8globals.h b/src/v8globals.h
index aff27579f8d78b6cd8337f9b8fba578237993c42..1fcd812b3ddd9e16f0a248373ab5eb91077f3614 100644
--- a/src/v8globals.h
+++ b/src/v8globals.h
@@ -482,8 +482,8 @@ enum CpuFeature { SSE4_1 = 32 + 19,  // x86
 
 // The Strict Mode (ECMA-262 5th edition, 4.2.2).
 enum StrictModeFlag {
-  kNonStrictMode,
-  kStrictMode,
+  kNonStrictMode = 0,

[Kevin Millikin (Chromium), 2011/09/01 07:58:24]

I couldn't find a site where we relied on a specific values for kNonStrictMode
or kStrictMode, so I'd rather not make this change.

It suggests the reader here that we already *do* rely on a specific value and so
encourages him to make use of that fact.  There doesn't seem a good reason to
introduce that dependency.

[Lasse Reichstein, 2011/09/01 09:26:58]

Good catch. It's an artifact of a preliminary change that didn't make it.

+  kStrictMode = 1,
   // This value is never used, but is needed to prevent GCC 4.5 from failing
   // to compile when we assert that a flag is either kNonStrictMode or
   // kStrictMode.