Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(159)

Unified Diff: components/browser_watcher/postmortem_report_collector.cc

Issue 2372633002: Use the correct product/version for postmortem reports (Closed)
Patch Set: Direct access to GetExecutableVersionDetails via ChromePostmortemReportCollector Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/browser_watcher/postmortem_report_collector.cc
diff --git a/components/browser_watcher/postmortem_report_collector.cc b/components/browser_watcher/postmortem_report_collector.cc
index 8d16952414a433bce6f6e40eed6898d7fde5fd2f..2ce656c9a61fa885e1d4b8b8a1879e1142a0a18f 100644
--- a/components/browser_watcher/postmortem_report_collector.cc
+++ b/components/browser_watcher/postmortem_report_collector.cc
@@ -4,6 +4,7 @@
#include "components/browser_watcher/postmortem_report_collector.h"
+#include <string>
#include <utility>
#include "base/debug/activity_analyzer.h"
@@ -13,7 +14,6 @@
#include "base/metrics/histogram_macros.h"
#include "base/path_service.h"
#include "components/browser_watcher/postmortem_minidump_writer.h"
-#include "components/version_info/version_info.h"
#include "third_party/crashpad/crashpad/client/settings.h"
#include "third_party/crashpad/crashpad/util/misc/uuid.h"
@@ -190,8 +190,14 @@ bool PostmortemReportCollector::WriteReportToMinidump(
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 = GetProductName();
+ minidump_info.version_number = GetProductVersion();
+ minidump_info.channel_name = GetProductChannel();
+#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);
}

Powered by Google App Engine
This is Rietveld 408576698