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

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

Issue 2739033004: Send origin of the iframe browsing context (Closed)
Patch Set: Address review comments Created 3 years, 9 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/AndroidPaymentApp.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java
index f7658449477bea2799a134638a6ab16c33d4bb94..7b6be68591cb0067ea5eb823c2555154344dd728 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/AndroidPaymentApp.java
@@ -45,6 +45,7 @@ public class AndroidPaymentApp extends PaymentInstrument implements PaymentApp,
private static final String EXTRA_METHOD_NAME = "methodName";
private static final String EXTRA_DATA = "data";
private static final String EXTRA_ORIGIN = "origin";
+ private static final String EXTRA_IFRAME_ORIGIN = "iframeOrigin";
private static final String EXTRA_DETAILS = "details";
private static final String EXTRA_INSTRUMENT_DETAILS = "instrumentDetails";
private static final String EXTRA_CERTIFICATE_CHAIN = "certificateChain";
@@ -123,7 +124,7 @@ public class AndroidPaymentApp extends PaymentInstrument implements PaymentApp,
@Override
public void getInstruments(Map<String, PaymentMethodData> methodData, String origin,
- byte[][] certificateChain, InstrumentsCallback callback) {
+ String iframeOrigin, byte[][] certificateChain, InstrumentsCallback callback) {
assert mInstrumentsCallback == null
: "Have not responded to previous request for instruments yet";
mInstrumentsCallback = callback;
@@ -134,6 +135,7 @@ public class AndroidPaymentApp extends PaymentInstrument implements PaymentApp,
Bundle extras = new Bundle();
extras.putString(EXTRA_METHOD_NAME, mMethodNames.iterator().next());
extras.putString(EXTRA_ORIGIN, origin);
+ extras.putString(EXTRA_IFRAME_ORIGIN, iframeOrigin);
PaymentMethodData data = methodData.get(mMethodNames.iterator().next());
extras.putString(EXTRA_DATA, data == null ? EMPTY_JSON_DATA : data.stringifiedData);
addCertificateChain(extras, certificateChain);
@@ -219,13 +221,14 @@ public class AndroidPaymentApp extends PaymentInstrument implements PaymentApp,
}
@Override
- public void invokePaymentApp(String merchantName, String origin, byte[][] certificateChain,
- Map<String, PaymentMethodData> methodDataMap, PaymentItem total,
- List<PaymentItem> displayItems, Map<String, PaymentDetailsModifier> modifiers,
- InstrumentDetailsCallback callback) {
+ public void invokePaymentApp(String merchantName, String origin, String iframeOrigin,
+ byte[][] certificateChain, Map<String, PaymentMethodData> methodDataMap,
+ PaymentItem total, List<PaymentItem> displayItems,
+ Map<String, PaymentDetailsModifier> modifiers, InstrumentDetailsCallback callback) {
assert !mMethodNames.isEmpty();
Bundle extras = new Bundle();
extras.putString(EXTRA_ORIGIN, origin);
+ extras.putString(EXTRA_IFRAME_ORIGIN, iframeOrigin);
addCertificateChain(extras, certificateChain);
String methodName = mMethodNames.iterator().next();

Powered by Google App Engine
This is Rietveld 408576698