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

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

Issue 2692023002: Make PaymentRequestImpl work with RenderFrameHost (Closed)
Patch Set: Address review comment 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/PaymentRequestFactory.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestFactory.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestFactory.java
index 17f19dff9379f5d98dfedb603b00b0260f013a86..dd84cca6892773ba62f5f840c5213a0bb5496441 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestFactory.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestFactory.java
@@ -5,7 +5,7 @@
package org.chromium.chrome.browser.payments;
import org.chromium.chrome.browser.ChromeFeatureList;
-import org.chromium.content_public.browser.WebContents;
+import org.chromium.content_public.browser.RenderFrameHost;
import org.chromium.mojo.system.MojoException;
import org.chromium.payments.mojom.CanMakePaymentQueryResult;
import org.chromium.payments.mojom.PaymentDetails;
@@ -20,7 +20,7 @@ import org.chromium.services.service_manager.InterfaceFactory;
* Creates instances of PaymentRequest.
*/
public class PaymentRequestFactory implements InterfaceFactory<PaymentRequest> {
- private final WebContents mWebContents;
+ private final RenderFrameHost mRenderFrameHost;
/**
* An implementation of PaymentRequest that immediately rejects all connections.
@@ -71,8 +71,8 @@ public class PaymentRequestFactory implements InterfaceFactory<PaymentRequest> {
*
* @param webContents The web contents that may invoke the PaymentRequest API.
*/
- public PaymentRequestFactory(WebContents webContents) {
- mWebContents = webContents;
+ public PaymentRequestFactory(RenderFrameHost renderFrameHost) {
+ mRenderFrameHost = renderFrameHost;
}
@Override
@@ -81,8 +81,8 @@ public class PaymentRequestFactory implements InterfaceFactory<PaymentRequest> {
return new InvalidPaymentRequest();
}
- if (mWebContents == null) return new InvalidPaymentRequest();
+ if (mRenderFrameHost == null) return new InvalidPaymentRequest();
- return new PaymentRequestImpl(mWebContents);
+ return new PaymentRequestImpl(mRenderFrameHost);
}
}

Powered by Google App Engine
This is Rietveld 408576698