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

Side by Side Diff: chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc

Issue 366813007: Removed InProcessBrowserTest::CleanUpOnMainThread() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removed superfluous override. Created 6 years, 5 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 "base/basictypes.h" 5 #include "base/basictypes.h"
6 #include "base/compiler_specific.h" 6 #include "base/compiler_specific.h"
7 #include "base/prefs/pref_service.h" 7 #include "base/prefs/pref_service.h"
8 #include "chrome/browser/chromeos/login/login_manager_test.h" 8 #include "chrome/browser/chromeos/login/login_manager_test.h"
9 #include "chrome/browser/chromeos/login/startup_utils.h" 9 #include "chrome/browser/chromeos/login/startup_utils.h"
10 #include "chrome/browser/chromeos/login/ui/user_adding_screen.h" 10 #include "chrome/browser/chromeos/login/ui/user_adding_screen.h"
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 } 42 }
43 43
44 virtual void SetUpOnMainThread() OVERRIDE { 44 virtual void SetUpOnMainThread() OVERRIDE {
45 LoginManagerTest::SetUpOnMainThread(); 45 LoginManagerTest::SetUpOnMainThread();
46 CrosSettings* settings = CrosSettings::Get(); 46 CrosSettings* settings = CrosSettings::Get();
47 device_settings_provider_ = settings->GetProvider(kDeviceOwner); 47 device_settings_provider_ = settings->GetProvider(kDeviceOwner);
48 settings->RemoveSettingsProvider(device_settings_provider_); 48 settings->RemoveSettingsProvider(device_settings_provider_);
49 settings->AddSettingsProvider(&stub_settings_provider_); 49 settings->AddSettingsProvider(&stub_settings_provider_);
50 } 50 }
51 51
52 virtual void CleanUpOnMainThread() OVERRIDE { 52 virtual void TearDownOnMainThread() OVERRIDE {
53 CrosSettings* settings = CrosSettings::Get(); 53 CrosSettings* settings = CrosSettings::Get();
54 settings->RemoveSettingsProvider(&stub_settings_provider_); 54 settings->RemoveSettingsProvider(&stub_settings_provider_);
55 settings->AddSettingsProvider(device_settings_provider_); 55 settings->AddSettingsProvider(device_settings_provider_);
56 LoginManagerTest::CleanUpOnMainThread(); 56 LoginManagerTest::TearDownOnMainThread();
57 } 57 }
58 58
59 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE { 59 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
60 LoginManagerTest::SetUpCommandLine(command_line); 60 LoginManagerTest::SetUpCommandLine(command_line);
61 } 61 }
62 62
63 protected: 63 protected:
64 void CheckAccountsUI(const User* user, bool is_owner) { 64 void CheckAccountsUI(const User* user, bool is_owner) {
65 Profile* profile = ProfileHelper::Get()->GetProfileByUser(user); 65 Profile* profile = ProfileHelper::Get()->GetProfileByUser(user);
66 profile->GetPrefs()->SetString(prefs::kGoogleServicesUsername, 66 profile->GetPrefs()->SetString(prefs::kGoogleServicesUsername,
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 AddUser(kTestUsers[1]); 137 AddUser(kTestUsers[1]);
138 138
139 UserManager* manager = UserManager::Get(); 139 UserManager* manager = UserManager::Get();
140 ASSERT_EQ(2u, manager->GetLoggedInUsers().size()); 140 ASSERT_EQ(2u, manager->GetLoggedInUsers().size());
141 141
142 CheckAccountsUI(manager->FindUser(kTestUsers[0]), true /* is_owner */); 142 CheckAccountsUI(manager->FindUser(kTestUsers[0]), true /* is_owner */);
143 CheckAccountsUI(manager->FindUser(kTestUsers[1]), false /* is_owner */); 143 CheckAccountsUI(manager->FindUser(kTestUsers[1]), false /* is_owner */);
144 } 144 }
145 145
146 } // namespace chromeos 146 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698