Stability instrumentation: metrics for collection on crash

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 <utility>
 
 #include "base/files/file_enumerator.h"
 #include "base/files/file_util.h"
@@ skipping 196 matching lines @@
   call_error_writing_crash_report.Disarm();
   crashpad::UUID unused_report_id;
   database_status = report_database->FinishedWritingCrashReport(
       new_report, &unused_report_id);
   if (database_status != CrashReportDatabase::kNoError) {
     DLOG(ERROR) << "FinishedWritingCrashReport failed";
     LogCollectionStatus(FINISHED_WRITING_CRASH_REPORT_FAILED);
     return false;
   }
 
-  // Report collection is successful. We may increment |unclean_system_cnt|.
+  LogCollectionStatus(SUCCESS);
   if (report_proto.system_state().session_state() == SystemState::UNCLEAN)
     *system_unclean = true;
-
   return true;
 }
 
 CollectionStatus PostmortemReportCollector::CollectOneReport(
     const base::FilePath& file,
     StabilityReport* report) {
   DCHECK(report);
 
   CollectionStatus status = Extract(file, report);
   if (status != SUCCESS)
@@ skipping 71 matching lines @@
     StabilityReport* report,
     const crashpad::UUID& client_id,
     const crashpad::UUID& report_id,
     base::PlatformFile minidump_file) {
   DCHECK(report);
 
   return WritePostmortemDump(minidump_file, client_id, report_id, report);
 }
 
 }  // namespace browser_watcher