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

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

Issue 2789093002: [Payments] Desktop: implement shipping address/option change (Closed)
Patch Set: compile fix 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/browser/ui/views/payments/payment_request_dialog_view.cc
diff --git a/chrome/browser/ui/views/payments/payment_request_dialog_view.cc b/chrome/browser/ui/views/payments/payment_request_dialog_view.cc
index c29aa0efebc47eaea1132d0e98ada0976f084641..778239cff36272394a852279b7f6a881d500a57a 100644
--- a/chrome/browser/ui/views/payments/payment_request_dialog_view.cc
+++ b/chrome/browser/ui/views/payments/payment_request_dialog_view.cc
@@ -54,6 +54,8 @@ PaymentRequestDialogView::PaymentRequestDialogView(
PaymentRequestDialogView::ObserverForTest* observer)
: request_(request), observer_for_testing_(observer), being_closed_(false) {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+ if (observer_for_testing_)
+ request->spec()->add_observer_for_testing(this);
SetLayoutManager(new views::FillLayout());
view_stack_.set_owned_by_client();
@@ -117,6 +119,12 @@ void PaymentRequestDialogView::ShowErrorMessage() {
observer_for_testing_->OnErrorMessageShown();
}
+void PaymentRequestDialogView::OnSpecUpdated() {
+ // Since this is called in tests only, |observer_for_testing_| is defined.
+ DCHECK(observer_for_testing_);
+ observer_for_testing_->OnSpecDoneUpdating();
+}
+
void PaymentRequestDialogView::Pay() {
request_->Pay();
}

Powered by Google App Engine
This is Rietveld 408576698