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

Side by Side Diff: chrome/browser/chromeos/login/session/chrome_session_manager_browsertest.cc

Issue 2721553004: Remove auto raw pointer deduction from non-linux specific code. (Closed)
Patch Set: rebase Created 3 years, 9 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/chromeos/login/session/chrome_session_manager.h" 5 #include "chrome/browser/chromeos/login/session/chrome_session_manager.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 login_display->ShowSigninScreenForCreds(kTestUsers[0], "fake_password"); 105 login_display->ShowSigninScreenForCreds(kTestUsers[0], "fake_password");
106 106
107 session_start_waiter.Wait(); 107 session_start_waiter.Wait();
108 108
109 // Verify that session state is ACTIVE with one user session. 109 // Verify that session state is ACTIVE with one user session.
110 EXPECT_EQ(session_manager::SessionState::ACTIVE, manager->session_state()); 110 EXPECT_EQ(session_manager::SessionState::ACTIVE, manager->session_state());
111 EXPECT_EQ(1u, manager->sessions().size()); 111 EXPECT_EQ(1u, manager->sessions().size());
112 } 112 }
113 113
114 IN_PROC_BROWSER_TEST_F(ChromeSessionManagerTest, PRE_LoginExistingUsers) { 114 IN_PROC_BROWSER_TEST_F(ChromeSessionManagerTest, PRE_LoginExistingUsers) {
115 for (auto user : kTestUsers) { 115 for (auto* user : kTestUsers) {
116 RegisterUser(user); 116 RegisterUser(user);
117 } 117 }
118 StartupUtils::MarkOobeCompleted(); 118 StartupUtils::MarkOobeCompleted();
119 } 119 }
120 120
121 IN_PROC_BROWSER_TEST_F(ChromeSessionManagerTest, LoginExistingUsers) { 121 IN_PROC_BROWSER_TEST_F(ChromeSessionManagerTest, LoginExistingUsers) {
122 // Verify that session state is LOGIN_PRIMARY with existing user data dir. 122 // Verify that session state is LOGIN_PRIMARY with existing user data dir.
123 session_manager::SessionManager* manager = 123 session_manager::SessionManager* manager =
124 session_manager::SessionManager::Get(); 124 session_manager::SessionManager::Get();
125 EXPECT_EQ(session_manager::SessionState::LOGIN_PRIMARY, 125 EXPECT_EQ(session_manager::SessionState::LOGIN_PRIMARY,
(...skipping 26 matching lines...) Expand all
152 152
153 // Verify that session manager has the correct user session info. 153 // Verify that session manager has the correct user session info.
154 ASSERT_EQ(arraysize(kTestUsers), manager->sessions().size()); 154 ASSERT_EQ(arraysize(kTestUsers), manager->sessions().size());
155 for (size_t i = 0; i < arraysize(kTestUsers); ++i) { 155 for (size_t i = 0; i < arraysize(kTestUsers); ++i) {
156 EXPECT_EQ(kTestUsers[i], 156 EXPECT_EQ(kTestUsers[i],
157 manager->sessions()[i].user_account_id.GetUserEmail()); 157 manager->sessions()[i].user_account_id.GetUserEmail());
158 } 158 }
159 } 159 }
160 160
161 } // namespace chromeos 161 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698