Use the correct product/version for postmortem reports

components/browser_watcher/postmortem_report_collector.cc

 // Copyright 2016 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.
 
 #include "components/browser_watcher/postmortem_report_collector.h"
 
+#include <string>
 #include <utility>
 
 #include "base/debug/activity_analyzer.h"
 #include "base/files/file_enumerator.h"
 #include "base/files/file_util.h"
 #include "base/logging.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/path_service.h"
+#include "base/strings/string16.h"
+#include "base/strings/utf_string_conversions.h"
 #include "components/browser_watcher/postmortem_minidump_writer.h"
-#include "components/version_info/version_info.h"
+#include "components/crash/content/app/crash_reporter_client.h"
 #include "third_party/crashpad/crashpad/client/settings.h"
 #include "third_party/crashpad/crashpad/util/misc/uuid.h"
 
 using base::FilePath;
 
 namespace browser_watcher {
 
 using base::debug::GlobalActivityAnalyzer;
 using base::debug::ThreadActivityAnalyzer;
 using crashpad::CrashReportDatabase;
@@ skipping 153 matching lines @@
   }
 
   return SUCCESS;
 }
 
 bool PostmortemReportCollector::WriteReportToMinidump(
     const StabilityReport& report,
     const crashpad::UUID& client_id,
     const crashpad::UUID& report_id,
     base::PlatformFile minidump_file) {
+  CrashReporterClient* reporter_client = GetCrashReporterClient();

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

does this compile? GetCrashReporterClient is within a CRASH_IMPLEMENTATION
block, so i thought it was unreachable by client code.

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

It did. Not sure why though. Removed this code as per other comment.

+  wchar_t exe_file[MAX_PATH] = {};
+  CHECK(::GetModuleFileName(nullptr, exe_file, arraysize(exe_file)));
+  base::string16 product_name, version_number, special_build, channel_name;
+  reporter_client->GetProductNameAndVersion(
+      exe_file, &product_name, &version_number, &special_build, &channel_name);
+
   MinidumpInfo minidump_info;
   minidump_info.client_id = client_id;
   minidump_info.report_id = report_id;
-  minidump_info.product_name = version_info::GetProductName();
-  minidump_info.version_number = version_info::GetVersionNumber();
+  minidump_info.product_name = base::UTF16ToUTF8(product_name);
+  minidump_info.version_number = base::UTF16ToUTF8(version_number);
+  minidump_info.channel_name = base::UTF16ToUTF8(channel_name);
+#if defined(ARCH_CPU_X86)
+  minidump_info.platform = std::string("Win32");
+#elif defined(ARCH_CPU_X86_64)
+  minidump_info.platform = std::string("Win64");
+#endif
 
   return WritePostmortemDump(minidump_file, report, minidump_info);
 }
 
 }  // namespace browser_watcher