Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3303)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/payments/JourneyLogger.java

Issue 2923243003: Merge-60 [Payments] Move the RequestedInformation metric to native. (Closed)
Patch Set: Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/payments/JourneyLogger.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/JourneyLogger.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/JourneyLogger.java
index 878f853f399cbc2842facfd84c4b0699330ca96d..7332b433e66261d392d546686092cde24a6b19e8 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/JourneyLogger.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/JourneyLogger.java
@@ -102,6 +102,20 @@ public class JourneyLogger {
nativeSetEventOccurred(mJourneyLoggerAndroid, event);
}
+ /*
+ * Records what user information were requested by the merchant to complete the Payment Request.
+ *
+ * @param requestShipping Whether the merchant requested a shipping address.
+ * @param requestEmail Whether the merchant requested an email address.
+ * @param requestPhone Whether the merchant requested a phone number.
+ * @param requestName Whether the merchant requestes a name.
+ */
+ public void setRequestedInformation(boolean requestShipping, boolean requestEmail,
+ boolean requestPhone, boolean requestName) {
+ nativeSetRequestedInformation(
+ mJourneyLoggerAndroid, requestShipping, requestEmail, requestPhone, requestName);
+ }
+
/**
* Records the payment method that was selected by the user.
*
@@ -175,6 +189,9 @@ public class JourneyLogger {
private native void nativeSetEventOccurred(long nativeJourneyLoggerAndroid, int event);
private native void nativeSetSelectedPaymentMethod(
long nativeJourneyLoggerAndroid, int paymentMethod);
+ private native void nativeSetRequestedInformation(long nativeJourneyLoggerAndroid,
+ boolean requestShipping, boolean requestEmail, boolean requestPhone,
+ boolean requestName);
private native void nativeSetCompleted(long nativeJourneyLoggerAndroid);
private native void nativeSetAborted(long nativeJourneyLoggerAndroid, int reason);
private native void nativeSetNotShown(long nativeJourneyLoggerAndroid, int reason);
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698