commit 52c678cddc5dd7bda8a29e74ce6ef4d67e8d1487
author Tianjie Xu <xunchang@google.com> Wed Oct 18 15:52:27 2017 -0700
committer Tianjie Xu <xunchang@google.com> Mon Oct 23 18:26:47 2017 +0000
tree c2e598ddaded7929c409c38ba6eb79bfaa59f400
parent 5c9082dd722318d736158a90ff85a9d57bc0ffe5

Report more android update metrics in update engine

Report more metrics with android::metricslogger.
Metrics include:
  ota_update_engine_attempt_number
  ota_update_engine_attempt_result
  ota_update_engine_attempt_duration_in_minutes
  ota_update_engine_attempt_duration_uptime_in_minutes
  ota_update_engine_attempt_payload_size_mib
  ota_update_engine_attempt_payload_type

  ota_update_engine_successful_update_attempt_count
  ota_update_engine_successful_update_total_duration_in_minutes
  ota_update_engine_successful_update_payload_size_mib
  ota_update_engine_successful_update_payload_type
  ota_update_engine_successful_update_reboot_count

And the google3 CL in cr/review/172829231

Bug: 30989466
Test: compile update_engine for marlin
Change-Id: Ic0ead07d1bd4a575244eb5e05588b47ba7b0bafe

metrics_reporter_android.h

diff --git a/metrics_reporter_android.h b/metrics_reporter_android.h
index b83e651..373730b 100644
--- a/metrics_reporter_android.h
+++ b/metrics_reporter_android.h
@@ -57,7 +57,7 @@
       metrics::DownloadErrorCode payload_download_error_code,
       metrics::ConnectionType connection_type) override {}
 
-  void ReportAbnormallyTerminatedUpdateAttemptMetrics() override {}
+  void ReportAbnormallyTerminatedUpdateAttemptMetrics() override;
 
   void ReportSuccessfulUpdateMetrics(
       int attempt_count,
@@ -68,7 +68,7 @@
       int download_overhead_percentage,
       base::TimeDelta total_duration,
       int reboot_count,
-      int url_switch_count) override {}
+      int url_switch_count) override;
 
   void ReportCertificateCheckMetrics(ServerToCheck server_to_check,
                                      CertificateCheckResult result) override {}