commit d4c5debc65bb35ee61c2764bda059b7f307fd093
author Tianjie Xu <xunchang@google.com> Tue Oct 24 11:17:03 2017 -0700
committer Tianjie Xu <xunchang@google.com> Wed Oct 25 11:37:04 2017 -0700
tree 379432d9dd0da75c114784a14a0530e375c741df
parent 4a7e26b998807f315c6e8b9f77f194cd3d3b12c8

Create a dummy MetricsReporterStub class

This handles the update_engine_sideload target, and thus avoiding
the insertion of "#ifdef _UE_SIDELOAD" everywhere when we report the
metrics for android.

Bug: 30989466
Test: build pass on marlin
Change-Id: Ib00ab8476218df15768bdbae1a3510f8584aa7c7

update_attempter_android.cc

diff --git a/update_attempter_android.cc b/update_attempter_android.cc
index dbddfe3..6d67000 100644
--- a/update_attempter_android.cc
+++ b/update_attempter_android.cc
@@ -34,7 +34,7 @@
 #include "update_engine/common/file_fetcher.h"
 #include "update_engine/common/utils.h"
 #include "update_engine/daemon_state_interface.h"
-#include "update_engine/metrics_reporter_android.h"
+#include "update_engine/metrics_reporter_interface.h"
 #include "update_engine/metrics_utils.h"
 #include "update_engine/network_selector.h"
 #include "update_engine/payload_consumer/download_action.h"
@@ -92,8 +92,8 @@
       boot_control_(boot_control),
       hardware_(hardware),
       processor_(new ActionProcessor()),
-      clock_(new Clock()),
-      metrics_reporter_(new MetricsReporterAndroid()) {
+      clock_(new Clock()) {
+  metrics_reporter_ = metrics::CreateMetricsReporter();
   network_selector_ = network::CreateNetworkSelector();
 }