Avi Drissman | 8ba1bad | 2022-09-13 19:22:36 | [diff] [blame] | 1 | // Copyright 2010 The Chromium Authors |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
vabr | 8684c9a | 2017-03-29 13:14:57 | [diff] [blame] | 5 | #include "components/prefs/pref_change_registrar.h" |
| 6 | |
Jinho Bang | 84b58bd | 2018-01-01 21:44:48 | [diff] [blame] | 7 | #include <memory> |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 8 | #include <string_view> |
Jinho Bang | 84b58bd | 2018-01-01 21:44:48 | [diff] [blame] | 9 | |
Avi Drissman | 12be031 | 2023-01-11 09:16:09 | [diff] [blame] | 10 | #include "base/functional/bind.h" |
| 11 | #include "base/functional/callback_helpers.h" |
brettw | f00b9b4 | 2016-02-01 22:11:38 | [diff] [blame] | 12 | #include "components/prefs/pref_observer.h" |
| 13 | #include "components/prefs/pref_registry_simple.h" |
| 14 | #include "components/prefs/testing_pref_service.h" |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 15 | #include "testing/gmock/include/gmock/gmock.h" |
| 16 | #include "testing/gtest/include/gtest/gtest.h" |
| 17 | |
[email protected] | 7e3ec42c | 2012-12-16 05:13:21 | [diff] [blame] | 18 | namespace base { |
[email protected] | f2d1f61 | 2010-12-09 15:10:17 | [diff] [blame] | 19 | namespace { |
| 20 | |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 21 | using testing::Mock; |
| 22 | |
[email protected] | 5bfdcfd | 2012-11-22 22:08:24 | [diff] [blame] | 23 | const char kHomePage[] = "homepage"; |
| 24 | const char kHomePageIsNewTabPage[] = "homepage_is_newtabpage"; |
| 25 | const char kApplicationLocale[] = "intl.app_locale"; |
| 26 | |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 27 | // A mock provider that allows us to capture pref observer changes. |
[email protected] | 5b19952 | 2012-12-22 17:24:44 | [diff] [blame] | 28 | class MockPrefService : public TestingPrefServiceSimple { |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 29 | public: |
Jan Keitel | 0a58e90 | 2024-07-10 15:54:59 | [diff] [blame] | 30 | MockPrefService() = default; |
| 31 | ~MockPrefService() override = default; |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 32 | |
Jan Keitel | 0a58e90 | 2024-07-10 15:54:59 | [diff] [blame] | 33 | MOCK_METHOD(void, |
| 34 | AddPrefObserver, |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 35 | (std::string_view, PrefObserver*), |
Jan Keitel | 0a58e90 | 2024-07-10 15:54:59 | [diff] [blame] | 36 | (override)); |
| 37 | MOCK_METHOD(void, |
| 38 | RemovePrefObserver, |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 39 | (std::string_view, PrefObserver*), |
Jan Keitel | 0a58e90 | 2024-07-10 15:54:59 | [diff] [blame] | 40 | (override)); |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 41 | }; |
| 42 | |
Sylvain Defresne | 073318e2 | 2019-10-14 11:06:02 | [diff] [blame] | 43 | // Due to overloads, base::DoNothing() cannot be passed directly to |
| 44 | // PrefChangeRegistrar::Add() as it is convertible to all callbacks. |
| 45 | base::RepeatingClosure DoNothingClosure() { |
| 46 | return base::DoNothing(); |
| 47 | } |
| 48 | |
[email protected] | f2d1f61 | 2010-12-09 15:10:17 | [diff] [blame] | 49 | } // namespace |
| 50 | |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 51 | class PrefChangeRegistrarTest : public testing::Test { |
| 52 | public: |
Jan Keitel | 0a58e90 | 2024-07-10 15:54:59 | [diff] [blame] | 53 | PrefChangeRegistrarTest() = default; |
| 54 | ~PrefChangeRegistrarTest() override = default; |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 55 | |
| 56 | protected: |
dcheng | 8aef3761 | 2014-12-23 02:56:47 | [diff] [blame] | 57 | void SetUp() override; |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 58 | |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 59 | MockPrefService* service() const { return service_.get(); } |
| 60 | |
| 61 | private: |
dcheng | 5f043bc | 2016-04-22 19:09:06 | [diff] [blame] | 62 | std::unique_ptr<MockPrefService> service_; |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 63 | }; |
| 64 | |
| 65 | void PrefChangeRegistrarTest::SetUp() { |
Peter Boström | 665b49d | 2021-04-05 19:29:58 | [diff] [blame] | 66 | service_ = std::make_unique<MockPrefService>(); |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 67 | } |
| 68 | |
| 69 | TEST_F(PrefChangeRegistrarTest, AddAndRemove) { |
| 70 | PrefChangeRegistrar registrar; |
| 71 | registrar.Init(service()); |
| 72 | |
| 73 | // Test adding. |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 74 | EXPECT_CALL(*service(), AddPrefObserver("test.pref.1", ®istrar)); |
| 75 | EXPECT_CALL(*service(), AddPrefObserver("test.pref.2", ®istrar)); |
Sylvain Defresne | 073318e2 | 2019-10-14 11:06:02 | [diff] [blame] | 76 | registrar.Add("test.pref.1", DoNothingClosure()); |
| 77 | registrar.Add("test.pref.2", DoNothingClosure()); |
[email protected] | 2fb7dc98 | 2010-09-29 12:24:28 | [diff] [blame] | 78 | EXPECT_FALSE(registrar.IsEmpty()); |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 79 | |
| 80 | // Test removing. |
| 81 | Mock::VerifyAndClearExpectations(service()); |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 82 | EXPECT_CALL(*service(), RemovePrefObserver("test.pref.1", ®istrar)); |
| 83 | EXPECT_CALL(*service(), RemovePrefObserver("test.pref.2", ®istrar)); |
[email protected] | 54ffd94a | 2012-11-12 15:29:20 | [diff] [blame] | 84 | registrar.Remove("test.pref.1"); |
| 85 | registrar.Remove("test.pref.2"); |
[email protected] | 2fb7dc98 | 2010-09-29 12:24:28 | [diff] [blame] | 86 | EXPECT_TRUE(registrar.IsEmpty()); |
| 87 | |
| 88 | // Explicitly check the expectations now to make sure that the Removes |
| 89 | // worked (rather than the registrar destructor doing the work). |
| 90 | Mock::VerifyAndClearExpectations(service()); |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 91 | } |
| 92 | |
| 93 | TEST_F(PrefChangeRegistrarTest, AutoRemove) { |
| 94 | PrefChangeRegistrar registrar; |
| 95 | registrar.Init(service()); |
| 96 | |
| 97 | // Setup of auto-remove. |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 98 | EXPECT_CALL(*service(), AddPrefObserver("test.pref.1", ®istrar)); |
Sylvain Defresne | 073318e2 | 2019-10-14 11:06:02 | [diff] [blame] | 99 | registrar.Add("test.pref.1", DoNothingClosure()); |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 100 | Mock::VerifyAndClearExpectations(service()); |
[email protected] | 2fb7dc98 | 2010-09-29 12:24:28 | [diff] [blame] | 101 | EXPECT_FALSE(registrar.IsEmpty()); |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 102 | |
| 103 | // Test auto-removing. |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 104 | EXPECT_CALL(*service(), RemovePrefObserver("test.pref.1", ®istrar)); |
[email protected] | 492d214 | 2010-09-10 13:55:18 | [diff] [blame] | 105 | } |
[email protected] | 2fb7dc98 | 2010-09-29 12:24:28 | [diff] [blame] | 106 | |
| 107 | TEST_F(PrefChangeRegistrarTest, RemoveAll) { |
| 108 | PrefChangeRegistrar registrar; |
| 109 | registrar.Init(service()); |
| 110 | |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 111 | EXPECT_CALL(*service(), AddPrefObserver("test.pref.1", ®istrar)); |
| 112 | EXPECT_CALL(*service(), AddPrefObserver("test.pref.2", ®istrar)); |
Sylvain Defresne | 073318e2 | 2019-10-14 11:06:02 | [diff] [blame] | 113 | registrar.Add("test.pref.1", DoNothingClosure()); |
| 114 | registrar.Add("test.pref.2", DoNothingClosure()); |
[email protected] | 2fb7dc98 | 2010-09-29 12:24:28 | [diff] [blame] | 115 | Mock::VerifyAndClearExpectations(service()); |
| 116 | |
Jan Keitel | 6288ad6b | 2024-07-26 09:00:14 | [diff] [blame] | 117 | EXPECT_CALL(*service(), RemovePrefObserver("test.pref.1", ®istrar)); |
| 118 | EXPECT_CALL(*service(), RemovePrefObserver("test.pref.2", ®istrar)); |
[email protected] | 2fb7dc98 | 2010-09-29 12:24:28 | [diff] [blame] | 119 | registrar.RemoveAll(); |
| 120 | EXPECT_TRUE(registrar.IsEmpty()); |
| 121 | |
| 122 | // Explicitly check the expectations now to make sure that the RemoveAll |
| 123 | // worked (rather than the registrar destructor doing the work). |
| 124 | Mock::VerifyAndClearExpectations(service()); |
| 125 | } |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 126 | |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 127 | class ObserveSetOfPreferencesTest : public testing::Test { |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 128 | public: |
Daniel Cheng | 6784852 | 2018-04-27 22:04:41 | [diff] [blame] | 129 | void SetUp() override { |
Peter Boström | 665b49d | 2021-04-05 19:29:58 | [diff] [blame] | 130 | pref_service_ = std::make_unique<TestingPrefServiceSimple>(); |
[email protected] | b1de2c7 | 2013-02-06 02:45:47 | [diff] [blame] | 131 | PrefRegistrySimple* registry = pref_service_->registry(); |
| 132 | registry->RegisterStringPref(kHomePage, "http://google.com"); |
| 133 | registry->RegisterBooleanPref(kHomePageIsNewTabPage, false); |
[email protected] | 007b3f8 | 2013-04-09 08:46:45 | [diff] [blame] | 134 | registry->RegisterStringPref(kApplicationLocale, std::string()); |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 135 | } |
| 136 | |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 137 | PrefChangeRegistrar* CreatePrefChangeRegistrar() { |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 138 | PrefChangeRegistrar* pref_set = new PrefChangeRegistrar(); |
| 139 | pref_set->Init(pref_service_.get()); |
Sylvain Defresne | 073318e2 | 2019-10-14 11:06:02 | [diff] [blame] | 140 | pref_set->Add(kHomePage, DoNothingClosure()); |
| 141 | pref_set->Add(kHomePageIsNewTabPage, DoNothingClosure()); |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 142 | return pref_set; |
| 143 | } |
| 144 | |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 145 | MOCK_METHOD1(OnPreferenceChanged, void(const std::string&)); |
[email protected] | 54ffd94a | 2012-11-12 15:29:20 | [diff] [blame] | 146 | |
dcheng | 5f043bc | 2016-04-22 19:09:06 | [diff] [blame] | 147 | std::unique_ptr<TestingPrefServiceSimple> pref_service_; |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 148 | }; |
| 149 | |
| 150 | TEST_F(ObserveSetOfPreferencesTest, IsObserved) { |
dcheng | 5f043bc | 2016-04-22 19:09:06 | [diff] [blame] | 151 | std::unique_ptr<PrefChangeRegistrar> pref_set(CreatePrefChangeRegistrar()); |
[email protected] | 5bfdcfd | 2012-11-22 22:08:24 | [diff] [blame] | 152 | EXPECT_TRUE(pref_set->IsObserved(kHomePage)); |
| 153 | EXPECT_TRUE(pref_set->IsObserved(kHomePageIsNewTabPage)); |
| 154 | EXPECT_FALSE(pref_set->IsObserved(kApplicationLocale)); |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 155 | } |
| 156 | |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 157 | TEST_F(ObserveSetOfPreferencesTest, Observe) { |
| 158 | using testing::_; |
| 159 | using testing::Mock; |
| 160 | |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 161 | PrefChangeRegistrar pref_set; |
Sylvain Defresne | 073318e2 | 2019-10-14 11:06:02 | [diff] [blame] | 162 | PrefChangeRegistrar::NamedChangeCallback callback = |
| 163 | base::BindRepeating(&ObserveSetOfPreferencesTest::OnPreferenceChanged, |
| 164 | base::Unretained(this)); |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 165 | pref_set.Init(pref_service_.get()); |
| 166 | pref_set.Add(kHomePage, callback); |
| 167 | pref_set.Add(kHomePageIsNewTabPage, callback); |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 168 | |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 169 | EXPECT_CALL(*this, OnPreferenceChanged(kHomePage)); |
vabr | 8684c9a | 2017-03-29 13:14:57 | [diff] [blame] | 170 | pref_service_->SetUserPref(kHomePage, |
Jinho Bang | 84b58bd | 2018-01-01 21:44:48 | [diff] [blame] | 171 | std::make_unique<Value>("http://crbug.com")); |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 172 | Mock::VerifyAndClearExpectations(this); |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 173 | |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 174 | EXPECT_CALL(*this, OnPreferenceChanged(kHomePageIsNewTabPage)); |
vabr | 8684c9a | 2017-03-29 13:14:57 | [diff] [blame] | 175 | pref_service_->SetUserPref(kHomePageIsNewTabPage, |
Jinho Bang | 84b58bd | 2018-01-01 21:44:48 | [diff] [blame] | 176 | std::make_unique<Value>(true)); |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 177 | Mock::VerifyAndClearExpectations(this); |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 178 | |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 179 | EXPECT_CALL(*this, OnPreferenceChanged(_)).Times(0); |
vabr | 8684c9a | 2017-03-29 13:14:57 | [diff] [blame] | 180 | pref_service_->SetUserPref(kApplicationLocale, |
Jinho Bang | 84b58bd | 2018-01-01 21:44:48 | [diff] [blame] | 181 | std::make_unique<Value>("en_US.utf8")); |
[email protected] | 96a5c34 | 2012-12-04 18:14:02 | [diff] [blame] | 182 | Mock::VerifyAndClearExpectations(this); |
[email protected] | cb7cec1 | 2012-10-15 23:54:59 | [diff] [blame] | 183 | } |
[email protected] | 7e3ec42c | 2012-12-16 05:13:21 | [diff] [blame] | 184 | |
| 185 | } // namespace base |