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

Unified Diff: src/objects.cc

Issue 1412893005: Revert of Use C++ implementation of Object.definePropert{y,ies} (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/js/v8natives.js ('k') | src/property-descriptor.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index 0079ac134546ec471a61cc7956f4ce4d9ddf3148..97b4a0e1782999c39840ddfc29ff1761edd32fd7 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -6516,7 +6516,7 @@
if (!success && should_throw == THROW_ON_ERROR) {
isolate->Throw(*isolate->factory()->NewTypeError(
MessageTemplate::kStrictDeleteProperty,
- isolate->factory()->NewNumberFromUint(actual_new_len - 1), a));
+ isolate->factory()->NewNumberFromUint(actual_new_len - 1)));
}
return success;
}
@@ -7784,10 +7784,7 @@
DCHECK(filter == INCLUDE_SYMBOLS);
PropertyAttributes attr_filter =
static_cast<PropertyAttributes>(DONT_ENUM | PRIVATE_SYMBOL);
- Handle<FixedArray> property_keys = isolate->factory()->NewFixedArray(
- current->NumberOfOwnProperties(attr_filter));
- current->GetOwnPropertyNames(*property_keys, 0, attr_filter);
- accumulator.AddKeys(property_keys);
+ JSObject::CollectOwnElementKeys(current, &accumulator, attr_filter);
}
// Add the property keys from the interceptor.
« no previous file with comments | « src/js/v8natives.js ('k') | src/property-descriptor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698