Quantify instability according to the stability instrumentation

components/browser_watcher/postmortem_report_collector.h

Index: components/browser_watcher/postmortem_report_collector.h
diff --git a/components/browser_watcher/postmortem_report_collector.h b/components/browser_watcher/postmortem_report_collector.h
index 08e4b9ba17d9d441cc56ab966084b55963c0fcf6..ec94fd8f8ac5451887a926b0d3f444e3b7b656ff 100644
--- a/components/browser_watcher/postmortem_report_collector.h
+++ b/components/browser_watcher/postmortem_report_collector.h
@@ -87,10 +87,12 @@ class PostmortemReportCollector {
       const base::FilePath::StringType& debug_file_pattern,
       const std::set<base::FilePath>& excluded_debug_files);
 
-  CollectionStatus CollectAndSubmitOneReport(
-      const crashpad::UUID& client_id,
-      const base::FilePath& file,
-      crashpad::CrashReportDatabase* report_database);
+  // Collects a stability file, generates a report and registers it with the
+  // database. Returns true on success. False otherwise.
+  bool CollectAndSubmitOneReport(const crashpad::UUID& client_id,
+                                 const base::FilePath& file,
+                                 crashpad::CrashReportDatabase* report_database,
+                                 int* unclean_system_cnt);

[Sigurður Ásgeirsson, 2017/05/18 15:10:04]

sorry - what I meant that instead of passing an int* here, you can pass a bool*.
As the count is incremented by zero or one, it's IMHO clearer to return a bool
in the *OneReport function and have the counting done in the *AllReports
function.

[manzagop (departed), 2017/05/18 16:02:49]

Oh, my bad! Done. Though I left the previous change.

 
   virtual CollectionStatus CollectOneReport(
       const base::FilePath& stability_file,