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

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

Issue 2434333005: Introduce method data map in getInstrument (Closed)
Patch Set: Fix compilation Created 4 years, 2 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 | « chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 6428564c69e259178965164db444dad0c2c52385..6670a6749dc8f6f0192b3a2c11830e542f42f41f 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
@@ -40,6 +40,7 @@ import org.chromium.payments.mojom.PaymentItem;
import java.util.ArrayList;
import java.util.HashSet;
import java.util.List;
+import java.util.Map;
import java.util.Set;
import java.util.concurrent.Callable;
import java.util.concurrent.ExecutionException;
@@ -677,7 +678,7 @@ abstract class PaymentRequestTestBase extends ChromeActivityTestCaseBase<ChromeT
@Override
public void getInstruments(
- JSONObject details, final InstrumentsCallback instrumentsCallback) {
+ Map<String, JSONObject> methodData, final InstrumentsCallback instrumentsCallback) {
mCallback = instrumentsCallback;
respond();
}
@@ -702,14 +703,14 @@ abstract class PaymentRequestTestBase extends ChromeActivityTestCaseBase<ChromeT
}
@Override
- public Set<String> getSupportedMethodNames() {
+ public Set<String> getAppMethodNames() {
Set<String> methodNames = new HashSet<>();
methodNames.add(mMethodName);
return methodNames;
}
@Override
- public String getIdentifier() {
+ public String getAppIdentifier() {
return mMethodName;
}
}
@@ -724,18 +725,19 @@ abstract class PaymentRequestTestBase extends ChromeActivityTestCaseBase<ChromeT
}
@Override
- public String getMethodName() {
+ public String getInstrumentMethodName() {
return mMethodName;
}
@Override
- public void getDetails(String merchantName, String origin, PaymentItem total,
- List<PaymentItem> cart, JSONObject details, DetailsCallback detailsCallback) {
+ public void getInstrumentDetails(String merchantName, String origin, PaymentItem total,
+ List<PaymentItem> cart, JSONObject details,
+ InstrumentDetailsCallback detailsCallback) {
detailsCallback.onInstrumentDetailsReady(
mMethodName, "{\"transaction\": 1337}");
}
@Override
- public void dismiss() {}
+ public void dismissInstrument() {}
}
}
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698