win: Plumb module PDB name through snapshot

snapshot/win/module_snapshot_win.h

Index: snapshot/win/module_snapshot_win.h
diff --git a/snapshot/win/module_snapshot_win.h b/snapshot/win/module_snapshot_win.h
index 7cdcc5668eb632ef470d513eac2864b27376e008..853baca4400a14bbf9bbbf5f5116810c16c3d1d5 100644
--- a/snapshot/win/module_snapshot_win.h
+++ b/snapshot/win/module_snapshot_win.h
@@ -81,6 +81,7 @@ class ModuleSnapshotWin final : public ModuleSnapshot {
                      uint16_t* version_3) const override;
   ModuleType GetModuleType() const override;
   void UUIDAndAge(crashpad::UUID* uuid, uint32_t* age) const override;
+  std::string DebugFileName() const override;
   std::vector<std::string> AnnotationsVector() const override;
   std::map<std::string, std::string> AnnotationsSimpleMap() const override;
 
@@ -89,6 +90,9 @@ class ModuleSnapshotWin final : public ModuleSnapshot {
   void GetCrashpadOptionsInternal(CrashpadInfoClientOptions* options);
 
   std::wstring name_;
+  std::string pdb_name_;
+  UUID uuid_;
+  uint32_t age_;
   time_t timestamp_;
   scoped_ptr<PEImageReader> pe_image_reader_;

[Mark Mentovai, 2015/10/29 13:11:10]

Best would be name, pdb_name, uuid, pe_image_reader, process_reader, timestamp,
age, initialized.

pe_image_reader, process_reader, and timestamp are the things that might be
either 4- or 8-byte quantities.

Might save a few bytes in a big vector of these.

[scottmg, 2015/10/29 17:47:00]

Done.

   ProcessReaderWin* process_reader_;  // weak