| OLD | NEW |
| 1 // Copyright 2013 The Chromium Authors. All rights reserved. | 1 // Copyright 2013 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/profiles/profile_loader.h" | 5 #include "chrome/browser/profiles/profile_loader.h" |
| 6 | 6 |
| 7 #include <map> | 7 #include <map> |
| 8 | 8 |
| 9 #include "base/bind.h" | 9 #include "base/bind.h" |
| 10 #include "base/files/file_path.h" | 10 #include "base/files/file_path.h" |
| 11 #include "base/memory/weak_ptr.h" | 11 #include "base/memory/weak_ptr.h" |
| 12 #include "chrome/browser/lifetime/application_lifetime.h" | 12 #include "chrome/browser/lifetime/application_lifetime.h" |
| 13 #include "chrome/browser/profiles/profile.h" | 13 #include "chrome/browser/profiles/profile.h" |
| 14 #include "chrome/browser/profiles/profile_loader.h" | 14 #include "chrome/browser/profiles/profile_loader.h" |
| 15 #include "chrome/test/base/testing_browser_process.h" |
| 15 #include "chrome/test/base/testing_profile.h" | 16 #include "chrome/test/base/testing_profile.h" |
| 17 #include "chrome/test/base/testing_profile_manager.h" |
| 18 #include "content/public/test/test_browser_thread_bundle.h" |
| 16 #include "testing/gmock/include/gmock/gmock.h" | 19 #include "testing/gmock/include/gmock/gmock.h" |
| 17 #include "testing/gtest/include/gtest/gtest.h" | 20 #include "testing/gtest/include/gtest/gtest.h" |
| 21 #include "ui/message_center/message_center.h" |
| 22 |
| 23 #if defined(OS_CHROMEOS) |
| 24 #include "base/command_line.h" |
| 25 #include "chrome/common/chrome_switches.h" |
| 26 #endif // defined(OS_CHROMEOS) |
| 18 | 27 |
| 19 namespace { | 28 namespace { |
| 20 | 29 |
| 21 using ::testing::_; | 30 using ::testing::_; |
| 22 using ::testing::Invoke; | 31 using ::testing::Invoke; |
| 23 using ::testing::Return; | 32 using ::testing::Return; |
| 24 using ::testing::StrictMock; | 33 using ::testing::StrictMock; |
| 25 using ::testing::WithArgs; | 34 using ::testing::WithArgs; |
| 26 | 35 |
| 27 class TestProfileLoader : public ProfileLoader { | 36 class TestProfileLoader : public ProfileLoader { |
| (...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 62 friend class base::RefCountedThreadSafe<MockCallback>; | 71 friend class base::RefCountedThreadSafe<MockCallback>; |
| 63 virtual ~MockCallback(); | 72 virtual ~MockCallback(); |
| 64 | 73 |
| 65 private: | 74 private: |
| 66 DISALLOW_COPY_AND_ASSIGN(MockCallback); | 75 DISALLOW_COPY_AND_ASSIGN(MockCallback); |
| 67 }; | 76 }; |
| 68 | 77 |
| 69 MockCallback::MockCallback() {} | 78 MockCallback::MockCallback() {} |
| 70 MockCallback::~MockCallback() {} | 79 MockCallback::~MockCallback() {} |
| 71 | 80 |
| 72 TEST(ProfileLoaderTest, LoadProfileInvalidatingOtherLoads) { | 81 class ProfileLoaderTest : public testing::Test { |
| 73 TestingProfile profile; | 82 protected: |
| 83 static void SetUpTestCase() { |
| 84 #if defined(OS_CHROMEOS) |
| 85 // Needed to handle http://crbug.com/119175. |
| 86 CommandLine::ForCurrentProcess()->AppendSwitch( |
| 87 switches::kDisableZeroBrowsersOpenForTests); |
| 88 #endif // defined(OS_CHROMEOS) |
| 89 message_center::MessageCenter::Initialize(); |
| 90 } |
| 91 |
| 92 static void TearDownTestCase() { |
| 93 message_center::MessageCenter::Shutdown(); |
| 94 } |
| 95 |
| 96 private: |
| 97 content::TestBrowserThreadBundle thread_bundle_; |
| 98 }; |
| 99 |
| 100 TEST_F(ProfileLoaderTest, LoadProfileInvalidatingOtherLoads) { |
| 101 TestingProfileManager profile_manager(TestingBrowserProcess::GetGlobal()); |
| 102 ASSERT_TRUE(profile_manager.SetUp()); |
| 103 TestingProfile* profile = |
| 104 profile_manager.CreateTestingProfile("TestProfile"); |
| 74 base::FilePath fake_profile_path_1 = | 105 base::FilePath fake_profile_path_1 = |
| 75 base::FilePath::FromUTF8Unsafe("fake/profile 1"); | 106 base::FilePath::FromUTF8Unsafe("fake/profile 1"); |
| 76 base::FilePath fake_profile_path_2 = | 107 base::FilePath fake_profile_path_2 = |
| 77 base::FilePath::FromUTF8Unsafe("fake/profile 2"); | 108 base::FilePath::FromUTF8Unsafe("fake/profile 2"); |
| 78 | 109 |
| 79 TestProfileLoader loader; | 110 TestProfileLoader loader; |
| 80 EXPECT_FALSE(loader.IsAnyProfileLoading()); | 111 EXPECT_FALSE(loader.IsAnyProfileLoading()); |
| 81 | 112 |
| 82 // path_1 never loads. | 113 // path_1 never loads. |
| 83 EXPECT_CALL(loader, GetProfileByPath(fake_profile_path_1)) | 114 EXPECT_CALL(loader, GetProfileByPath(fake_profile_path_1)) |
| 84 .WillRepeatedly(Return(static_cast<Profile*>(NULL))); | 115 .WillRepeatedly(Return(static_cast<Profile*>(NULL))); |
| 85 EXPECT_CALL(loader, | 116 EXPECT_CALL(loader, |
| 86 CreateProfileAsync(fake_profile_path_1, _, _, _, std::string())) | 117 CreateProfileAsync(fake_profile_path_1, _, _, _, std::string())) |
| 87 .WillRepeatedly(WithArgs<0, 1>( | 118 .WillRepeatedly(WithArgs<0, 1>( |
| 88 Invoke(&loader, &TestProfileLoader::SetCreateCallback))); | 119 Invoke(&loader, &TestProfileLoader::SetCreateCallback))); |
| 89 | 120 |
| 90 // path_2 loads after the first request. | 121 // path_2 loads after the first request. |
| 91 EXPECT_CALL(loader, GetProfileByPath(fake_profile_path_2)) | 122 EXPECT_CALL(loader, GetProfileByPath(fake_profile_path_2)) |
| 92 .WillOnce(Return(static_cast<Profile*>(NULL))) | 123 .WillOnce(Return(static_cast<Profile*>(NULL))) |
| 93 .WillRepeatedly(Return(&profile)); | 124 .WillRepeatedly(Return(profile)); |
| 94 EXPECT_CALL(loader, | 125 EXPECT_CALL(loader, |
| 95 CreateProfileAsync(fake_profile_path_2, _, _, _, std::string())) | 126 CreateProfileAsync(fake_profile_path_2, _, _, _, std::string())) |
| 96 .WillRepeatedly(WithArgs<0, 1>( | 127 .WillRepeatedly(WithArgs<0, 1>( |
| 97 Invoke(&loader, &TestProfileLoader::SetCreateCallback))); | 128 Invoke(&loader, &TestProfileLoader::SetCreateCallback))); |
| 98 | 129 |
| 99 // Try to load both paths twice. | 130 // Try to load both paths twice. |
| 100 // path_1_load is never called because it is first invalidated by the load | 131 // path_1_load is never called because it is first invalidated by the load |
| 101 // request for (path_2), and then invalidated manually. | 132 // request for (path_2), and then invalidated manually. |
| 102 // path_2_load is called both times. | 133 // path_2_load is called both times. |
| 103 StrictMock<MockCallback>* path_1_load = new StrictMock<MockCallback>(); | 134 StrictMock<MockCallback>* path_1_load = new StrictMock<MockCallback>(); |
| 104 StrictMock<MockCallback>* path_2_load = new StrictMock<MockCallback>(); | 135 StrictMock<MockCallback>* path_2_load = new StrictMock<MockCallback>(); |
| 105 EXPECT_CALL(*path_2_load, Run(&profile)) | 136 EXPECT_CALL(*path_2_load, Run(profile)) |
| 106 .Times(2); | 137 .Times(2); |
| 107 | 138 |
| 108 // Try to load path_1. | 139 // Try to load path_1. |
| 109 loader.LoadProfileInvalidatingOtherLoads( | 140 loader.LoadProfileInvalidatingOtherLoads( |
| 110 fake_profile_path_1, base::Bind(&MockCallback::Run, path_1_load)); | 141 fake_profile_path_1, base::Bind(&MockCallback::Run, path_1_load)); |
| 111 EXPECT_TRUE(loader.IsAnyProfileLoading()); | 142 EXPECT_TRUE(loader.IsAnyProfileLoading()); |
| 112 | 143 |
| 113 // Try to load path_2, this invalidates the previous request. | 144 // Try to load path_2, this invalidates the previous request. |
| 114 loader.LoadProfileInvalidatingOtherLoads( | 145 loader.LoadProfileInvalidatingOtherLoads( |
| 115 fake_profile_path_2, base::Bind(&MockCallback::Run, path_2_load)); | 146 fake_profile_path_2, base::Bind(&MockCallback::Run, path_2_load)); |
| 116 | 147 |
| 117 // Finish the load request for path_1, then for path_2. | 148 // Finish the load request for path_1, then for path_2. |
| 118 loader.RunCreateCallback(fake_profile_path_1, &profile, | 149 loader.RunCreateCallback(fake_profile_path_1, profile, |
| 119 Profile::CREATE_STATUS_INITIALIZED); | 150 Profile::CREATE_STATUS_INITIALIZED); |
| 120 loader.RunCreateCallback(fake_profile_path_2, &profile, | 151 loader.RunCreateCallback(fake_profile_path_2, profile, |
| 121 Profile::CREATE_STATUS_INITIALIZED); | 152 Profile::CREATE_STATUS_INITIALIZED); |
| 122 EXPECT_FALSE(loader.IsAnyProfileLoading()); | 153 EXPECT_FALSE(loader.IsAnyProfileLoading()); |
| 123 | 154 |
| 124 // The second request for path_2 should return immediately. | 155 // The second request for path_2 should return immediately. |
| 125 loader.LoadProfileInvalidatingOtherLoads( | 156 loader.LoadProfileInvalidatingOtherLoads( |
| 126 fake_profile_path_2, base::Bind(&MockCallback::Run, path_2_load)); | 157 fake_profile_path_2, base::Bind(&MockCallback::Run, path_2_load)); |
| 127 | 158 |
| 128 // Make a second request for path_1, and invalidate it. | 159 // Make a second request for path_1, and invalidate it. |
| 129 loader.LoadProfileInvalidatingOtherLoads( | 160 loader.LoadProfileInvalidatingOtherLoads( |
| 130 fake_profile_path_1, base::Bind(&MockCallback::Run, path_1_load)); | 161 fake_profile_path_1, base::Bind(&MockCallback::Run, path_1_load)); |
| 131 loader.InvalidatePendingProfileLoads(); | 162 loader.InvalidatePendingProfileLoads(); |
| 132 } | 163 } |
| 133 | 164 |
| 134 } // namespace | 165 } // namespace |
| OLD | NEW |