win: Handle binary with embedded CodeView debug record

snapshot/win/module_snapshot_win.cc

Index: snapshot/win/module_snapshot_win.cc
diff --git a/snapshot/win/module_snapshot_win.cc b/snapshot/win/module_snapshot_win.cc
index f45bedc68741075ba5c4523dbba1bf60b2fd17ae..9e9d14c3aedb0c041039719239e00f8547147a9a 100644
--- a/snapshot/win/module_snapshot_win.cc
+++ b/snapshot/win/module_snapshot_win.cc
@@ -60,6 +60,8 @@ bool ModuleSnapshotWin::Initialize(
           &uuid_, &age_dword, &pdb_name_)) {
     static_assert(sizeof(DWORD) == sizeof(uint32_t), "unexpected age size");
     age_ = age_dword;
+  } else {
+    pdb_name_ = base::UTF16ToUTF8(name_);

[Mark Mentovai, 2015/10/30 19:42:44]

Add a short comment saying that this isn’t how the MS tools deal with this
situation.

I think what you’ve done here is totally reasonable.

[scottmg, 2015/10/30 23:52:07]

Done.

   }
 
   INITIALIZATION_STATE_SET_VALID(initialized_);