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

Side by Side Diff: src/accessors.cc

Issue 1704353002: [runtime] Force internalize names used before lookup in in DescriptorArray and TransitionArray (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 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
« no previous file with comments | « no previous file | src/compiler/access-info.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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/accessors.h" 5 #include "src/accessors.h"
6 6
7 #include "src/api.h" 7 #include "src/api.h"
8 #include "src/contexts.h" 8 #include "src/contexts.h"
9 #include "src/deoptimizer.h" 9 #include "src/deoptimizer.h"
10 #include "src/execution.h" 10 #include "src/execution.h"
(...skipping 14 matching lines...) Expand all
25 Handle<Name> name, 25 Handle<Name> name,
26 AccessorNameGetterCallback getter, 26 AccessorNameGetterCallback getter,
27 AccessorNameSetterCallback setter, 27 AccessorNameSetterCallback setter,
28 PropertyAttributes attributes) { 28 PropertyAttributes attributes) {
29 Factory* factory = isolate->factory(); 29 Factory* factory = isolate->factory();
30 Handle<AccessorInfo> info = factory->NewAccessorInfo(); 30 Handle<AccessorInfo> info = factory->NewAccessorInfo();
31 info->set_property_attributes(attributes); 31 info->set_property_attributes(attributes);
32 info->set_all_can_read(false); 32 info->set_all_can_read(false);
33 info->set_all_can_write(false); 33 info->set_all_can_write(false);
34 info->set_is_special_data_property(true); 34 info->set_is_special_data_property(true);
35 info->set_name(*name); 35 info->set_name(*factory->InternalizeName(name));
36 Handle<Object> get = v8::FromCData(isolate, getter); 36 Handle<Object> get = v8::FromCData(isolate, getter);
37 if (setter == nullptr) setter = &ReconfigureToDataProperty; 37 if (setter == nullptr) setter = &ReconfigureToDataProperty;
38 Handle<Object> set = v8::FromCData(isolate, setter); 38 Handle<Object> set = v8::FromCData(isolate, setter);
39 info->set_getter(*get); 39 info->set_getter(*get);
40 info->set_setter(*set); 40 info->set_setter(*set);
41 return info; 41 return info;
42 } 42 }
43 43
44 44
45 static V8_INLINE bool CheckForName(Handle<Name> name, 45 static V8_INLINE bool CheckForName(Handle<Name> name,
(...skipping 29 matching lines...) Expand all
75 } 75 }
76 76
77 return false; 77 return false;
78 } 78 }
79 } 79 }
80 80
81 81
82 bool Accessors::IsJSArrayBufferViewFieldAccessor(Handle<Map> map, 82 bool Accessors::IsJSArrayBufferViewFieldAccessor(Handle<Map> map,
83 Handle<Name> name, 83 Handle<Name> name,
84 int* object_offset) { 84 int* object_offset) {
85 DCHECK(name->IsUniqueName());
85 Isolate* isolate = name->GetIsolate(); 86 Isolate* isolate = name->GetIsolate();
86 87
87 switch (map->instance_type()) { 88 switch (map->instance_type()) {
88 case JS_TYPED_ARRAY_TYPE: { 89 case JS_TYPED_ARRAY_TYPE: {
89 if (!CheckForName(name, isolate->factory()->length_string(), 90 if (!CheckForName(name, isolate->factory()->length_string(),
90 JSTypedArray::kLengthOffset, object_offset) && 91 JSTypedArray::kLengthOffset, object_offset) &&
91 !CheckForName(name, isolate->factory()->byte_length_string(), 92 !CheckForName(name, isolate->factory()->byte_length_string(),
92 JSTypedArray::kByteLengthOffset, object_offset) && 93 JSTypedArray::kByteLengthOffset, object_offset) &&
93 !CheckForName(name, isolate->factory()->byte_offset_string(), 94 !CheckForName(name, isolate->factory()->byte_offset_string(),
94 JSTypedArray::kByteOffsetOffset, object_offset)) { 95 JSTypedArray::kByteOffsetOffset, object_offset)) {
(...skipping 1109 matching lines...) Expand 10 before | Expand all | Expand 10 after
1204 Isolate* isolate = name->GetIsolate(); 1205 Isolate* isolate = name->GetIsolate();
1205 Handle<AccessorInfo> info = MakeAccessor(isolate, name, &ModuleGetExport, 1206 Handle<AccessorInfo> info = MakeAccessor(isolate, name, &ModuleGetExport,
1206 &ModuleSetExport, attributes); 1207 &ModuleSetExport, attributes);
1207 info->set_data(Smi::FromInt(index)); 1208 info->set_data(Smi::FromInt(index));
1208 return info; 1209 return info;
1209 } 1210 }
1210 1211
1211 1212
1212 } // namespace internal 1213 } // namespace internal
1213 } // namespace v8 1214 } // namespace v8
OLDNEW
« no previous file with comments | « no previous file | src/compiler/access-info.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698