Add Installer.PayloadFormat metric

Adds a new PayloadFormat metric with three different values: Delta, Full
and ForcedFull. The metric is send whenever an update is applied successfully
indicating the type of the used payload. The difference between Full and
ForcedFull is that in the ForcedFull, the request sent to omaha included a
directive saying that a delta payload wasn't accepted, where a Full payload
is one where a delta payload was accepted but a Full payload was provided.

BUG=chromium:225977
TEST=cros_workon_make update_engine --test # Three tests added, one for each condition.

Change-Id: If44cd96df325d320ed327114cc5e5de3d34a5c62
Reviewed-on: https://gerrit.chromium.org/gerrit/60318
Reviewed-by: David Zeuthen <zeuthen@chromium.org>
Commit-Queue: Alex Deymo <deymo@chromium.org>
Tested-by: Alex Deymo <deymo@chromium.org>

1 files changed, 23 insertions, 0 deletions

diff --git a/payload_state.cc b/payload_state.cc
index 20e2cacb..a933ea53 100644
--- a/payload_state.cc
+++ b/payload_state.cc
@@ -156,6 +156,7 @@ void PayloadState::UpdateSucceeded() {
   ReportRebootMetrics();
   ReportDurationMetrics();
   ReportUpdatesAbandonedCountMetric();
+  ReportPayloadTypeMetric();
 
   // Reset the number of responses seen since it counts from the last
   // successful update, e.g. now.
@@ -877,6 +878,28 @@ void PayloadState::ReportDurationMetrics() {
   prefs_->Delete(kPrefsUpdateDurationUptime);
 }
 
+void PayloadState::ReportPayloadTypeMetric() {
+  string metric;
+  PayloadType uma_payload_type;
+  OmahaRequestParams* params = system_state_->request_params();
+
+  if (response_.is_delta_payload) {
+    uma_payload_type = kPayloadTypeDelta;
+  } else if (params->delta_okay()) {
+    uma_payload_type = kPayloadTypeFull;
+  } else { // Full payload, delta was not allowed by request.
+    uma_payload_type = kPayloadTypeForcedFull;
+  }
+
+  metric = "Installer.PayloadFormat";
+  system_state_->metrics_lib()->SendEnumToUMA(
+      metric,
+      uma_payload_type,
+      kNumPayloadTypes);
+  LOG(INFO) << "Uploading " << utils::ToString(uma_payload_type)
+            << " for metric " <<  metric;
+}
+
 string PayloadState::GetPrefsKey(const string& prefix, DownloadSource source) {
   return prefix + "-from-" + utils::ToString(source);
 }