Handlify Runtime::InitializeIntrinsicFunctionNames.

src/runtime.h

Index: src/runtime.h
diff --git a/src/runtime.h b/src/runtime.h
index 3a5c4546a7f35ac099924558bf8f97eb011487b2..5ee345c645480bd7247b44670bc65c19b16706ef 100644
--- a/src/runtime.h
+++ b/src/runtime.h
@@ -801,8 +801,9 @@ class Runtime : public AllStatic {
   // Returns failure if an allocation fails.  In this case, it must be

[Michael Starzinger, 2014/04/14 11:10:50]

nit: Comment is outdated, I think we can drop the last two sentences.

   // retried with a new, empty StringDictionary, not with the same one.
   // Alternatively, heap initialization can be completely restarted.
-  MUST_USE_RESULT static MaybeObject* InitializeIntrinsicFunctionNames(
-      Heap* heap, Object* dictionary);
+  static Handle<NameDictionary> InitializeIntrinsicFunctionNames(

[Michael Starzinger, 2014/04/14 11:10:50]

Since we have the invariant that the dictionary does not change, can we change
the signature to return "void" instead?

+      Isolate* isolate,
+      Handle<NameDictionary> dict);
 
   // Get the intrinsic function with the given name, which must be internalized.
   static const Function* FunctionForName(Handle<String> name);