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

Unified Diff: chrome/browser/payments/chrome_payment_request_delegate.cc

Issue 2649683002: [Payments] Improve the closing of the PR dialog. (Closed)
Patch Set: Initial Created 3 years, 11 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/browser/payments/chrome_payment_request_delegate.cc
diff --git a/chrome/browser/payments/chrome_payment_request_delegate.cc b/chrome/browser/payments/chrome_payment_request_delegate.cc
index 5a65cb149f25e3d81d52e2607c36cd3c6d0aee15..d9afc42dd5c38c01056eb8bd738e42b2fb9343e6 100644
--- a/chrome/browser/payments/chrome_payment_request_delegate.cc
+++ b/chrome/browser/payments/chrome_payment_request_delegate.cc
@@ -7,17 +7,25 @@
#include "chrome/browser/autofill/personal_data_manager_factory.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser_dialogs.h"
+#include "components/payments/payment_request_dialog.h"
#include "content/public/browser/web_contents.h"
namespace payments {
ChromePaymentRequestDelegate::ChromePaymentRequestDelegate(
content::WebContents* web_contents)
- : web_contents_(web_contents) {}
+ : dialog_(nullptr), web_contents_(web_contents) {}
-void ChromePaymentRequestDelegate::ShowPaymentRequestDialog(
- PaymentRequest* request) {
- chrome::ShowPaymentRequestDialog(request);
+void ChromePaymentRequestDelegate::ShowDialog(PaymentRequest* request) {
+ dialog_ = chrome::CreatePaymentRequestDialog(request);
+ dialog_->ShowDialog();
+}
+
+void ChromePaymentRequestDelegate::CloseDialog() {
+ if (dialog_) {
+ // Do not send a user cancellation message back to the renderer.
+ dialog_->CloseDialog(false /* propagate_to_renderer */);
+ }
}
autofill::PersonalDataManager*

Powered by Google App Engine
This is Rietveld 408576698