Some cleanup fixes

src/heap.h

Index: src/heap.h
diff --git a/src/heap.h b/src/heap.h
index fd361fc7a4655709385ab0f560b750caf8990c8a..565ec6f0b9b3aadb68125332a04da5157f60b659 100644
--- a/src/heap.h
+++ b/src/heap.h
@@ -666,10 +666,9 @@ class Heap {
   // Returns a deep copy of the JavaScript object.
   // Properties and elements are copied too.
   // Returns failure if allocation failed.
-  MUST_USE_RESULT MaybeObject* CopyJSObject(JSObject* source);
-
-  MUST_USE_RESULT MaybeObject* CopyJSObjectWithAllocationSite(
-      JSObject* source, AllocationSite* site);
+  // Optionally takes an AllocationSite to be appended in an AllocationMemento.
+  MUST_USE_RESULT MaybeObject* CopyJSObject(JSObject* source,
+                                            AllocationSite* site = NULL);
 
   // Allocates the function prototype.
   // Returns Failure::RetryAfterGC(requested_bytes, space) if the allocation
@@ -1007,7 +1006,7 @@ class Heap {
                                                     Context* previous,
                                                     ScopeInfo* info);
 
-  // Allocates a new utility object in the old generation.

[Michael Starzinger, 2013/09/19 16:45:25]

I liked that comment, can we get it back?

[mvstanton, 2013/09/20 09:19:15]

absolutely, it was removed because for a while I was allowing creation of the
structs in the new generation, but as we discussed with always_allocate(), the
scenario is not worth it.

+  // Allocates a new utility object
   MUST_USE_RESULT MaybeObject* AllocateStruct(InstanceType type);
 
   // Allocates a function initialized with a shared part.