Chromium Code Reviews| Index: chrome/browser/chromeos/login/existing_user_controller_browsertest.cc |
| diff --git a/chrome/browser/chromeos/login/existing_user_controller_browsertest.cc b/chrome/browser/chromeos/login/existing_user_controller_browsertest.cc |
| index 5a824957d565fab148f7286275fe2d8302a8a202..c2d8f70292d645dc5ee97adba1db839ad26bc30e 100644 |
| --- a/chrome/browser/chromeos/login/existing_user_controller_browsertest.cc |
| +++ b/chrome/browser/chromeos/login/existing_user_controller_browsertest.cc |
| @@ -8,7 +8,6 @@ |
| #include "base/memory/scoped_ptr.h" |
| #include "chrome/browser/chromeos/cros/cros_in_process_browser_test.h" |
| #include "chrome/browser/chromeos/cros/cros_mock.h" |
| -#include "chrome/browser/chromeos/cros/mock_cryptohome_library.h" |
| #include "chrome/browser/chromeos/cros/mock_network_library.h" |
| #include "chrome/browser/chromeos/login/existing_user_controller.h" |
| #include "chrome/browser/chromeos/login/helper.h" |
| @@ -37,6 +36,7 @@ namespace chromeos { |
| using ::testing::AnyNumber; |
| using ::testing::AnyOf; |
| +using ::testing::Invoke; |
| using ::testing::InvokeWithoutArgs; |
| using ::testing::Return; |
| using ::testing::ReturnNull; |
| @@ -97,8 +97,7 @@ class MockLoginDisplayHost : public LoginDisplayHost { |
| class ExistingUserControllerTest : public CrosInProcessBrowserTest { |
| protected: |
| ExistingUserControllerTest() |
| - : mock_cryptohome_library_(NULL), |
| - mock_network_library_(NULL), |
| + : mock_network_library_(NULL), |
| mock_login_display_(NULL), |
| mock_login_display_host_(NULL), |
| testing_profile_(NULL) { |
| @@ -126,12 +125,6 @@ class ExistingUserControllerTest : public CrosInProcessBrowserTest { |
| EXPECT_CALL(*mock_session_manager_client, RetrieveDevicePolicy(_)) |
| .Times(AnyNumber()); |
| - cros_mock_->InitMockCryptohomeLibrary(); |
| - mock_cryptohome_library_ = cros_mock_->mock_cryptohome_library(); |
| - EXPECT_CALL(*mock_cryptohome_library_, IsMounted()) |
|
Nikita (slow)
2012/07/27 22:06:46
How do we check this expectation now?
hashimoto
2012/07/28 02:56:37
This expectation is set in the ctor of MockCryptoh
|
| - .Times(AnyNumber()) |
| - .WillRepeatedly(Return(true)); |
| - |
| mock_login_utils_ = new MockLoginUtils(); |
| LoginUtils::Set(mock_login_utils_); |
| EXPECT_CALL(*mock_login_utils_, PrewarmAuthentication()) |
| @@ -208,7 +201,6 @@ class ExistingUserControllerTest : public CrosInProcessBrowserTest { |
| scoped_ptr<ExistingUserController> existing_user_controller_; |
| // These mocks are owned by CrosLibrary class. |
| - MockCryptohomeLibrary* mock_cryptohome_library_; |
| MockNetworkLibrary* mock_network_library_; |
| MockLoginDisplay* mock_login_display_; |