[Payments] Move journey logger to native.

components/payments/content/android/java/src/org/chromium/components/payments/JourneyLogger.java

+// Copyright 2016 The Chromium Authors. All rights reserved.

[Mathieu, 2017/03/23 21:19:48]

would have been great to "git mv" the PaymentRequestJourneyLogger to
JourneyLogger to keep the history of the previous file. Is it too late?

[sebsg, 2017/03/24 13:35:29]

Hum, I actually used git mv... Maybe because I changed the name at the same
time?

All the logic has moved out to C++ but I can try if you want. 

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+package org.chromium.components.payments;
+
+import org.chromium.base.annotations.JNINamespace;
+
+/**
+ * A class used to record journey metrics for the Payment Request feature.
+ */
+@JNINamespace("payments")
+public class JourneyLogger {
+    // The index of the different sections of a Payment Request. Used to record journey stats.

[Mathieu, 2017/03/23 21:19:48]

if we are going to mirror these constants to C++, let's add a link that they
should stay in sync to both sides

[sebsg, 2017/03/24 13:35:29]

Done.

+    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 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.
+    private static final int MIN_EXPECTED_SAMPLE = 1;
+    private static final int MAX_EXPECTED_SAMPLE = 49;
+    private static final int NUMBER_BUCKETS = 50;
+
+    /**
+     * Pointer to the native implementation.
+     */
+    private long mJourneyLoggerAndroid;
+
+    public JourneyLogger() {
+        // Note that this pointer could leak the native object. The called must call destroy() to
+        // ensure that the native object is destroyed.
+        mJourneyLoggerAndroid = nativeInitJourneyLoggerAndroid();
+    }
+
+    /** Will destroy the native object. This class shouldn't be used afterwards. */
+    public void destroy() {
+        if (mJourneyLoggerAndroid != 0) {
+            nativeDestroy(mJourneyLoggerAndroid);
+            mJourneyLoggerAndroid = 0;
+        }
+    }
+
+    /**
+     * Sets the number of suggestions shown for the specified section.
+     *
+     * @param section The section for which to log.
+     * @param number The number of suggestions.
+     */
+    public void setNumberOfSuggestionsShown(int section, int number) {
+        assert section < SECTION_MAX;
+        nativeSetNumberOfSuggestionsShown(mJourneyLoggerAndroid, section, number);
+    }
+
+    /**
+     * Increments the number of selection changes for the specified section.
+     *
+     * @param section The section for which to log.
+     */
+    public void incrementSelectionChanges(int section) {
+        assert section < SECTION_MAX;
+        nativeIncrementSelectionChanges(mJourneyLoggerAndroid, section);
+    }
+
+    /**
+     * Increments the number of selection edits for the specified section.
+     *
+     * @param section The section for which to log.
+     */
+    public void incrementSelectionEdits(int section) {
+        assert section < SECTION_MAX;
+        nativeIncrementSelectionEdits(mJourneyLoggerAndroid, section);
+    }
+
+    /**
+     * Increments the number of selection adds for the specified section.
+     *
+     * @param section The section for which to log.
+     */
+    public void incrementSelectionAdds(int section) {
+        assert section < SECTION_MAX;
+        nativeIncrementSelectionAdds(mJourneyLoggerAndroid, section);
+    }
+
+    /**
+     * Records the fact that the merchant called CanMakePayment and records it's return value.
+     *
+     * @param value The return value of the CanMakePayment call.
+     */
+    public void setCanMakePaymentValue(boolean value) {
+        nativeSetCanMakePaymentValue(mJourneyLoggerAndroid, value);
+    }
+
+    /**
+     * Records the fact that the Payment Request was shown to the user.
+     */
+    public void setShowCalled() {
+        nativeSetShowCalled(mJourneyLoggerAndroid);
+    }
+
+    /*
+     * Records the histograms for all the sections that were requested by the merchant and for the
+     * 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) {
+        nativeRecordJourneyStatsHistograms(mJourneyLoggerAndroid, submissionType);
+    }
+
+    private native long nativeInitJourneyLoggerAndroid();
+    private native void nativeDestroy(long nativeJourneyLoggerAndroid);
+    private native void nativeSetNumberOfSuggestionsShown(
+            long nativeJourneyLoggerAndroid, int section, int number);
+    private native void nativeIncrementSelectionChanges(
+            long nativeJourneyLoggerAndroid, int section);
+    private native void nativeIncrementSelectionEdits(long nativeJourneyLoggerAndroid, int section);
+    private native void nativeIncrementSelectionAdds(long nativeJourneyLoggerAndroid, int section);
+    private native void nativeSetCanMakePaymentValue(
+            long nativeJourneyLoggerAndroid, boolean value);
+    private native void nativeSetShowCalled(long nativeJourneyLoggerAndroid);
+    private native void nativeRecordJourneyStatsHistograms(
+            long nativeJourneyLoggerAndroid, String submissionType);
+}