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

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

Issue 2436883002: Make PaymentOption store a Drawable instead of id (Closed)
Patch Set: Fix presubmit warning 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
Index: chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
index 44bc75d4b1ef2bfa34541677372ae808c686100a..f8adf42d27b1028d20d8ec4266aafae20ae8f92b 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
@@ -234,7 +234,7 @@ public class PaymentRequestImpl implements PaymentRequest, PaymentRequestUI.Clie
}
});
- mApps = PaymentAppFactory.create(webContents);
+ mApps = PaymentAppFactory.create(mContext, webContents);
mAddressEditor = new AddressEditor();
mCardEditor = new CardEditor(webContents, mAddressEditor, sObserverForTest);
@@ -654,7 +654,7 @@ public class PaymentRequestImpl implements PaymentRequest, PaymentRequestUI.Clie
for (int i = 0; i < options.length; i++) {
PaymentShippingOption option = options[i];
result.add(new PaymentOption(option.id, option.label,
- formatter.format(option.amount.value), PaymentOption.NO_ICON));
+ formatter.format(option.amount.value), null));
if (option.selected) selectedItemIndex = i;
}

Powered by Google App Engine
This is Rietveld 408576698