Chromium Code Reviews| Index: src/runtime.cc |
| diff --git a/src/runtime.cc b/src/runtime.cc |
| index 15cfc854bfad6501cab80faed3743f81663fc629..d805a9a849c3f3d87e84573cfbf4176eb751b324 100644 |
| --- a/src/runtime.cc |
| +++ b/src/runtime.cc |
| @@ -566,6 +566,11 @@ RUNTIME_FUNCTION(MaybeObject*, Runtime_CreateArrayLiteral) { |
| RETURN_IF_EMPTY_HANDLE(isolate, site); |
| Handle<JSObject> boilerplate(JSObject::cast(site->transition_info())); |
| + if (boilerplate->elements()->map() == |
| + isolate->heap()->fixed_cow_array_map()) { |
| + isolate->counters()->cow_arrays_created_runtime()->Increment(); |
|
Michael Starzinger
2013/11/04 14:12:03
See comments in objects.cc about this.
mvstanton
2013/11/04 14:47:15
Right on, removed this code.
|
| + } |
| + |
| AllocationSiteUsageContext usage_context(isolate, site, true); |
| usage_context.EnterNewScope(); |
| Handle<JSObject> copy = JSObject::DeepCopy(boilerplate, &usage_context); |
| @@ -575,32 +580,6 @@ RUNTIME_FUNCTION(MaybeObject*, Runtime_CreateArrayLiteral) { |
| } |
| -RUNTIME_FUNCTION(MaybeObject*, Runtime_CreateArrayLiteralShallow) { |
| - HandleScope scope(isolate); |
| - ASSERT(args.length() == 3); |
| - CONVERT_ARG_HANDLE_CHECKED(FixedArray, literals, 0); |
| - CONVERT_SMI_ARG_CHECKED(literals_index, 1); |
| - CONVERT_ARG_HANDLE_CHECKED(FixedArray, elements, 2); |
| - |
| - Handle<AllocationSite> site = GetLiteralAllocationSite(isolate, literals, |
| - literals_index, elements); |
| - RETURN_IF_EMPTY_HANDLE(isolate, site); |
| - |
| - JSObject* boilerplate = JSObject::cast(site->transition_info()); |
| - if (boilerplate->elements()->map() == |
| - isolate->heap()->fixed_cow_array_map()) { |
| - isolate->counters()->cow_arrays_created_runtime()->Increment(); |
| - } |
| - |
| - if (AllocationSite::GetMode(boilerplate->GetElementsKind()) == |
| - TRACK_ALLOCATION_SITE) { |
| - return isolate->heap()->CopyJSObject(boilerplate, *site); |
| - } |
| - |
| - return isolate->heap()->CopyJSObject(boilerplate); |
| -} |
| - |
| - |
| RUNTIME_FUNCTION(MaybeObject*, Runtime_CreateSymbol) { |
| HandleScope scope(isolate); |
| ASSERT(args.length() == 1); |