[Merge-57] Add completion metrics based on user had suggestions for all.

chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestJourneyLogger.java

 // 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.
 
 package org.chromium.chrome.browser.payments;
 
 import org.chromium.base.metrics.RecordHistogram;
 
 /**
  * A class used to record journey metrics for the Payment Request feature.
  */
 public class PaymentRequestJourneyLogger {
     // The index of the different sections of a Payment Request. Used to record journey stats.
     public static final int SECTION_CONTACT_INFO = 0;
     public static final int SECTION_CREDIT_CARDS = 1;
     public static final int SECTION_SHIPPING_ADDRESS = 2;
     public static final int SECTION_MAX = 3;
 
     // For the CanMakePayment histograms.
     public static final int CAN_MAKE_PAYMENT_USED = 0;
     public static final int CAN_MAKE_PAYMENT_NOT_USED = 1;
     public static final int CAN_MAKE_PAYMENT_USE_MAX = 2;
 
-    // Used to log CanMakePayment's effect on whether the transaction was completed.
-    public static final int CMP_EFFECT_ON_COMPLETION_COMPLETED = 0;
-    public static final int CMP_EFFECT_ON_COMPLETION_ABORTED = 1;
-    public static final int CMP_EFFECT_ON_COMPLETION_MAX = 2;
+    // Used to log different parameters' effect on whether the transaction was completed.
+    public static final int COMPLETION_STATUS_COMPLETED = 0;
+    public static final int COMPLETION_STATUS_ABORTED = 1;
+    public static final int COMPLETION_STATUS_MAX = 2;
 
     // Used to mesure the impact of the CanMakePayment return value on whether the Payment Request
     // is shown to the user.
     public static final int CMP_SHOW_COULD_NOT_MAKE_PAYMENT_AND_DID_NOT_SHOW = 0;
     public static final int CMP_SHOW_DID_SHOW = 1 << 0;
     public static final int CMP_SHOW_COULD_MAKE_PAYMENT = 1 << 1;
     public static final int CMP_SHOW_MAX = 4;
 
     // The minimum expected value of CustomCountHistograms is always set to 1. It is still possible
     // to log the value 0 to that type of histogram.
@@ skipping 86 matching lines @@
      * usage of the CanMakePayment method and its effect on the transaction. This method should be
      * called when the payment request has either been completed or aborted.
      *
      * @param submissionType A string indicating the way the payment request was concluded.
      */
     public void recordJourneyStatsHistograms(String submissionType) {
         recordSectionSpecificStats(submissionType);
 
         // Record the CanMakePayment metrics based on whether the transaction was completed or
         // aborted by the user (UserAborted) or otherwise (OtherAborted).
-        recordCanMakePaymentStats(submissionType.contains("Abort")
-                        ? CMP_EFFECT_ON_COMPLETION_ABORTED
-                        : CMP_EFFECT_ON_COMPLETION_COMPLETED);
+        recordCanMakePaymentStats(submissionType.contains("Abort") ? COMPLETION_STATUS_ABORTED
+                                                                   : COMPLETION_STATUS_COMPLETED);
     }
 
     /**
      * Records the histograms for all the sections that were requested by the merchant.
      *
      * @param submissionType A string indicating the way the payment request was concluded.
      */
     private void recordSectionSpecificStats(String submissionType) {
+        // Record whether the user had suggestions for each requested information.
+        boolean userHadAllRequestedInformation = true;
+
         for (int i = 0; i < mSections.length; ++i) {
             String nameSuffix = "";
             switch (i) {
                 case SECTION_SHIPPING_ADDRESS:
                     nameSuffix = "ShippingAddress." + submissionType;
                     break;
                 case SECTION_CONTACT_INFO:
                     nameSuffix = "ContactInfo." + submissionType;
                     break;
                 case SECTION_CREDIT_CARDS:
@@ skipping 16 matching lines @@
                         Math.min(mSections[i].mNumberSelectionChanges, MAX_EXPECTED_SAMPLE),
                         MIN_EXPECTED_SAMPLE, MAX_EXPECTED_SAMPLE, NUMBER_BUCKETS);
                 RecordHistogram.recordCustomCountHistogram(
                         "PaymentRequest.NumberOfSelectionEdits." + nameSuffix,
                         Math.min(mSections[i].mNumberSelectionEdits, MAX_EXPECTED_SAMPLE),
                         MIN_EXPECTED_SAMPLE, MAX_EXPECTED_SAMPLE, NUMBER_BUCKETS);
                 RecordHistogram.recordCustomCountHistogram(
                         "PaymentRequest.NumberOfSuggestionsShown." + nameSuffix,
                         Math.min(mSections[i].mNumberSuggestionsShown, MAX_EXPECTED_SAMPLE),
                         MIN_EXPECTED_SAMPLE, MAX_EXPECTED_SAMPLE, NUMBER_BUCKETS);
+
+                if (mSections[i].mNumberSuggestionsShown == 0) {
+                    userHadAllRequestedInformation = false;
+                }
             }
         }
+
+        // Record metrics about completion based on whether the user had suggestions for each
+        // requested information.
+        int completionStatus = submissionType.contains("Abort") ? COMPLETION_STATUS_ABORTED
+                                                                : COMPLETION_STATUS_COMPLETED;
+        if (userHadAllRequestedInformation) {
+            RecordHistogram.recordEnumeratedHistogram(
+                    "PaymentRequest.UserHadSuggestionsForEverything.EffectOnCompletion",
+                    completionStatus, COMPLETION_STATUS_MAX);
+        } else {
+            RecordHistogram.recordEnumeratedHistogram(
+                    "PaymentRequest.UserDidNotHaveSuggestionsForEverything.EffectOnCompletion",
+                    completionStatus, COMPLETION_STATUS_MAX);
+        }
     }
 
     /**
      * Records the metrics related the the CanMakePayment method.
      *
      * @param completionStatus Whether the transaction was completed or aborted.
      */
     private void recordCanMakePaymentStats(int completionStatus) {
         // Record CanMakePayment usage.
         RecordHistogram.recordEnumeratedHistogram("PaymentRequest.CanMakePayment.Usage",
@@ skipping 31 matching lines @@
 
         if (!mWasCanMakePaymentUsed) {
             histogramName += "NotUsed.WithShowEffectOnCompletion";
         } else if (mCouldMakePayment) {
             histogramName += "Used.TrueWithShowEffectOnCompletion";
         } else {
             histogramName += "Used.FalseWithShowEffectOnCompletion";
         }
 
         RecordHistogram.recordEnumeratedHistogram(
-                histogramName, completionStatus, CMP_EFFECT_ON_COMPLETION_MAX);
+                histogramName, completionStatus, COMPLETION_STATUS_MAX);
     }
 }