Use the correct product/version for postmortem reports

chrome/chrome_watcher/BUILD.gn

 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//chrome/process_version_rc_template.gni")
 import("//testing/test.gni")
 
 source_set("client") {
   sources = [
     "chrome_watcher_main_api.cc",
@@ skipping 58 matching lines @@
     "//base:base_static",
     "//build/config/sanitizers:deps",
     "//chrome/common:metrics_constants_util_win",
     "//chrome/installer/util:with_no_strings",
     "//components/browser_watcher",
   ]
   ldflags = [ "/DEF:" + rebase_path("chrome_watcher.def", root_build_dir) ]
   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",
+  ]
+}