win: Save contents of PEB to minidump

minidump/minidump_file_writer.cc

 // Copyright 2014 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 75 matching lines @@
     AddStream(exception.Pass());
   }
 
   auto module_list = make_scoped_ptr(new MinidumpModuleListWriter());
   module_list->InitializeFromSnapshot(process_snapshot->Modules());
   AddStream(module_list.Pass());
 
   auto crashpad_info = make_scoped_ptr(new MinidumpCrashpadInfoWriter());
   crashpad_info->InitializeFromSnapshot(process_snapshot);
 
+  memory_list->AddFromSnapshot(process_snapshot->ExtraMemory());

[Mark Mentovai, 2015/09/25 17:32:32]

It’s OK to do this after the crashpad_info stuff, but right here, it’s in the
middle of the crashpad_info.

[scottmg, 2015/09/25 19:20:34]

Hmm, you're right. OK, at the end it is:
https://codereview.chromium.org/1369823002.

+
   // Since the MinidumpCrashpadInfo stream is an extension, it’s safe to not add
   // it to the minidump file if it wouldn’t carry any useful information.
   if (crashpad_info->IsUseful()) {
     AddStream(crashpad_info.Pass());
   }
 
   AddStream(memory_list.Pass());
 }
 
 void MinidumpFileWriter::SetTimestamp(time_t timestamp) {
@@ skipping 114 matching lines @@
   for (internal::MinidumpStreamWriter* stream : streams_) {
     iov.iov_base = stream->DirectoryListEntry();
     iov.iov_len = sizeof(MINIDUMP_DIRECTORY);
     iovecs.push_back(iov);
   }
 
   return file_writer->WriteIoVec(&iovecs);
 }
 
 }  // namespace crashpad