AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

[api] Use handle for ConfigureInstance

Browse Source 
BUG=chromium:641267
R=verwaest@chromium.org

Review-Url: https://codereview.chromium.org/2283713002
Cr-Commit-Position: refs/heads/master@{#38942}
This commit is contained in:
mlippautz 2016-08-26 05:35:40 -07:00 committed by Commit bot
parent db56339952
commit 2059fb0ba6
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/api-natives.cc
View File

@ -199,15 +199,14 @@ MaybeHandle<JSObject> ConfigureInstance(Isolate* isolate, Handle<JSObject> obj,
Handle<FixedArray> array = Handle<FixedArray> array =
isolate->factory()->NewFixedArray(max_number_of_properties); isolate->factory()->NewFixedArray(max_number_of_properties);
info = *data; for (Handle<TemplateInfoT> temp(*data); *temp != nullptr;
while (info != nullptr) { temp = handle(temp->GetParent(isolate), isolate)) {
// Accumulate accessors. // Accumulate accessors.
Object* maybe_properties = info->property_accessors(); Object* maybe_properties = temp->property_accessors();
if (!maybe_properties->IsUndefined(isolate)) { if (!maybe_properties->IsUndefined(isolate)) {
valid_descriptors = AccessorInfo::AppendUnique( valid_descriptors = AccessorInfo::AppendUnique(
handle(maybe_properties, isolate), array, valid_descriptors); handle(maybe_properties, isolate), array, valid_descriptors);
} }
info = info->GetParent(isolate);
} }
// Install accumulated accessors. // Install accumulated accessors.