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

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

Issue 2899313002: [Payments] Move the logging for checkout flow to native. (Closed)
Patch Set: Addressed comments Created 3 years, 7 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
Index: chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
index d7a9fb9e85409f665655ce009c9e3fb70d6e9516..b9a00f31feb57814f61e8b9257c727c9ed50e4f0 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
@@ -129,7 +129,7 @@ public class PaymentRequestMetricsTest implements MainActivityStartCallback {
mPaymentRequestTestRule.expectResultContains(new String[] {"Request cancelled"});
mPaymentRequestTestRule.assertOnlySpecificAbortMetricLogged(
- PaymentRequestMetrics.ABORT_REASON_ABORTED_BY_USER);
+ JourneyLogger.ABORT_REASON_ABORTED_BY_USER);
}
/**
@@ -151,7 +151,7 @@ public class PaymentRequestMetricsTest implements MainActivityStartCallback {
mPaymentRequestTestRule.expectResultContains(new String[] {"Request cancelled"});
mPaymentRequestTestRule.assertOnlySpecificAbortMetricLogged(
- PaymentRequestMetrics.ABORT_REASON_ABORTED_BY_USER);
+ JourneyLogger.ABORT_REASON_ABORTED_BY_USER);
}
/**
@@ -179,7 +179,7 @@ public class PaymentRequestMetricsTest implements MainActivityStartCallback {
mPaymentRequestTestRule.expectResultContains(new String[] {"Request cancelled"});
mPaymentRequestTestRule.assertOnlySpecificAbortMetricLogged(
- PaymentRequestMetrics.ABORT_REASON_ABORTED_BY_USER);
+ JourneyLogger.ABORT_REASON_ABORTED_BY_USER);
}
/**
@@ -200,7 +200,7 @@ public class PaymentRequestMetricsTest implements MainActivityStartCallback {
mPaymentRequestTestRule.getActivity());
mPaymentRequestTestRule.assertOnlySpecificAbortMetricLogged(
- PaymentRequestMetrics.ABORT_REASON_MOJO_RENDERER_CLOSING);
+ JourneyLogger.ABORT_REASON_MOJO_RENDERER_CLOSING);
}
/**
@@ -219,7 +219,7 @@ public class PaymentRequestMetricsTest implements MainActivityStartCallback {
mPaymentRequestTestRule.expectResultContains(new String[] {"Abort"});
mPaymentRequestTestRule.assertOnlySpecificAbortMetricLogged(
- PaymentRequestMetrics.ABORT_REASON_ABORTED_BY_MERCHANT);
+ JourneyLogger.ABORT_REASON_ABORTED_BY_MERCHANT);
}
/**
@@ -248,7 +248,7 @@ public class PaymentRequestMetricsTest implements MainActivityStartCallback {
Assert.assertEquals(1,
RecordHistogram.getHistogramValueCountForTesting(
"PaymentRequest.CheckoutFunnel.NoShow",
- PaymentRequestMetrics.NO_SHOW_NO_MATCHING_PAYMENT_METHOD));
+ JourneyLogger.NO_SHOW_NO_MATCHING_PAYMENT_METHOD));
}
/**
@@ -273,7 +273,7 @@ public class PaymentRequestMetricsTest implements MainActivityStartCallback {
Assert.assertEquals(1,
RecordHistogram.getHistogramValueCountForTesting(
"PaymentRequest.CheckoutFunnel.NoShow",
- PaymentRequestMetrics.NO_SHOW_NO_SUPPORTED_PAYMENT_METHOD));
+ JourneyLogger.NO_SHOW_NO_SUPPORTED_PAYMENT_METHOD));
}
/**

Powered by Google App Engine
This is Rietveld 408576698