TraversableMap only dependent on DescriptorArray

src/objects.h

Index: src/objects.h
diff --git a/src/objects.h b/src/objects.h
index 4fd29ad56227d368d2ebb0773c566f6cba3909b0..116c92b54025c5ec938a81c04c0c4d87085d5460 100644
--- a/src/objects.h
+++ b/src/objects.h
@@ -2475,6 +2475,10 @@ class DescriptorArray: public FixedArray {
   inline String* GetKey(int descriptor_number);
   inline Object* GetValue(int descriptor_number);
   inline PropertyDetails GetDetails(int descriptor_number);
+
+  inline Object* RawGetValue(int descriptor_number);

[Michael Starzinger, 2012/05/24 14:02:25]

These methods are only needed by the IntrusiveMapTransitionIterator, can we move
them into that class (into the private section)? That prevents others from
accidentally using these methods.

Also I would rename them to GetRawValue and GetEawDetails which is more natural
sounding.

[Toon Verwaest, 2012/05/25 11:36:46]

Done.

+  inline PropertyDetails RawGetDetails(int descriptor_number);
+
   inline PropertyType GetType(int descriptor_number);
   inline int GetFieldIndex(int descriptor_number);
   inline JSFunction* GetConstantFunction(int descriptor_number);
@@ -2650,6 +2654,7 @@ class DescriptorArray: public FixedArray {
   FixedArray* GetContentArray() {
     return FixedArray::cast(get(kContentArrayIndex));
   }
+  inline FixedArray* RawGetContentArray();

[Michael Starzinger, 2012/05/24 14:02:25]

Likewise.

[Toon Verwaest, 2012/05/25 11:36:46]

Done.

   DISALLOW_IMPLICIT_CONSTRUCTORS(DescriptorArray);
 };