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

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

Issue 2199813003: [Payments] Add metrics for Payment Request aborts. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@metrics
Patch Set: Rebase Created 4 years, 4 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/PaymentRequestTestBase.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestTestBase.java b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestTestBase.java
index 6f6187f6fd0717609eee624f852f89ef89a0f99e..242b7b4daf6cb449a0a26a38ad2ed3e55bfcbd2a 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestTestBase.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestTestBase.java
@@ -14,7 +14,7 @@ import android.widget.Spinner;
import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.UrlUtils;
import org.chromium.chrome.R;
-import org.chromium.chrome.browser.ChromeActivity;
+import org.chromium.chrome.browser.ChromeTabbedActivity;
import org.chromium.chrome.browser.autofill.CardUnmaskPrompt;
import org.chromium.chrome.browser.autofill.CardUnmaskPrompt.CardUnmaskObserverForTest;
import org.chromium.chrome.browser.payments.PaymentAppFactory.PaymentAppFactoryAddition;
@@ -46,7 +46,7 @@ import java.util.concurrent.atomic.AtomicReference;
/**
* A base integration test for payments.
*/
-abstract class PaymentRequestTestBase extends ChromeActivityTestCaseBase<ChromeActivity>
+abstract class PaymentRequestTestBase extends ChromeActivityTestCaseBase<ChromeTabbedActivity>
implements PaymentRequestObserverForTest, PaymentRequestServiceObserverForTest,
CardUnmaskObserverForTest {
/** Flag for installing a payment app without instruments. */
@@ -82,7 +82,7 @@ abstract class PaymentRequestTestBase extends ChromeActivityTestCaseBase<ChromeA
private CardUnmaskPrompt mCardUnmaskPrompt;
protected PaymentRequestTestBase(String testFileName) {
- super(ChromeActivity.class);
+ super(ChromeTabbedActivity.class);
mReadyForInput = new PaymentsCallbackHelper<>();
mReadyToPay = new PaymentsCallbackHelper<>();
mSelectionChecked = new PaymentsCallbackHelper<>();
@@ -110,12 +110,17 @@ abstract class PaymentRequestTestBase extends ChromeActivityTestCaseBase<ChromeA
protected void triggerUIAndWait(PaymentsCallbackHelper<PaymentRequestUI> helper)
throws InterruptedException, ExecutionException, TimeoutException {
- triggerUIAndWait((CallbackHelper) helper);
+ triggerUIAndWait("buy", (CallbackHelper) helper);
mUI = helper.getTarget();
}
protected void triggerUIAndWait(CallbackHelper helper)
throws InterruptedException, ExecutionException, TimeoutException {
+ triggerUIAndWait("buy", helper);
+ }
+
+ protected void triggerUIAndWait(String nodeId, CallbackHelper helper)
+ throws InterruptedException, ExecutionException, TimeoutException {
startMainActivityWithURL(mTestFilePath);
onMainActivityStarted();
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@@ -129,7 +134,7 @@ abstract class PaymentRequestTestBase extends ChromeActivityTestCaseBase<ChromeA
}
});
assertWaitForPageScaleFactorMatch(1);
- clickNodeAndWait("buy", helper);
+ clickNodeAndWait(nodeId, helper);
}
/** Clicks on an HTML node. */

Powered by Google App Engine
This is Rietveld 408576698