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

Side by Side Diff: components/browser_watcher/postmortem_report_collector.h

Issue 2883103002: Quantify instability according to the stability instrumentation (Closed)
Patch Set: final nit Created 3 years, 7 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
« no previous file with comments | « no previous file | components/browser_watcher/postmortem_report_collector.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // Following an unclean shutdown, a stability report can be collected and 5 // Following an unclean shutdown, a stability report can be collected and
6 // submitted for upload to a reporter. 6 // submitted for upload to a reporter.
7 7
8 #ifndef COMPONENTS_BROWSER_WATCHER_POSTMORTEM_REPORT_COLLECTOR_H_ 8 #ifndef COMPONENTS_BROWSER_WATCHER_POSTMORTEM_REPORT_COLLECTOR_H_
9 #define COMPONENTS_BROWSER_WATCHER_POSTMORTEM_REPORT_COLLECTOR_H_ 9 #define COMPONENTS_BROWSER_WATCHER_POSTMORTEM_REPORT_COLLECTOR_H_
10 10
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 FRIEND_TEST_ALL_PREFIXES( 80 FRIEND_TEST_ALL_PREFIXES(
81 PostmortemReportCollectorCollectionFromGlobalTrackerTest, 81 PostmortemReportCollectorCollectionFromGlobalTrackerTest,
82 SystemStateTest); 82 SystemStateTest);
83 83
84 // Virtual for unittesting. 84 // Virtual for unittesting.
85 virtual std::vector<base::FilePath> GetDebugStateFilePaths( 85 virtual std::vector<base::FilePath> GetDebugStateFilePaths(
86 const base::FilePath& debug_info_dir, 86 const base::FilePath& debug_info_dir,
87 const base::FilePath::StringType& debug_file_pattern, 87 const base::FilePath::StringType& debug_file_pattern,
88 const std::set<base::FilePath>& excluded_debug_files); 88 const std::set<base::FilePath>& excluded_debug_files);
89 89
90 CollectionStatus CollectAndSubmitOneReport( 90 // Collects a stability file, generates a report and registers it with the
91 const crashpad::UUID& client_id, 91 // database. Returns true on success. False otherwise.
92 const base::FilePath& file, 92 bool CollectAndSubmitOneReport(const crashpad::UUID& client_id,
93 crashpad::CrashReportDatabase* report_database); 93 const base::FilePath& file,
94 crashpad::CrashReportDatabase* report_database,
95 bool* system_unclean);
94 96
95 virtual CollectionStatus CollectOneReport( 97 virtual CollectionStatus CollectOneReport(
96 const base::FilePath& stability_file, 98 const base::FilePath& stability_file,
97 StabilityReport* report); 99 StabilityReport* report);
98 100
99 void SetReporterDetails(StabilityReport* report) const; 101 void SetReporterDetails(StabilityReport* report) const;
100 102
101 void RecordSystemShutdownState(StabilityReport* report) const; 103 void RecordSystemShutdownState(StabilityReport* report) const;
102 104
103 virtual bool WriteReportToMinidump(StabilityReport* report, 105 virtual bool WriteReportToMinidump(StabilityReport* report,
104 const crashpad::UUID& client_id, 106 const crashpad::UUID& client_id,
105 const crashpad::UUID& report_id, 107 const crashpad::UUID& report_id,
106 base::PlatformFile minidump_file); 108 base::PlatformFile minidump_file);
107 109
108 std::string product_name_; 110 std::string product_name_;
109 std::string version_number_; 111 std::string version_number_;
110 std::string channel_name_; 112 std::string channel_name_;
111 113
112 SystemSessionAnalyzer* system_session_analyzer_; // Not owned. 114 SystemSessionAnalyzer* system_session_analyzer_; // Not owned.
113 115
114 DISALLOW_COPY_AND_ASSIGN(PostmortemReportCollector); 116 DISALLOW_COPY_AND_ASSIGN(PostmortemReportCollector);
115 }; 117 };
116 118
117 } // namespace browser_watcher 119 } // namespace browser_watcher
118 120
119 #endif // COMPONENTS_BROWSER_WATCHER_POSTMORTEM_REPORT_COLLECTOR_H_ 121 #endif // COMPONENTS_BROWSER_WATCHER_POSTMORTEM_REPORT_COLLECTOR_H_
OLDNEW
« no previous file with comments | « no previous file | components/browser_watcher/postmortem_report_collector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698