Heap snapshot visualizations

runtime/vm/service.cc

 // Copyright (c) 2013, 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/service.h"
 
 #include "include/dart_api.h"
 #include "platform/globals.h"
 
 #include "vm/compiler.h"
@@ skipping 2234 matching lines @@
   jsobj.AddProperty("type", "OK");
   jsobj.AddProperty("id", "ok");
   return true;
 }
 
 
 void Service::SendGraphEvent(Isolate* isolate) {
   uint8_t* buffer = NULL;
   WriteStream stream(&buffer, &allocator, 1 * MB);
   ObjectGraph graph(isolate);
-  graph.Serialize(&stream);
-  JSONStream js;
-  {
-    JSONObject jsobj(&js);
+  intptr_t node_count = graph.Serialize(&stream);
+
+  // Chrome crashes receiving a single tens-of-megabytes blob, so send the
+  // snapshot in megabyte-sized chunks instead.

[koda, 2015/05/19 20:12:36]

Add named constant for chunk size.

[rmacnak, 2015/05/19 22:16:08]

Done.

+  intptr_t num_chunks = (stream.bytes_written() + (MB - 1)) / MB;
+  for (intptr_t i = 0; i < num_chunks; i++) {
+    JSONStream js;
     {
-      JSONObject event(&jsobj, "event");
-      event.AddProperty("type", "ServiceEvent");
-      event.AddProperty("eventType", "_Graph");
-      event.AddProperty("isolate", isolate);
+      JSONObject jsobj(&js);
+      {
+        JSONObject event(&jsobj, "event");
+        event.AddProperty("type", "ServiceEvent");
+        event.AddProperty("eventType", "_Graph");
+        event.AddProperty("isolate", isolate);
+
+        event.AddProperty("i", i);
+        event.AddProperty("n", num_chunks);
+        event.AddProperty("nodeCount", node_count);
+      }
     }
+    const String& message = String::Handle(String::New(js.ToCString()));
+
+    uint8_t* chunk_start = buffer + (i * MB);
+    intptr_t chunk_size = (i + 1 == num_chunks)

[koda, 2015/05/19 20:12:35]

Does this do the right thing when the last chunk is exactly 1 MB? The remainder
will be 0.

[rmacnak, 2015/05/19 22:16:08]

No, fixed.

+      ? stream.bytes_written() % MB
+      : MB;
+
+    SendEvent(message, chunk_start, chunk_size);
   }
-  const String& message = String::Handle(String::New(js.ToCString()));
-  SendEvent(message, buffer, stream.bytes_written());
 }
 
 
 void Service::SendInspectEvent(Isolate* isolate, const Object& inspectee) {
   ServiceEvent event(isolate, ServiceEvent::kInspect);
   event.set_inspectee(&inspectee);
   Service::HandleEvent(&event);
 }
 
 
@@ skipping 368 matching lines @@
     ServiceMethodDescriptor& method = service_methods_[i];
     if (strcmp(method_name, method.name) == 0) {
       return &method;
     }
   }
   return NULL;
 }
 
 
 }  // namespace dart