Converted Observatory heap-profile element

runtime/vm/class_table.cc

Index: runtime/vm/class_table.cc
diff --git a/runtime/vm/class_table.cc b/runtime/vm/class_table.cc
index 334bca9c2f974122eb5a16edc1d995f4d463f744..bc45839fe1f06de10ae3310b98b26663c61235ff 100644
--- a/runtime/vm/class_table.cc
+++ b/runtime/vm/class_table.cc
@@ -521,14 +521,18 @@ void ClassTable::AllocationProfilePrintJSON(JSONStream* stream) {
   ASSERT(heap != NULL);
   JSONObject obj(stream);
   obj.AddProperty("type", "AllocationProfile");
-  obj.AddPropertyF(
-      "dateLastAccumulatorReset",
-      "%" Pd64 "",
-      isolate->last_allocationprofile_accumulator_reset_timestamp());
-  obj.AddPropertyF(
-      "dateLastServiceGC",
-      "%" Pd64 "",
-      isolate->last_allocationprofile_gc_timestamp());
+  if (isolate->last_allocationprofile_accumulator_reset_timestamp()) {

[Cutch, 2016/08/17 17:35:10]

the type of

isolate->last_allocationprofile_accumulator_reset_timestamp()

is int64_t

Please change this so that the expression is a bool:

isolate->last_allocationprofile_accumulator_reset_timestamp() != 0

[cbernaschina, 2016/08/17 17:55:52]

Done.

+    obj.AddPropertyF(
+        "dateLastAccumulatorReset",
+        "%" Pd64 "",
+        isolate->last_allocationprofile_accumulator_reset_timestamp());
+  }
+  if (isolate->last_allocationprofile_gc_timestamp()) {

[Cutch, 2016/08/17 17:35:10]

ditto

[cbernaschina, 2016/08/17 17:55:52]

Done.

+    obj.AddPropertyF(
+        "dateLastServiceGC",
+        "%" Pd64 "",
+        isolate->last_allocationprofile_gc_timestamp());
+  }
 
   {
     JSONObject heaps(&obj, "heaps");