win: Plumb module PDB name through snapshot

snapshot/module_snapshot.h

Index: snapshot/module_snapshot.h
diff --git a/snapshot/module_snapshot.h b/snapshot/module_snapshot.h
index 6c397ed0a71e6210f988807d8ffea5e9b14ff340..189c1d7319a5cbfe3dd729993fb3281e15ae2655 100644
--- a/snapshot/module_snapshot.h
+++ b/snapshot/module_snapshot.h
@@ -120,6 +120,13 @@ class ModuleSnapshot {
   //! `0`.

[Mark Mentovai, 2015/10/28 20:01:10]

//! \sa DebugFileName()

[scottmg, 2015/10/28 20:59:25]

Done.

   virtual void UUIDAndAge(crashpad::UUID* uuid, uint32_t* age) const = 0;
 
+  //! \brief Returns the module’s PDB link name.
+  //!
+  //! On Windows, the PDB contains symbol information, held separately from the
+  //! module itself. On other platforms, this should just be the basename of the

[Mark Mentovai, 2015/10/28 20:01:10]

“should” → “is normally”

[scottmg, 2015/10/28 20:59:25]

Done.

+  //! module.

[Mark Mentovai, 2015/10/28 20:01:10]

“, because the debug info file’s name is not relevant even in split-debug
scenarios.”

[Mark Mentovai, 2015/10/28 20:01:11]

//! \sa UUIDAndAge()

[scottmg, 2015/10/28 20:59:25]

Done.

[scottmg, 2015/10/28 20:59:25]

Done.

+  virtual std::string PDBName() const = 0;

[Mark Mentovai, 2015/10/28 20:01:10]

Let’s try to keep things more generic in snapshot, like DebugFileName(). Applies
to the \brief on line 123 too.

[scottmg, 2015/10/28 20:59:24]

Done.

+
   //! \brief Returns string annotations recorded in the module.
   //!
   //! This method retrieves annotations recorded in a module. These annotations