Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(131)

Side by Side Diff: src/accessors.cc

Issue 6577036: [Isolates] Merge from bleeding_edge to isolates, revisions 6100-6300. (Closed) Base URL: http://v8.googlecode.com/svn/branches/experimental/isolates/
Patch Set: '' Created 9 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/SConscript ('k') | src/api.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2006-2008 the V8 project authors. All rights reserved. 1 // Copyright 2006-2008 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 object = *object_handle; 119 object = *object_handle;
120 value = *value_handle; 120 value = *value_handle;
121 121
122 if (uint32_v->Number() == number_v->Number()) { 122 if (uint32_v->Number() == number_v->Number()) {
123 if (object->IsJSArray()) { 123 if (object->IsJSArray()) {
124 return JSArray::cast(object)->SetElementsLength(*uint32_v); 124 return JSArray::cast(object)->SetElementsLength(*uint32_v);
125 } else { 125 } else {
126 // This means one of the object's prototypes is a JSArray and 126 // This means one of the object's prototypes is a JSArray and
127 // the object does not have a 'length' property. 127 // the object does not have a 'length' property.
128 // Calling SetProperty causes an infinite loop. 128 // Calling SetProperty causes an infinite loop.
129 return object->IgnoreAttributesAndSetLocalProperty(HEAP->length_symbol(), 129 return object->SetLocalPropertyIgnoreAttributes(HEAP->length_symbol(),
130 value, NONE); 130 value, NONE);
131 } 131 }
132 } 132 }
133 return Isolate::Current()->Throw( 133 return Isolate::Current()->Throw(
134 *FACTORY->NewRangeError("invalid_array_length", 134 *FACTORY->NewRangeError("invalid_array_length",
135 HandleVector<Object>(NULL, 0))); 135 HandleVector<Object>(NULL, 0)));
136 } 136 }
137 137
138 138
139 const AccessorDescriptor Accessors::ArrayLength = { 139 const AccessorDescriptor Accessors::ArrayLength = {
140 ArrayGetLength, 140 ArrayGetLength,
(...skipping 631 matching lines...) Expand 10 before | Expand all | Expand 10 after
772 if (i > 0) { 772 if (i > 0) {
773 // Function in question was inlined. 773 // Function in question was inlined.
774 return ConstructArgumentsObjectForInlinedFunction(frame, function, i); 774 return ConstructArgumentsObjectForInlinedFunction(frame, function, i);
775 } else { 775 } else {
776 // If there is an arguments variable in the stack, we return that. 776 // If there is an arguments variable in the stack, we return that.
777 int index = function->shared()->scope_info()-> 777 int index = function->shared()->scope_info()->
778 StackSlotIndex(isolate->heap()->arguments_symbol()); 778 StackSlotIndex(isolate->heap()->arguments_symbol());
779 if (index >= 0) { 779 if (index >= 0) {
780 Handle<Object> arguments = 780 Handle<Object> arguments =
781 Handle<Object>(frame->GetExpression(index), isolate); 781 Handle<Object>(frame->GetExpression(index), isolate);
782 if (!arguments->IsTheHole()) return *arguments; 782 if (!arguments->IsArgumentsMarker()) return *arguments;
783 } 783 }
784 784
785 // If there isn't an arguments variable in the stack, we need to 785 // If there isn't an arguments variable in the stack, we need to
786 // find the frame that holds the actual arguments passed to the 786 // find the frame that holds the actual arguments passed to the
787 // function on the stack. 787 // function on the stack.
788 it.AdvanceToArgumentsFrame(); 788 it.AdvanceToArgumentsFrame();
789 frame = it.frame(); 789 frame = it.frame();
790 790
791 // Get the number of arguments and construct an arguments object 791 // Get the number of arguments and construct an arguments object
792 // mirror for the right frame. 792 // mirror for the right frame.
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
899 } 899 }
900 900
901 901
902 const AccessorDescriptor Accessors::ObjectPrototype = { 902 const AccessorDescriptor Accessors::ObjectPrototype = {
903 ObjectGetPrototype, 903 ObjectGetPrototype,
904 ObjectSetPrototype, 904 ObjectSetPrototype,
905 0 905 0
906 }; 906 };
907 907
908 } } // namespace v8::internal 908 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/SConscript ('k') | src/api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698