Use the correct product/version for postmortem reports

chrome/chrome_watcher/BUILD.gn

Index: chrome/chrome_watcher/BUILD.gn
diff --git a/chrome/chrome_watcher/BUILD.gn b/chrome/chrome_watcher/BUILD.gn
index 0d9265bd300b231f72c7f5421abad1bf3df09637..2bc822190205b4cced1e41896e01e7eb61d6d826 100644
--- a/chrome/chrome_watcher/BUILD.gn
+++ b/chrome/chrome_watcher/BUILD.gn
@@ -76,3 +76,18 @@ shared_library("chrome_watcher") {
   configs -= [ "//build/config/win:console" ]
   configs += [ "//build/config/win:windowed" ]
 }
+
+static_library("chrome_postmortem_report_collector") {

[grt (UTC plus 2), 2016/09/27 20:43:09]

i'm wondering if this is the right thing. some questions:
- since the CPRC is only used in browser/metrics, why not put the code for it
there?
- if not, why not in "client" above?
- if you think making this its own target is really the right thing, then i
think it should be a source_set rather than a static_library

please see my other comment about using a simple callback instead

[manzagop (departed), 2016/09/28 16:29:17]

Went with using a callback. Theoretically, the collector could be used by any
embedder. Keeping the questions about relocation in mind. 

+  sources = [
+    "chrome_postmortem_report_collector.cc",
+    "chrome_postmortem_report_collector.h",
+  ]
+  deps = [
+    "//base",
+    "//chrome/install_static:install_static_util",
+    "//components/browser_watcher:postmortem_report_collector",
+    "//components/browser_watcher:stability_report_proto",

[grt (UTC plus 2), 2016/09/27 20:43:09]

this is an unfortunate side effect of making cprc a subclass of prc. i think
it's best to avoid this.

[manzagop (departed), 2016/09/28 16:29:17]

Acknowledged.

+    "//third_party/crashpad/crashpad/client",
+    "//third_party/crashpad/crashpad/util",
+  ]
+}