[runtime] use std::vector in KeyAccumulator

src/objects.h

Index: src/objects.h
diff --git a/src/objects.h b/src/objects.h
index 2f529a730eb1c8151ca4ddd236b7bafd1209b7ca..1e0fb942f4e62e5f57bb9809ec3ff12a0ad29405 100644
--- a/src/objects.h
+++ b/src/objects.h
@@ -10754,6 +10754,8 @@ class KeyAccumulator final BASE_EMBEDDED {
       : isolate_(isolate), filter_(filter), length_(0), levelLength_(0) {}
   ~KeyAccumulator();
 
+  enum ElementKeyAddCheckLimit { EXCLUDE_CURRENT_LEVEL, INCLUDE_CURRENT_LEVEL };
+
   bool AddKey(uint32_t key);
   bool AddKey(Object* key, AddKeyConversion convert = DO_NOT_CONVERT);
   bool AddKey(Handle<Object> key, AddKeyConversion convert = DO_NOT_CONVERT);
@@ -10762,6 +10764,7 @@ class KeyAccumulator final BASE_EMBEDDED {
   void AddKeys(Handle<JSObject> array,
                AddKeyConversion convert = DO_NOT_CONVERT);
   void AddKeysFromProxy(Handle<JSObject> array);
+  void AddKeysFromInterceptor(Handle<JSObject> array);
   // Jump to the next level, pushing the current |levelLength_| to
   // |levelLengths_| and adding a new list to |elements_|.
   void NextPrototype();
@@ -10774,10 +10777,10 @@ class KeyAccumulator final BASE_EMBEDDED {
   Isolate* isolate_;
   KeyFilter filter_;
   // |elements_| contains the sorted element keys (indices) per level.
-  List<List<uint32_t>*> elements_;
+  std::vector<std::vector<uint32_t>*> elements_;

[Igor Sheludko, 2015/10/22 21:42:57]

Suggestion: use std::unique_ptr:
  std::vector<std::unique_ptr<std::vector<uint32_t>>> elements_;
In this case you can remove destructor.

[Camillo Bruni, 2015/10/23 08:56:35]

I think I prefer not using smart pointers since we do not leak any objects, if
that is fine with you?

   // |protoLengths_| contains the total number of keys (elements + properties)
   // per level. Negative values mark counts for a level with keys from a proxy.
-  List<int> levelLengths_;
+  std::vector<int> levelLengths_;
   // |properties_| contains the property keys per level in insertion order.
   Handle<OrderedHashSet> properties_;
   // |length_| keeps track of the total number of all element and property keys.
@@ -10785,6 +10788,8 @@ class KeyAccumulator final BASE_EMBEDDED {
   // |levelLength_| keeps track of the total number of keys
   // (elements + properties) in the current level.
   int levelLength_;
+  // Indicate where to stop looking for duplicates when adding new element keys.
+  ElementKeyAddCheckLimit elementKeysAddCheckLimit_ = EXCLUDE_CURRENT_LEVEL;

[Igor Sheludko, 2015/10/22 21:42:57]

Nice! What about initializing lengths fields this way?

[Camillo Bruni, 2015/10/23 08:56:35]

done

 
   DISALLOW_COPY_AND_ASSIGN(KeyAccumulator);
 };