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

Unified Diff: chrome/browser/ui/views/payments/order_summary_view_controller.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/ui/views/payments/order_summary_view_controller.cc
diff --git a/chrome/browser/ui/views/payments/order_summary_view_controller.cc b/chrome/browser/ui/views/payments/order_summary_view_controller.cc
index 0d0933628ceedeba9c196bb575b78d620a0d64d3..cad23e8fa6b9b4b771a91cf127429bb6768b978b 100644
--- a/chrome/browser/ui/views/payments/order_summary_view_controller.cc
+++ b/chrome/browser/ui/views/payments/order_summary_view_controller.cc
@@ -11,7 +11,7 @@
#include "base/memory/ptr_util.h"
#include "base/strings/utf_string_conversions.h"
#include "chrome/browser/browser_process.h"
-#include "chrome/browser/ui/views/payments/payment_request_dialog.h"
+#include "chrome/browser/ui/views/payments/payment_request_dialog_view.h"
#include "chrome/browser/ui/views/payments/payment_request_views_util.h"
#include "chrome/grit/generated_resources.h"
#include "components/payments/currency_formatter.h"
@@ -74,7 +74,7 @@ std::unique_ptr<views::View> CreateLineItemView(const base::string16& label,
OrderSummaryViewController::OrderSummaryViewController(
PaymentRequest* request,
- PaymentRequestDialog* dialog)
+ PaymentRequestDialogView* dialog)
: PaymentRequestSheetController(request, dialog) {}
OrderSummaryViewController::~OrderSummaryViewController() {}
@@ -123,7 +123,7 @@ void OrderSummaryViewController::ButtonPressed(
views::Button* sender, const ui::Event& event) {
switch (sender->tag()) {
case static_cast<int>(PaymentRequestCommonTags::CLOSE_BUTTON_TAG):
- dialog()->CloseDialog();
+ dialog()->CloseDialog(true);
break;
case static_cast<int>(PaymentRequestCommonTags::BACK_BUTTON_TAG):
dialog()->GoBack();

Powered by Google App Engine
This is Rietveld 408576698