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

Unified Diff: chrome/browser/chromeos/login/screens/network_screen_browsertest.cc

Issue 679133002: ScreenObserver is renamed to BaseScreenDelegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes. Created 6 years, 2 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/chromeos/login/screens/network_screen_browsertest.cc
diff --git a/chrome/browser/chromeos/login/screens/network_screen_browsertest.cc b/chrome/browser/chromeos/login/screens/network_screen_browsertest.cc
index ec678815f04a12d2276ba64fab2ba922ca071f86..d7e82f73b1f954f0551427058da2d5f28085a99e 100644
--- a/chrome/browser/chromeos/login/screens/network_screen_browsertest.cc
+++ b/chrome/browser/chromeos/login/screens/network_screen_browsertest.cc
@@ -5,7 +5,7 @@
#include "base/memory/scoped_ptr.h"
#include "chrome/browser/chromeos/login/helper.h"
#include "chrome/browser/chromeos/login/screens/base_screen.h"
-#include "chrome/browser/chromeos/login/screens/mock_screen_observer.h"
+#include "chrome/browser/chromeos/login/screens/mock_base_screen_delegate.h"
#include "chrome/browser/chromeos/login/screens/network_screen.h"
#include "chrome/browser/chromeos/login/test/wizard_in_process_browser_test.h"
#include "chrome/browser/chromeos/login/wizard_controller.h"
@@ -60,14 +60,14 @@ class NetworkScreenTest : public WizardInProcessBrowserTest {
virtual void SetUpOnMainThread() override {
WizardInProcessBrowserTest::SetUpOnMainThread();
- mock_screen_observer_.reset(new MockScreenObserver());
+ mock_base_screen_delegate_.reset(new MockBaseScreenDelegate());
ASSERT_TRUE(WizardController::default_controller() != NULL);
network_screen_ =
NetworkScreen::Get(WizardController::default_controller());
ASSERT_TRUE(network_screen_ != NULL);
ASSERT_EQ(WizardController::default_controller()->current_screen(),
network_screen_);
- network_screen_->screen_observer_ = mock_screen_observer_.get();
+ network_screen_->base_screen_delegate_ = mock_base_screen_delegate_.get();
ASSERT_TRUE(network_screen_->actor() != NULL);
mock_network_state_helper_ = new login::MockNetworkStateHelper;
@@ -80,9 +80,8 @@ class NetworkScreenTest : public WizardInProcessBrowserTest {
}
void EmulateContinueButtonExit(NetworkScreen* network_screen) {
- EXPECT_CALL(*mock_screen_observer_,
- OnExit(ScreenObserver::NETWORK_CONNECTED))
- .Times(1);
+ EXPECT_CALL(*mock_base_screen_delegate_,
+ OnExit(BaseScreenDelegate::NETWORK_CONNECTED)).Times(1);
EXPECT_CALL(*mock_network_state_helper_, IsConnected())
.WillOnce(Return(true));
network_screen->OnContinuePressed();
@@ -101,7 +100,7 @@ class NetworkScreenTest : public WizardInProcessBrowserTest {
.WillRepeatedly((Return(false)));
}
- scoped_ptr<MockScreenObserver> mock_screen_observer_;
+ scoped_ptr<MockBaseScreenDelegate> mock_base_screen_delegate_;
login::MockNetworkStateHelper* mock_network_state_helper_;
NetworkScreen* network_screen_;
FakeSessionManagerClient* fake_session_manager_client_;
« no previous file with comments | « chrome/browser/chromeos/login/screens/network_screen.cc ('k') | chrome/browser/chromeos/login/screens/reset_screen.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698