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

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

Issue 2969713002: [Payments] Present web payment app instead of instruments to user (Closed)
Patch Set: rebase Created 3 years, 5 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/java/src/org/chromium/chrome/browser/payments/ServiceWorkerPaymentAppBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/ServiceWorkerPaymentAppBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/ServiceWorkerPaymentAppBridge.java
index 99c8d7c52e848fe78964de651cb9fd98e0519900..42030573c2d5e7dfd4892ebbc0cc2fd310d23bd6 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/ServiceWorkerPaymentAppBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/ServiceWorkerPaymentAppBridge.java
@@ -16,9 +16,6 @@ import org.chromium.payments.mojom.PaymentDetailsModifier;
import org.chromium.payments.mojom.PaymentItem;
import org.chromium.payments.mojom.PaymentMethodData;
-import java.util.ArrayList;
-import java.util.HashSet;
-import java.util.List;
import java.util.Set;
import javax.annotation.Nullable;
@@ -62,28 +59,6 @@ public class ServiceWorkerPaymentAppBridge implements PaymentAppFactory.PaymentA
modifiers.toArray(new PaymentDetailsModifier[0]), instrumentId, callback);
}
- @CalledByNative
- private static List<PaymentInstrument> createInstrumentList() {
- return new ArrayList<PaymentInstrument>();
- }
-
- @CalledByNative
- private static void addInstrument(List<PaymentInstrument> instruments, WebContents webContents,
- long swRegistrationId, String instrumentId, String label, String[] methodNameArray,
- @Nullable Bitmap icon) {
- Context context = ChromeActivity.fromWebContents(webContents);
- if (context == null) return;
-
- Set<String> methodNames = new HashSet<String>();
- for (int i = 0; i < methodNameArray.length; i++) {
- methodNames.add(methodNameArray[i]);
- }
-
- instruments.add(new ServiceWorkerPaymentInstrument(webContents, swRegistrationId,
- instrumentId, label, methodNames,
- icon == null ? null : new BitmapDrawable(context.getResources(), icon)));
- }
-
@CalledByNative
private static String[] getSupportedMethodsFromMethodData(PaymentMethodData data) {
return data.supportedMethods;
@@ -120,11 +95,17 @@ public class ServiceWorkerPaymentAppBridge implements PaymentAppFactory.PaymentA
}
@CalledByNative
- private static void onPaymentAppCreated(
- List<PaymentInstrument> instruments, WebContents webContents, Object callback) {
+ private static void onPaymentAppCreated(long swRegistrationId, String label,
+ @Nullable Bitmap icon, String[] methodNameArray, WebContents webContents,
+ Object callback) {
assert callback instanceof PaymentAppFactory.PaymentAppCreatedCallback;
+ Context context = ChromeActivity.fromWebContents(webContents);
+ if (context == null) return;
((PaymentAppFactory.PaymentAppCreatedCallback) callback)
- .onPaymentAppCreated(new ServiceWorkerPaymentApp(webContents, instruments));
+ .onPaymentAppCreated(new ServiceWorkerPaymentApp(webContents, swRegistrationId,
+ label,
+ icon == null ? null : new BitmapDrawable(context.getResources(), icon),
+ methodNameArray));
}
@CalledByNative

Powered by Google App Engine
This is Rietveld 408576698