| 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" | |
| 16 #include "chrome/test/base/testing_profile.h" | 15 #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" | |
| 19 #include "testing/gmock/include/gmock/gmock.h" | 16 #include "testing/gmock/include/gmock/gmock.h" |
| 20 #include "testing/gtest/include/gtest/gtest.h" | 17 #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) | |
| 27 | 18 |
| 28 namespace { | 19 namespace { |
| 29 | 20 |
| 30 using ::testing::_; | 21 using ::testing::_; |
| 31 using ::testing::Invoke; | 22 using ::testing::Invoke; |
| 32 using ::testing::Return; | 23 using ::testing::Return; |
| 33 using ::testing::StrictMock; | 24 using ::testing::StrictMock; |
| 34 using ::testing::WithArgs; | 25 using ::testing::WithArgs; |
| 35 | 26 |
| 36 class TestProfileLoader : public ProfileLoader { | 27 class TestProfileLoader : public ProfileLoader { |
| (...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 71 friend class base::RefCountedThreadSafe<MockCallback>; | 62 friend class base::RefCountedThreadSafe<MockCallback>; |
| 72 virtual ~MockCallback(); | 63 virtual ~MockCallback(); |
| 73 | 64 |
| 74 private: | 65 private: |
| 75 DISALLOW_COPY_AND_ASSIGN(MockCallback); | 66 DISALLOW_COPY_AND_ASSIGN(MockCallback); |
| 76 }; | 67 }; |
| 77 | 68 |
| 78 MockCallback::MockCallback() {} | 69 MockCallback::MockCallback() {} |
| 79 MockCallback::~MockCallback() {} | 70 MockCallback::~MockCallback() {} |
| 80 | 71 |
| 81 class ProfileLoaderTest : public testing::Test { | 72 TEST(ProfileLoaderTest, LoadProfileInvalidatingOtherLoads) { |
| 82 protected: | 73 TestingProfile profile; |
| 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"); | |
| 105 base::FilePath fake_profile_path_1 = | 74 base::FilePath fake_profile_path_1 = |
| 106 base::FilePath::FromUTF8Unsafe("fake/profile 1"); | 75 base::FilePath::FromUTF8Unsafe("fake/profile 1"); |
| 107 base::FilePath fake_profile_path_2 = | 76 base::FilePath fake_profile_path_2 = |
| 108 base::FilePath::FromUTF8Unsafe("fake/profile 2"); | 77 base::FilePath::FromUTF8Unsafe("fake/profile 2"); |
| 109 | 78 |
| 110 TestProfileLoader loader; | 79 TestProfileLoader loader; |
| 111 EXPECT_FALSE(loader.IsAnyProfileLoading()); | 80 EXPECT_FALSE(loader.IsAnyProfileLoading()); |
| 112 | 81 |
| 113 // path_1 never loads. | 82 // path_1 never loads. |
| 114 EXPECT_CALL(loader, GetProfileByPath(fake_profile_path_1)) | 83 EXPECT_CALL(loader, GetProfileByPath(fake_profile_path_1)) |
| 115 .WillRepeatedly(Return(static_cast<Profile*>(NULL))); | 84 .WillRepeatedly(Return(static_cast<Profile*>(NULL))); |
| 116 EXPECT_CALL(loader, | 85 EXPECT_CALL(loader, |
| 117 CreateProfileAsync(fake_profile_path_1, _, _, _, std::string())) | 86 CreateProfileAsync(fake_profile_path_1, _, _, _, std::string())) |
| 118 .WillRepeatedly(WithArgs<0, 1>( | 87 .WillRepeatedly(WithArgs<0, 1>( |
| 119 Invoke(&loader, &TestProfileLoader::SetCreateCallback))); | 88 Invoke(&loader, &TestProfileLoader::SetCreateCallback))); |
| 120 | 89 |
| 121 // path_2 loads after the first request. | 90 // path_2 loads after the first request. |
| 122 EXPECT_CALL(loader, GetProfileByPath(fake_profile_path_2)) | 91 EXPECT_CALL(loader, GetProfileByPath(fake_profile_path_2)) |
| 123 .WillOnce(Return(static_cast<Profile*>(NULL))) | 92 .WillOnce(Return(static_cast<Profile*>(NULL))) |
| 124 .WillRepeatedly(Return(profile)); | 93 .WillRepeatedly(Return(&profile)); |
| 125 EXPECT_CALL(loader, | 94 EXPECT_CALL(loader, |
| 126 CreateProfileAsync(fake_profile_path_2, _, _, _, std::string())) | 95 CreateProfileAsync(fake_profile_path_2, _, _, _, std::string())) |
| 127 .WillRepeatedly(WithArgs<0, 1>( | 96 .WillRepeatedly(WithArgs<0, 1>( |
| 128 Invoke(&loader, &TestProfileLoader::SetCreateCallback))); | 97 Invoke(&loader, &TestProfileLoader::SetCreateCallback))); |
| 129 | 98 |
| 130 // Try to load both paths twice. | 99 // Try to load both paths twice. |
| 131 // path_1_load is never called because it is first invalidated by the load | 100 // path_1_load is never called because it is first invalidated by the load |
| 132 // request for (path_2), and then invalidated manually. | 101 // request for (path_2), and then invalidated manually. |
| 133 // path_2_load is called both times. | 102 // path_2_load is called both times. |
| 134 StrictMock<MockCallback>* path_1_load = new StrictMock<MockCallback>(); | 103 StrictMock<MockCallback>* path_1_load = new StrictMock<MockCallback>(); |
| 135 StrictMock<MockCallback>* path_2_load = new StrictMock<MockCallback>(); | 104 StrictMock<MockCallback>* path_2_load = new StrictMock<MockCallback>(); |
| 136 EXPECT_CALL(*path_2_load, Run(profile)) | 105 EXPECT_CALL(*path_2_load, Run(&profile)) |
| 137 .Times(2); | 106 .Times(2); |
| 138 | 107 |
| 139 // Try to load path_1. | 108 // Try to load path_1. |
| 140 loader.LoadProfileInvalidatingOtherLoads( | 109 loader.LoadProfileInvalidatingOtherLoads( |
| 141 fake_profile_path_1, base::Bind(&MockCallback::Run, path_1_load)); | 110 fake_profile_path_1, base::Bind(&MockCallback::Run, path_1_load)); |
| 142 EXPECT_TRUE(loader.IsAnyProfileLoading()); | 111 EXPECT_TRUE(loader.IsAnyProfileLoading()); |
| 143 | 112 |
| 144 // Try to load path_2, this invalidates the previous request. | 113 // Try to load path_2, this invalidates the previous request. |
| 145 loader.LoadProfileInvalidatingOtherLoads( | 114 loader.LoadProfileInvalidatingOtherLoads( |
| 146 fake_profile_path_2, base::Bind(&MockCallback::Run, path_2_load)); | 115 fake_profile_path_2, base::Bind(&MockCallback::Run, path_2_load)); |
| 147 | 116 |
| 148 // Finish the load request for path_1, then for path_2. | 117 // Finish the load request for path_1, then for path_2. |
| 149 loader.RunCreateCallback(fake_profile_path_1, profile, | 118 loader.RunCreateCallback(fake_profile_path_1, &profile, |
| 150 Profile::CREATE_STATUS_INITIALIZED); | 119 Profile::CREATE_STATUS_INITIALIZED); |
| 151 loader.RunCreateCallback(fake_profile_path_2, profile, | 120 loader.RunCreateCallback(fake_profile_path_2, &profile, |
| 152 Profile::CREATE_STATUS_INITIALIZED); | 121 Profile::CREATE_STATUS_INITIALIZED); |
| 153 EXPECT_FALSE(loader.IsAnyProfileLoading()); | 122 EXPECT_FALSE(loader.IsAnyProfileLoading()); |
| 154 | 123 |
| 155 // The second request for path_2 should return immediately. | 124 // The second request for path_2 should return immediately. |
| 156 loader.LoadProfileInvalidatingOtherLoads( | 125 loader.LoadProfileInvalidatingOtherLoads( |
| 157 fake_profile_path_2, base::Bind(&MockCallback::Run, path_2_load)); | 126 fake_profile_path_2, base::Bind(&MockCallback::Run, path_2_load)); |
| 158 | 127 |
| 159 // Make a second request for path_1, and invalidate it. | 128 // Make a second request for path_1, and invalidate it. |
| 160 loader.LoadProfileInvalidatingOtherLoads( | 129 loader.LoadProfileInvalidatingOtherLoads( |
| 161 fake_profile_path_1, base::Bind(&MockCallback::Run, path_1_load)); | 130 fake_profile_path_1, base::Bind(&MockCallback::Run, path_1_load)); |
| 162 loader.InvalidatePendingProfileLoads(); | 131 loader.InvalidatePendingProfileLoads(); |
| 163 } | 132 } |
| 164 | 133 |
| 165 } // namespace | 134 } // namespace |
| OLD | NEW |