Use the correct product/version for postmortem reports

chrome/app/chrome_crash_reporter_client_win.h

Index: chrome/app/chrome_crash_reporter_client_win.h
diff --git a/chrome/app/chrome_crash_reporter_client_win.h b/chrome/app/chrome_crash_reporter_client_win.h
index 8dc896444f91fb0ae1f7b5a08d34cee575e60235..450aba7383113c7c76fc2a49e3041659121738c8 100644
--- a/chrome/app/chrome_crash_reporter_client_win.h
+++ b/chrome/app/chrome_crash_reporter_client_win.h
@@ -6,6 +6,7 @@
 #define CHROME_APP_CHROME_CRASH_REPORTER_CLIENT_WIN_H_
 
 #include "base/macros.h"
+#include "base/strings/string16.h"
 #include "components/crash/content/app/crash_reporter_client.h"
 
 class ChromeCrashReporterClient : public crash_reporter::CrashReporterClient {
@@ -52,6 +53,15 @@ class ChromeCrashReporterClient : public crash_reporter::CrashReporterClient {
   bool EnableBreakpadForProcess(const std::string& process_type) override;
 
  private:
+  // Cached to ensure consistency across calls and because the functions that

[grt (UTC plus 2), 2016/09/26 18:59:06]

while i think this caching is ultimately the right thing, i'm not sure it's
needed right now. this choice means that you must get the one true CCRC for the
process in your own code. as per my comment in the other file, i'm not sure you
can. without trying to solve this problem here, you could call install_static
directly. it may be the case that the magic install_static function will
automagically do the once-per-process thing. since crash isn't the only consumer
of these values, i think this is the wrong place to do the caching anyway.

[manzagop (departed), 2016/09/27 19:37:52]

I've made the switch to directly call install static. Because that's in chrome/,
I couldn't do so in the component and I ended up writing a derived collector in
chrome/chrome_watcher that implements some pure functions to get the details.

+  // populate these are known to unexpectedly experience crashes on some users'
+  // machines.
+  base::string16 exe_path_;
+  base::string16 product_name_;
+  base::string16 version_;
+  base::string16 special_build_;
+  base::string16 channel_name_;
+
   DISALLOW_COPY_AND_ASSIGN(ChromeCrashReporterClient);
 };