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

Side by Side Diff: chrome/browser/chromeos/login/signin/oauth2_browsertest.cc

Issue 324463003: ChromeOS login webui refactoring : Simplify login methods. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge with ToT Created 6 years, 6 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 <string> 5 #include <string>
6 6
7 #include "base/message_loop/message_loop.h" 7 #include "base/message_loop/message_loop.h"
8 #include "base/prefs/pref_service.h" 8 #include "base/prefs/pref_service.h"
9 #include "base/strings/stringprintf.h" 9 #include "base/strings/stringprintf.h"
10 #include "base/synchronization/waitable_event.h" 10 #include "base/synchronization/waitable_event.h"
11 #include "chrome/browser/browser_process.h" 11 #include "chrome/browser/browser_process.h"
12 #include "chrome/browser/chrome_notification_types.h" 12 #include "chrome/browser/chrome_notification_types.h"
13 #include "chrome/browser/chromeos/login/auth/key.h" 13 #include "chrome/browser/chromeos/login/auth/key.h"
14 #include "chrome/browser/chromeos/login/auth/user_context.h" 14 #include "chrome/browser/chromeos/login/auth/user_context.h"
15 #include "chrome/browser/chromeos/login/signin/oauth2_login_manager.h" 15 #include "chrome/browser/chromeos/login/signin/oauth2_login_manager.h"
16 #include "chrome/browser/chromeos/login/signin/oauth2_login_manager_factory.h" 16 #include "chrome/browser/chromeos/login/signin/oauth2_login_manager_factory.h"
17 #include "chrome/browser/chromeos/login/signin_specifics.h"
17 #include "chrome/browser/chromeos/login/test/oobe_base_test.h" 18 #include "chrome/browser/chromeos/login/test/oobe_base_test.h"
18 #include "chrome/browser/chromeos/login/users/user.h" 19 #include "chrome/browser/chromeos/login/users/user.h"
19 #include "chrome/browser/chromeos/login/users/user_manager.h" 20 #include "chrome/browser/chromeos/login/users/user_manager.h"
20 #include "chrome/browser/chromeos/login/wizard_controller.h" 21 #include "chrome/browser/chromeos/login/wizard_controller.h"
21 #include "chrome/browser/extensions/extension_test_message_listener.h" 22 #include "chrome/browser/extensions/extension_test_message_listener.h"
22 #include "chrome/browser/profiles/profile_manager.h" 23 #include "chrome/browser/profiles/profile_manager.h"
23 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h" 24 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
24 #include "chrome/browser/ui/app_modal_dialogs/javascript_app_modal_dialog.h" 25 #include "chrome/browser/ui/app_modal_dialogs/javascript_app_modal_dialog.h"
25 #include "chrome/browser/ui/app_modal_dialogs/native_app_modal_dialog.h" 26 #include "chrome/browser/ui/app_modal_dialogs/native_app_modal_dialog.h"
26 #include "chrome/browser/ui/browser.h" 27 #include "chrome/browser/ui/browser.h"
(...skipping 198 matching lines...) Expand 10 before | Expand all | Expand 10 after
225 const std::string& password) { 226 const std::string& password) {
226 ExistingUserController* controller = 227 ExistingUserController* controller =
227 ExistingUserController::current_controller(); 228 ExistingUserController::current_controller();
228 if (!controller) { 229 if (!controller) {
229 ADD_FAILURE(); 230 ADD_FAILURE();
230 return false; 231 return false;
231 } 232 }
232 233
233 UserContext user_context(username); 234 UserContext user_context(username);
234 user_context.SetKey(Key(password)); 235 user_context.SetKey(Key(password));
235 controller->Login(user_context); 236 controller->Login(user_context, SigninSpecifics());
236 content::WindowedNotificationObserver( 237 content::WindowedNotificationObserver(
237 chrome::NOTIFICATION_SESSION_STARTED, 238 chrome::NOTIFICATION_SESSION_STARTED,
238 content::NotificationService::AllSources()).Wait(); 239 content::NotificationService::AllSources()).Wait();
239 const UserList& logged_users = UserManager::Get()->GetLoggedInUsers(); 240 const UserList& logged_users = UserManager::Get()->GetLoggedInUsers();
240 for (UserList::const_iterator it = logged_users.begin(); 241 for (UserList::const_iterator it = logged_users.begin();
241 it != logged_users.end(); ++it) { 242 it != logged_users.end(); ++it) {
242 if ((*it)->email() == username) 243 if ((*it)->email() == username)
243 return true; 244 return true;
244 } 245 }
245 return false; 246 return false;
(...skipping 529 matching lines...) Expand 10 before | Expand all | Expand 10 after
775 776
776 if (!catcher.GetNextResult()) { 777 if (!catcher.GetNextResult()) {
777 std::string message = catcher.message(); 778 std::string message = catcher.message();
778 ADD_FAILURE() << "Tests failed: " << message; 779 ADD_FAILURE() << "Tests failed: " << message;
779 } 780 }
780 781
781 EXPECT_TRUE(fake_google_.IsPageRequested()); 782 EXPECT_TRUE(fake_google_.IsPageRequested());
782 } 783 }
783 784
784 } // namespace chromeos 785 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698