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

Unified Diff: src/keys.cc

Issue 2624903003: [runtime] Use PropertyKind/PropertyLocation instead of PropertyType. (Closed)
Patch Set: Created 3 years, 11 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
Index: src/keys.cc
diff --git a/src/keys.cc b/src/keys.cc
index 9b6c8f3381f1f91e83ae4c422735435f0a1c86cb..34cd43b042ec520dbb6c7af8bc95bca3cd99bbb4 100644
--- a/src/keys.cc
+++ b/src/keys.cc
@@ -328,12 +328,12 @@ Handle<FixedArray> GetFastEnumPropertyKeys(Isolate* isolate,
if (key->IsSymbol()) continue;
storage->set(index, key);
if (!indices.is_null()) {
- if (details.type() != DATA) {
- indices = Handle<FixedArray>();
- } else {
+ if (details.location() == kField) {
FieldIndex field_index = FieldIndex::ForDescriptor(*map, i);
int load_by_field_index = field_index.GetLoadByFieldIndex();
indices->set(index, Smi::FromInt(load_by_field_index));
+ } else {
+ indices = Handle<FixedArray>();
}
}
index++;

Powered by Google App Engine
This is Rietveld 408576698