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

Unified Diff: chrome/browser/ui/views/payments/payment_request_dialog_view.h

Issue 2810193003: [Payments] Change how the dialog gets destroyed. (Closed)
Patch Set: added comment Created 3 years, 8 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/payment_request_dialog_view.h
diff --git a/chrome/browser/ui/views/payments/payment_request_dialog_view.h b/chrome/browser/ui/views/payments/payment_request_dialog_view.h
index b79c877ad70055ec11a128cafb8214193119b1ba..cfacf1d209f38a258fa216e022c52196c894dd6d 100644
--- a/chrome/browser/ui/views/payments/payment_request_dialog_view.h
+++ b/chrome/browser/ui/views/payments/payment_request_dialog_view.h
@@ -114,7 +114,7 @@ class PaymentRequestDialogView : public views::DialogDelegateView,
// a way of closing the dialog.
void ShowProcessingSpinner();
- ViewStack* view_stack_for_testing() { return &view_stack_; }
+ ViewStack* view_stack_for_testing() { return view_stack_.get(); }
private:
void ShowInitialPaymentSheet();
@@ -131,7 +131,7 @@ class PaymentRequestDialogView : public views::DialogDelegateView,
// between the two.
PaymentRequest* request_;
ControllerMap controller_map_;
- ViewStack view_stack_;
+ std::unique_ptr<ViewStack> view_stack_;
// A full dialog overlay that shows a spinner and the "processing" label. It's
// hidden until ShowProcessingSpinner is called.

Powered by Google App Engine
This is Rietveld 408576698