diff options
| author | mlerman <mlerman@chromium.org> | 2015-01-09 08:23:35 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-01-09 16:24:21 +0000 |
| commit | 3247f076b4c8b9be5281d34b217c9236688a6725 (patch) | |
| tree | d898afb2544adeeb0fa9247065f474d2cafceb9c /chrome/browser/ui/cocoa/browser_window_controller_unittest.mm | |
| parent | 2d4e62d2cbd43d83fc43d58f0fdaa904633c33c1 (diff) | |
| download | chromium_src-3247f076b4c8b9be5281d34b217c9236688a6725.zip chromium_src-3247f076b4c8b9be5281d34b217c9236688a6725.tar.gz chromium_src-3247f076b4c8b9be5281d34b217c9236688a6725.tar.bz2 | |
Make the SigninErrorController a PKS.
This is a prelude to https://codereview.chromium.org/813133003/, which will introduce some dependencies between the SigninErrorController and some other PKSes. To make these cleaner, this is a yak shave to move the SigninErrorController out from being a member of the ProfileOAuth2TokenService (which is a ProfileKeyedService) and into its own PKS.
BUG=434829
Review URL: https://codereview.chromium.org/809313009
Cr-Commit-Position: refs/heads/master@{#310774}
Diffstat (limited to 'chrome/browser/ui/cocoa/browser_window_controller_unittest.mm')
| -rw-r--r-- | chrome/browser/ui/cocoa/browser_window_controller_unittest.mm | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/chrome/browser/ui/cocoa/browser_window_controller_unittest.mm b/chrome/browser/ui/cocoa/browser_window_controller_unittest.mm index 1022ad4..2f081ba 100644 --- a/chrome/browser/ui/cocoa/browser_window_controller_unittest.mm +++ b/chrome/browser/ui/cocoa/browser_window_controller_unittest.mm @@ -12,7 +12,7 @@ #include "chrome/app/chrome_command_ids.h" #include "chrome/browser/chrome_notification_types.h" #include "chrome/browser/signin/fake_signin_manager.h" -#include "chrome/browser/signin/profile_oauth2_token_service_factory.h" +#include "chrome/browser/signin/signin_error_controller_factory.h" #include "chrome/browser/signin/signin_manager_factory.h" #include "chrome/browser/sync/profile_sync_service.h" #include "chrome/browser/sync/profile_sync_service_factory.h" @@ -30,7 +30,6 @@ #include "chrome/grit/generated_resources.h" #include "chrome/test/base/testing_profile.h" #include "components/signin/core/browser/fake_auth_status_provider.h" -#include "components/signin/core/browser/profile_oauth2_token_service.h" #include "components/signin/core/browser/signin_error_controller.h" #include "components/signin/core/browser/signin_manager.h" #include "content/public/browser/notification_service.h" @@ -738,8 +737,7 @@ TEST_F(BrowserWindowControllerTest, TestSigninMenuItemAuthError) { sync->SetSyncSetupCompleted(); // Force an auth error. FakeAuthStatusProvider provider( - ProfileOAuth2TokenServiceFactory::GetForProfile(profile())-> - signin_error_controller()); + SigninErrorControllerFactory::GetForProfile(profile()));; GoogleServiceAuthError error( GoogleServiceAuthError::INVALID_GAIA_CREDENTIALS); provider.SetAuthError("user@gmail.com", "user@gmail.com", error); |