ES6 symbols: implement name property

src/objects.cc

Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index 5fb973894d01865bf5e7a7d3dd428316e9eb43ae..cedba4cf03ae74da1a00fd8e067bd1ebcc745dc0 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -1460,9 +1460,15 @@ void HeapObject::HeapObjectShortPrint(StringStream* accumulator) {
         accumulator->Add("<Odd Oddball>");
       break;
     }
-    case SYMBOL_TYPE:
-      accumulator->Add("<Symbol: %d>", Symbol::cast(this)->Hash());
+    case SYMBOL_TYPE: {
+      Symbol* symbol = Symbol::cast(this);
+      accumulator->Add("<Symbol: %d", symbol->Hash());
+      if (!symbol->name()->IsUndefined()) {
+        String::cast(symbol->name())->StringShortPrint(accumulator);

[Michael Starzinger, 2013/03/22 12:03:25]

I think there is a white-space missing between the hash and the name in the
printout.

[rossberg, 2013/03/22 13:02:42]

That actually was intentional, but I added a space anyway. :)

+      }
+      accumulator->Add(">");
       break;
+    }
     case HEAP_NUMBER_TYPE:
       accumulator->Add("<Number: ");
       HeapNumber::cast(this)->HeapNumberPrint(accumulator);
@@ -1572,6 +1578,8 @@ void HeapObject::IterateBody(InstanceType type, int object_size,
       JSGlobalPropertyCell::BodyDescriptor::IterateBody(this, v);
       break;
     case SYMBOL_TYPE:
+      Symbol::BodyDescriptor::IterateBody(this, v);
+      break;
     case HEAP_NUMBER_TYPE:
     case FILLER_TYPE:
     case BYTE_ARRAY_TYPE: