Implement Math.random() purely in JavaScript.

src/bootstrapper.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 1290 matching lines @@
     native_context()->set_call_as_constructor_delegate(*delegate);
     delegate->shared()->DontAdaptArguments();
   }
 
   // Initialize the out of memory slot.
   native_context()->set_out_of_memory(heap->false_value());
 
   // Initialize the embedder data slot.
   Handle<FixedArray> embedder_data = factory->NewFixedArray(2);
   native_context()->set_embedder_data(*embedder_data);
-
-  // Allocate the random seed slot.
-  Handle<ByteArray> random_seed = factory->NewByteArray(kRandomStateSize);
-  native_context()->set_random_seed(*random_seed);
 }
 
 
 Handle<JSFunction> Genesis::InstallTypedArray(
     const char* name, ElementsKind elementsKind) {
   Handle<JSObject> global = Handle<JSObject>(native_context()->global_object());
   Handle<JSFunction> result = InstallFunction(global, name, JS_TYPED_ARRAY_TYPE,
       JSTypedArray::kSize, isolate()->initial_object_prototype(),
       Builtins::kIllegal, false, true);
 
@@ skipping 1305 matching lines @@
     if (!ConfigureGlobalObjects(global_template)) return;
     isolate->counters()->contexts_created_from_scratch()->Increment();
   }
 
   // Initialize experimental globals and install experimental natives.
   InitializeExperimentalGlobal();
   if (!InstallExperimentalNatives()) return;
 
   // Initially seed the per-context random number generator
   // using the per-isolate random number generator.
-  uint32_t* state = reinterpret_cast<uint32_t*>(
-      native_context()->random_seed()->GetDataStartAddress());
+  uint32_t state[2];
   do {
-    isolate->random_number_generator()->NextBytes(state, kRandomStateSize);
+    isolate->random_number_generator()->NextBytes(state, sizeof(state));
   } while (state[0] == 0 || state[1] == 0);
+  Handle<JSBuiltinsObject> builtins(native_context()->builtins());
+  ForceSetProperty(builtins,
+                   factory()->InternalizeOneByteString(
+                       STATIC_ASCII_VECTOR("random0")),
+                   factory()->NewHeapNumber(state[0]),

[Michael Starzinger, 2013/11/11 14:11:12]

Better use Factory::NewNumberFromUint here and in the call below.

[Sven Panne, 2013/11/12 07:09:12]

Done.

+                   NONE);
+  ForceSetProperty(builtins,
+                   factory()->InternalizeOneByteString(
+                       STATIC_ASCII_VECTOR("random1")),
+                   factory()->NewHeapNumber(state[1]),
+                   NONE);
 
   result_ = native_context();
 }
 
 
 // Support for thread preemption.
 
 // Reserve space for statics needing saving and restoring.
 int Bootstrapper::ArchiveSpacePerThread() {
   return sizeof(NestingCounterType);
@@ skipping 14 matching lines @@
   return from + sizeof(NestingCounterType);
 }
 
 
 // Called when the top-level V8 mutex is destroyed.
 void Bootstrapper::FreeThreadResources() {
   ASSERT(!IsActive());
 }
 
 } }  // namespace v8::internal