DM: JSON output adds process_max_resident_set_size_MB

dm/DMJsonWriter.cpp

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "DMJsonWriter.h"
 
+#include "ProcStats.h"
 #include "SkCommonFlags.h"
 #include "SkData.h"
 #include "SkJSONCPP.h"
 #include "SkOSFile.h"
 #include "SkStream.h"
 #include "SkTArray.h"
 #include "SkThread.h"
 
 namespace DM {
 
@@ skipping 47 matching lines @@
             Json::Value result;
             result["file_name"]     = gFailures[i].fileName;
             result["line_no"]       = gFailures[i].lineNo;
             result["condition"]     = gFailures[i].condition;
             result["message"]       = gFailures[i].message.c_str();
 
             root["test_results"]["failures"].append(result);
         }
     }
 
+    int maxResidentSetSizeMB = sk_tools::getMaxResidentSetSizeMB();
+    if (maxResidentSetSizeMB != -1) {
+        root["process_max_resident_set_size_MB"] =

[jcgregorio, 2015/02/12 13:15:00]

Can you think of a slightly shorter name?

Also, does this mean you would like perf to ingest DM results also?

[hal.canary, 2015/04/01 20:12:26]

Done

+            sk_tools::getMaxResidentSetSizeMB();
+    }
+
     SkString path = SkOSPath::Join(FLAGS_writePath[0], "dm.json");
     sk_mkdir(FLAGS_writePath[0]);
     SkFILEWStream stream(path.c_str());
     stream.writeText(Json::StyledWriter().write(root).c_str());
     stream.flush();
 }
 
 bool JsonWriter::ReadJson(const char* path, void(*callback)(BitmapResult)) {
     SkAutoTUnref<SkData> json(SkData::NewFromFileName(path));
     if (!json) {
@@ skipping 15 matching lines @@
         br.config     = r["key"]["config"].asCString();
         br.sourceType = r["key"]["source_type"].asCString();
         br.ext        = r["ext"].asCString();
         br.md5        = r["md5"].asCString();
         callback(br);
     }
     return true;
 }
 
 } // namespace DM