| Index: chrome/browser/profiles/profile_manager_unittest.cc
|
| diff --git a/chrome/browser/profiles/profile_manager_unittest.cc b/chrome/browser/profiles/profile_manager_unittest.cc
|
| index dd2417bf219770324f4a70f890a22856c2d35bdb..1181443536cb46363b8d57c466703b0533f90482 100644
|
| --- a/chrome/browser/profiles/profile_manager_unittest.cc
|
| +++ b/chrome/browser/profiles/profile_manager_unittest.cc
|
| @@ -54,13 +54,9 @@ namespace {
|
| // observers is the same.
|
| Profile* g_created_profile;
|
|
|
| -} // namespace
|
| -
|
| -namespace testing {
|
| -
|
| -class ProfileManager : public ::ProfileManagerWithoutInit {
|
| +class UnittestProfileManager : public ::ProfileManagerWithoutInit {
|
| public:
|
| - explicit ProfileManager(const base::FilePath& user_data_dir)
|
| + explicit UnittestProfileManager(const base::FilePath& user_data_dir)
|
| : ::ProfileManagerWithoutInit(user_data_dir) {}
|
|
|
| protected:
|
| @@ -84,7 +80,7 @@ class ProfileManager : public ::ProfileManagerWithoutInit {
|
| }
|
| };
|
|
|
| -} // namespace testing
|
| +} // namespace
|
|
|
| class ProfileManagerTest : public testing::Test {
|
| protected:
|
| @@ -102,7 +98,7 @@ class ProfileManagerTest : public testing::Test {
|
| // Create a new temporary directory, and store the path
|
| ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
|
| TestingBrowserProcess::GetGlobal()->SetProfileManager(
|
| - new testing::ProfileManager(temp_dir_.path()));
|
| + new UnittestProfileManager(temp_dir_.path()));
|
|
|
| #if defined(OS_CHROMEOS)
|
| CommandLine* cl = CommandLine::ForCurrentProcess();
|
| @@ -417,11 +413,12 @@ TEST_F(ProfileManagerTest, GetLastUsedProfileAllowedByPolicy) {
|
|
|
| // Attach an incognito Profile to the TestingProfile.
|
| ASSERT_FALSE(profile->GetOffTheRecordProfile());
|
| - TestingProfile* incognito_profile = new TestingProfile();
|
| - incognito_profile->set_incognito(true);
|
| + TestingProfile::Builder builder;
|
| + builder.SetIncognito();
|
| + scoped_ptr<TestingProfile> incognito_profile = builder.Build();
|
| EXPECT_TRUE(incognito_profile->IsOffTheRecord());
|
| TestingProfile* testing_profile = static_cast<TestingProfile*>(profile);
|
| - testing_profile->SetOffTheRecordProfile(incognito_profile);
|
| + testing_profile->SetOffTheRecordProfile(incognito_profile.PassAs<Profile>());
|
| ASSERT_TRUE(profile->GetOffTheRecordProfile());
|
|
|
| IncognitoModePrefs::SetAvailability(prefs, IncognitoModePrefs::DISABLED);
|
| @@ -569,10 +566,10 @@ TEST_F(ProfileManagerTest, LastOpenedProfilesDoesNotContainIncognito) {
|
|
|
| // incognito profiles should not be managed by the profile manager but by the
|
| // original profile.
|
| - TestingProfile* profile2 = new TestingProfile();
|
| - ASSERT_TRUE(profile2);
|
| - profile2->set_incognito(true);
|
| - profile1->SetOffTheRecordProfile(profile2);
|
| + TestingProfile::Builder builder;
|
| + builder.SetIncognito();
|
| + scoped_ptr<TestingProfile> profile2 = builder.Build();
|
| + profile1->SetOffTheRecordProfile(profile2.PassAs<Profile>());
|
|
|
| std::vector<Profile*> last_opened_profiles =
|
| profile_manager->GetLastOpenedProfiles();
|
| @@ -588,7 +585,8 @@ TEST_F(ProfileManagerTest, LastOpenedProfilesDoesNotContainIncognito) {
|
| EXPECT_EQ(profile1, last_opened_profiles[0]);
|
|
|
| // And for profile2.
|
| - Browser::CreateParams profile2_params(profile2, chrome::GetActiveDesktop());
|
| + Browser::CreateParams profile2_params(profile1->GetOffTheRecordProfile(),
|
| + chrome::GetActiveDesktop());
|
| scoped_ptr<Browser> browser2a(
|
| chrome::CreateBrowserWithTestWindowForParams(&profile2_params));
|
|
|
|
|