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

Side by Side Diff: chrome/chrome_watcher/chrome_postmortem_report_collector.h

Issue 2372633002: Use the correct product/version for postmortem reports (Closed)
Patch Set: Direct access to GetExecutableVersionDetails via ChromePostmortemReportCollector Created 4 years, 2 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CHROME_CHROME_WATCHER_CHROME_POSTMORTEM_REPORT_COLLECTOR_H_
6 #define CHROME_CHROME_WATCHER_CHROME_POSTMORTEM_REPORT_COLLECTOR_H_
7
8 #include <string>
9
10 #include "components/browser_watcher/postmortem_report_collector.h"
11
12 namespace chrome_watcher {
13
14 class ChromePostmortemReportCollector
15 : public browser_watcher::PostmortemReportCollector {
16 public:
17 ChromePostmortemReportCollector();
18 ~ChromePostmortemReportCollector() override = default;
grt (UTC plus 2) 2016/09/27 20:43:09 is this needed?
manzagop (departed) 2016/09/28 16:29:17 Ack.
19
20 const std::string& GetProductName() override;
21 const std::string& GetProductVersion() override;
22 const std::string& GetProductChannel() override;
23
24 private:
25 void InitializeVersionDetails();
26
27 bool version_details_initialized_;
28 std::string product_name_;
29 std::string version_number_;
30 std::string channel_name_;
31
32 DISALLOW_COPY_AND_ASSIGN(ChromePostmortemReportCollector);
33 };
34
35 } // namespace chrome_watcher
36
37 #endif // CHROME_CHROME_WATCHER_CHROME_POSTMORTEM_REPORT_COLLECTOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698