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

Side by Side Diff: chrome/browser/ui/views/payments/payment_request_browsertest_base.cc

Issue 2817533006: [Web Payments] Add PopMany and Size functions to ViewStack (Closed)
Patch Set: Remove unused Observer that caused use-after-free in tests. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/payments/payment_request_browsertest_base.h" 5 #include "chrome/browser/ui/views/payments/payment_request_browsertest_base.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 void PaymentRequestBrowserTestBase::OnShippingAddressEditorOpened() { 129 void PaymentRequestBrowserTestBase::OnShippingAddressEditorOpened() {
130 if (event_observer_) 130 if (event_observer_)
131 event_observer_->Observe(DialogEvent::SHIPPING_ADDRESS_EDITOR_OPENED); 131 event_observer_->Observe(DialogEvent::SHIPPING_ADDRESS_EDITOR_OPENED);
132 } 132 }
133 133
134 void PaymentRequestBrowserTestBase::OnBackNavigation() { 134 void PaymentRequestBrowserTestBase::OnBackNavigation() {
135 if (event_observer_) 135 if (event_observer_)
136 event_observer_->Observe(DialogEvent::BACK_NAVIGATION); 136 event_observer_->Observe(DialogEvent::BACK_NAVIGATION);
137 } 137 }
138 138
139 void PaymentRequestBrowserTestBase::OnBackToPaymentSheetNavigation() {
140 if (event_observer_)
141 event_observer_->Observe(DialogEvent::BACK_TO_PAYMENT_SHEET_NAVIGATION);
142 }
143
139 void PaymentRequestBrowserTestBase::OnContactInfoOpened() { 144 void PaymentRequestBrowserTestBase::OnContactInfoOpened() {
140 if (event_observer_) 145 if (event_observer_)
141 event_observer_->Observe(DialogEvent::CONTACT_INFO_OPENED); 146 event_observer_->Observe(DialogEvent::CONTACT_INFO_OPENED);
142 } 147 }
143 148
144 void PaymentRequestBrowserTestBase::OnEditorViewUpdated() { 149 void PaymentRequestBrowserTestBase::OnEditorViewUpdated() {
145 if (event_observer_) 150 if (event_observer_)
146 event_observer_->Observe(DialogEvent::EDITOR_VIEW_UPDATED); 151 event_observer_->Observe(DialogEvent::EDITOR_VIEW_UPDATED);
147 } 152 }
148 153
(...skipping 392 matching lines...) Expand 10 before | Expand all | Expand 10 after
541 std::list<DialogEvent> event_sequence) { 546 std::list<DialogEvent> event_sequence) {
542 event_observer_ = 547 event_observer_ =
543 base::MakeUnique<DialogEventObserver>(std::move(event_sequence)); 548 base::MakeUnique<DialogEventObserver>(std::move(event_sequence));
544 } 549 }
545 550
546 void PaymentRequestBrowserTestBase::WaitForObservedEvent() { 551 void PaymentRequestBrowserTestBase::WaitForObservedEvent() {
547 event_observer_->Wait(); 552 event_observer_->Wait();
548 } 553 }
549 554
550 } // namespace payments 555 } // namespace payments
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698