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

Unified Diff: src/api-natives.cc

Issue 1225213008: Revert preallocating of descriptors since right now getters and setters cause copying of descriptor… (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 5 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/api-natives.cc
diff --git a/src/api-natives.cc b/src/api-natives.cc
index 9e4ff99ef241c450d61043b144ff372e8e2f0120..5993859710a5e1d4063b6900c5af6d1168e82a97 100644
--- a/src/api-natives.cc
+++ b/src/api-natives.cc
@@ -151,15 +151,6 @@ MaybeHandle<JSObject> ConfigureInstance(Isolate* isolate, Handle<JSObject> obj,
PropertyAttributes attributes = details.attributes();
PropertyKind kind = details.kind();
- if (obj->map()->owns_descriptors() &&
- obj->map()->instance_descriptors()->length() != 0 &&
- obj->map()->instance_descriptors()->NumberOfSlackDescriptors() == 0 &&
- TransitionArray::SearchTransition(obj->map(), kind, *name,
- attributes) == NULL) {
- Map::EnsureDescriptorSlack(handle(obj->map()),
- data->number_of_properties() - c);
- }
-
if (kind == kData) {
auto prop_data = handle(properties.get(i++), isolate);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698