[Payments] Add PaymentRequest checkout funnel UKMs.

components/payments/core/journey_logger.cc

 // Copyright 2017 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/payments/core/journey_logger.h"
 
 #include <algorithm>
 
 #include "base/metrics/histogram_functions.h"
 #include "base/metrics/histogram_macros.h"
+#include "components/autofill/core/browser/autofill_experiments.h"
+#include "components/ukm/ukm_entry_builder.h"
+#include "components/ukm/ukm_service.h"
+
+namespace internal {
+// Name constants are exposed here so they can be referenced from tests.

[Mathieu, 2017/04/10 01:03:01]

remove

[sebsg, 2017/04/10 15:30:28]

Done.

+extern const char kUKMCheckoutFunnelEntryName[] =
+    "PaymentRequest.CheckoutFunnel";
+extern const char kUKMCompletionStatusMetricName[] = "CompletionStatus";
+extern const char kUKMLastStepMetricName[] = "LastStep";
+}  // namespace internal
 
 namespace payments {
 
 namespace {
 
 // Returns the JourneyLogger histograms name suffix based on the |section| and
 // the |completion_status|.
 std::string GetHistogramNameSuffix(
     int section,
     JourneyLogger::CompletionStatus completion_status) {
@@ skipping 26 matching lines @@
     default:
       break;
   }
 
   DCHECK(!name_suffix.empty());
   return name_suffix;
 }
 
 }  // namespace
 
-JourneyLogger::JourneyLogger(bool is_incognito)
+JourneyLogger::JourneyLogger(bool is_incognito,
+                             GURL url,
+                             ukm::UkmService* ukm_service)
     : was_can_make_payments_used_(false),
       could_make_payment_(false),
       was_show_called_(false),
-      is_incognito_(is_incognito) {}
+      is_incognito_(is_incognito),
+      current_step_(STEP_INITIATED),
+      url_(url),
+      ukm_service_(ukm_service) {}
 
 JourneyLogger::~JourneyLogger() {}
 
 void JourneyLogger::IncrementSelectionAdds(Section section) {
   DCHECK_LT(section, SECTION_MAX);
   sections_[section].number_selection_adds_++;
 }
 
 void JourneyLogger::IncrementSelectionChanges(Section section) {
   DCHECK_LT(section, SECTION_MAX);
@@ skipping 13 matching lines @@
 
 void JourneyLogger::SetCanMakePaymentValue(bool value) {
   was_can_make_payments_used_ = true;
   could_make_payment_ |= value;
 }
 
 void JourneyLogger::SetShowCalled() {
   was_show_called_ = true;
 }
 
+void JourneyLogger::SetCurrentStep(CompletionStep step) {
+  current_step_ = step;

[Mathieu, 2017/04/10 01:03:01]

Should we dcheck that step > current_step_?

[sebsg, 2017/04/10 15:30:28]

Changed to bitfield as per you other suggestion

+}
+
 void JourneyLogger::RecordJourneyStatsHistograms(
     CompletionStatus completion_status) {
   RecordSectionSpecificStats(completion_status);
 
   // Record the CanMakePayment metrics based on whether the transaction was
   // completed or aborted by the user (UserAborted) or otherwise (OtherAborted).
   RecordCanMakePaymentStats(completion_status);
+
+  RecordUkms(completion_status);
 }
 
 void JourneyLogger::RecordSectionSpecificStats(
     CompletionStatus completion_status) {
   // Record whether the user had suggestions for each requested information.
   bool user_had_all_requested_information = true;
 
   for (int i = 0; i < NUMBER_OF_SECTIONS; ++i) {
     std::string name_suffix = GetHistogramNameSuffix(i, completion_status);
 
@@ skipping 80 matching lines @@
   } else if (could_make_payment_) {
     histogram_name += "Used.TrueWithShowEffectOnCompletion";
   } else {
     histogram_name += "Used.FalseWithShowEffectOnCompletion";
   }
 
   base::UmaHistogramEnumeration(histogram_name, completion_status,
                                 COMPLETION_STATUS_MAX);
 }
 
+void JourneyLogger::RecordUkms(CompletionStatus completion_status) {
+  if (!autofill::IsUkmLoggingEnabled() || !ukm_service_ || !url_.is_valid())
+    return;
+
+  // Record the Checkout Funnel UKM.
+  int32_t source_id = ukm_service_->GetNewSourceID();
+  ukm_service_->UpdateSourceURL(source_id, url_);
+  std::unique_ptr<ukm::UkmEntryBuilder> builder = ukm_service_->GetEntryBuilder(
+      source_id, internal::kUKMCheckoutFunnelEntryName);
+  builder->AddMetric(internal::kUKMCompletionStatusMetricName,
+                     completion_status);
+  builder->AddMetric(internal::kUKMLastStepMetricName, current_step_);
+}
+
 }  // namespace payments