win: Write memory map info as MINIDUMP_MEMORY_INFO[_LIST]

minidump/test/minidump_writable_test_util.cc

Index: minidump/test/minidump_writable_test_util.cc
diff --git a/minidump/test/minidump_writable_test_util.cc b/minidump/test/minidump_writable_test_util.cc
index d16aed9472fa9cd290f52c56d030939206bfa333..ef00f40a610f3bed68d7219d11d01225a9ad3bc7 100644
--- a/minidump/test/minidump_writable_test_util.cc
+++ b/minidump/test/minidump_writable_test_util.cc
@@ -181,6 +181,14 @@ struct MinidumpThreadListTraits {
   }
 };
 
+struct MinidumpMemoryInfoListTraits {
+  using ListType = MINIDUMP_MEMORY_INFO_LIST;
+  enum : size_t { kElementSize = sizeof(MINIDUMP_MEMORY_INFO) };

[Mark Mentovai, 2015/10/12 14:34:47]

Not mandated by the Minidump format, but since there isn’t anything like
MINIDUMP_MEMORY_INFO2 for now, this is OK.

[scottmg, 2015/10/13 19:51:29]

Good point, I was only looking at it from one side.

+  static size_t ElementCount(const ListType* list) {
+    return static_cast<size_t>(list->NumberOfEntries);
+  }
+};
+
 struct MinidumpModuleCrashpadInfoListTraits {
   using ListType = MinidumpModuleCrashpadInfoList;
   enum : size_t { kElementSize = sizeof(MinidumpModuleCrashpadInfoLink) };
@@ -245,6 +253,14 @@ const MINIDUMP_THREAD_LIST* MinidumpWritableAtLocationDescriptor<
 }
 
 template <>
+const MINIDUMP_MEMORY_INFO_LIST* MinidumpWritableAtLocationDescriptor<
+    MINIDUMP_MEMORY_INFO_LIST>(const std::string& file_contents,
+                               const MINIDUMP_LOCATION_DESCRIPTOR& location) {
+  return MinidumpListAtLocationDescriptor<MinidumpMemoryInfoListTraits>(
+      file_contents, location);
+}
+
+template <>
 const MinidumpModuleCrashpadInfoList*
 MinidumpWritableAtLocationDescriptor<MinidumpModuleCrashpadInfoList>(
     const std::string& file_contents,