Ensure PrintToJSONStream can be called on all heap objects without crashing.

runtime/vm/object_test.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/assembler.h"
 #include "vm/bigint_operations.h"
 #include "vm/class_finalizer.h"
 #include "vm/dart_api_impl.h"
 #include "vm/dart_entry.h"
 #include "vm/debugger.h"
@@ skipping 4040 matching lines @@
 
   // Simple map.
   //
   // TODO(turnidge): Consider showing something like: {1: 2, 2: 'otter'}
   result = Dart_GetField(lib, NewString("simple_map"));
   EXPECT_VALID(result);
   obj ^= Api::UnwrapHandle(result);
   EXPECT_STREQ("Instance of '_LinkedHashMap'", obj.ToUserCString());
 }
 
+
+class JSONTypeVerifier : public ObjectVisitor {
+ public:
+  JSONTypeVerifier() : ObjectVisitor(Isolate::Current()) {}
+  virtual ~JSONTypeVerifier() { }
+  virtual void VisitObject(RawObject* obj) {
+    // Free-list elements cannot even be wrapped in handles.
+    if (obj->IsFreeListElement()) {
+      return;
+    }
+    Object& handle = Object::Handle(obj);
+    // Skip some common simple objects to run in reasonable time.
+    if (handle.IsString() ||
+        handle.IsArray() ||
+        handle.IsLiteralToken()) {
+      return;
+    }
+    JSONStream js;
+    handle.PrintToJSONStream(&js, false);
+    // TODO(koda): When all objects include a "type" field, expect that here.
+  }
+};
+
+
+TEST_CASE(PrintToJSONStream) {
+  Heap* heap = Isolate::Current()->heap();
+  heap->CollectAllGarbage();
+  JSONTypeVerifier verifier;
+  heap->IterateObjects(&verifier);
+}
+
 }  // namespace dart