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