Made PropertyType handling even more explicit.

src/profile-generator.cc

Index: src/profile-generator.cc
diff --git a/src/profile-generator.cc b/src/profile-generator.cc
index 7760d8f90e46feb65b8f2ab3c7a615c12bb93aa1..97da3567dfdc7129ce85205417594c1e27cd894c 100644
--- a/src/profile-generator.cc
+++ b/src/profile-generator.cc
@@ -2112,7 +2112,17 @@ void V8HeapExplorer::ExtractPropertyReferences(JSObject* js_obj,
               js_obj, entry,
               descs->GetKey(i), descs->GetConstantFunction(i));
           break;
-        default: ;
+        case NORMAL:  // only in slow mode
+        case HANDLER:  // only in lookup results, not in descriptors
+        case INTERCEPTOR:  // only in lookup results, not in descriptors
+        case MAP_TRANSITION:  // we do not care about transitions here...
+        case ELEMENTS_TRANSITION:
+        case CONSTANT_TRANSITION:
+        case NULL_DESCRIPTOR:  // ... and not about "holes"
+          break;
+          // TODO(svenpanne): Should we really ignore accessors here?

[Kevin Millikin (Chromium), 2011/11/09 12:39:54]

That's a good question.  They are really a reference to a function or pair of
functions, so we probably should not skip them.  It also looks like they are
counted in the dictionary case.  I'll ping Mikhail about it.

+        case CALLBACKS:
+          break;
       }
     }
   } else {