| OLD | NEW |
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 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/prefs/profile_pref_store_manager.h" | 5 #include "chrome/browser/prefs/profile_pref_store_manager.h" |
| 6 | 6 |
| 7 #include <stddef.h> | 7 #include <stddef.h> |
| 8 | 8 |
| 9 #include <memory> | 9 #include <memory> |
| 10 #include <vector> | 10 #include <vector> |
| 11 | 11 |
| 12 #include "base/callback_helpers.h" |
| 12 #include "base/compiler_specific.h" | 13 #include "base/compiler_specific.h" |
| 13 #include "base/files/file_enumerator.h" | 14 #include "base/files/file_enumerator.h" |
| 14 #include "base/files/file_util.h" | 15 #include "base/files/file_util.h" |
| 15 #include "base/files/scoped_temp_dir.h" | 16 #include "base/files/scoped_temp_dir.h" |
| 16 #include "base/macros.h" | 17 #include "base/macros.h" |
| 17 #include "base/memory/ptr_util.h" | 18 #include "base/memory/ptr_util.h" |
| 18 #include "base/memory/ref_counted.h" | 19 #include "base/memory/ref_counted.h" |
| 19 #include "base/run_loop.h" | 20 #include "base/run_loop.h" |
| 20 #include "base/strings/string_util.h" | 21 #include "base/strings/string_util.h" |
| 21 #include "base/values.h" | 22 #include "base/values.h" |
| (...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 60 FirstEqualsPredicate(key))) | 61 FirstEqualsPredicate(key))) |
| 61 << "Unregistered key " << key << " was changed."; | 62 << "Unregistered key " << key << " was changed."; |
| 62 } | 63 } |
| 63 | 64 |
| 64 void OnInitializationCompleted(bool succeeded) override {} | 65 void OnInitializationCompleted(bool succeeded) override {} |
| 65 | 66 |
| 66 private: | 67 private: |
| 67 scoped_refptr<PrefRegistry> pref_registry_; | 68 scoped_refptr<PrefRegistry> pref_registry_; |
| 68 }; | 69 }; |
| 69 | 70 |
| 71 class PrefStoreReadObserver : public PrefStore::Observer { |
| 72 public: |
| 73 explicit PrefStoreReadObserver(scoped_refptr<PersistentPrefStore> pref_store) |
| 74 : pref_store_(std::move(pref_store)) { |
| 75 pref_store_->AddObserver(this); |
| 76 } |
| 77 |
| 78 ~PrefStoreReadObserver() override { pref_store_->RemoveObserver(this); } |
| 79 |
| 80 PersistentPrefStore::PrefReadError Read() { |
| 81 base::RunLoop run_loop; |
| 82 stop_waiting_ = run_loop.QuitClosure(); |
| 83 pref_store_->ReadPrefsAsync(nullptr); |
| 84 run_loop.Run(); |
| 85 return pref_store_->GetReadError(); |
| 86 } |
| 87 |
| 88 // PrefStore::Observer implementation |
| 89 void OnPrefValueChanged(const std::string& key) override {} |
| 90 |
| 91 void OnInitializationCompleted(bool succeeded) override { |
| 92 if (!stop_waiting_.is_null()) { |
| 93 base::ResetAndReturn(&stop_waiting_).Run(); |
| 94 } |
| 95 } |
| 96 |
| 97 private: |
| 98 scoped_refptr<PersistentPrefStore> pref_store_; |
| 99 base::Closure stop_waiting_; |
| 100 }; |
| 101 |
| 70 const char kUnprotectedPref[] = "unprotected_pref"; | 102 const char kUnprotectedPref[] = "unprotected_pref"; |
| 71 const char kTrackedAtomic[] = "tracked_atomic"; | 103 const char kTrackedAtomic[] = "tracked_atomic"; |
| 72 const char kProtectedAtomic[] = "protected_atomic"; | 104 const char kProtectedAtomic[] = "protected_atomic"; |
| 73 | 105 |
| 74 const char kFoobar[] = "FOOBAR"; | 106 const char kFoobar[] = "FOOBAR"; |
| 75 const char kBarfoo[] = "BARFOO"; | 107 const char kBarfoo[] = "BARFOO"; |
| 76 const char kHelloWorld[] = "HELLOWORLD"; | 108 const char kHelloWorld[] = "HELLOWORLD"; |
| 77 const char kGoodbyeWorld[] = "GOODBYEWORLD"; | 109 const char kGoodbyeWorld[] = "GOODBYEWORLD"; |
| 78 | 110 |
| 79 const PrefHashFilter::TrackedPreferenceMetadata kConfiguration[] = { | 111 const PrefHashFilter::TrackedPreferenceMetadata kConfiguration[] = { |
| 80 {0u, kTrackedAtomic, PrefHashFilter::NO_ENFORCEMENT, | 112 {0u, kTrackedAtomic, PrefHashFilter::NO_ENFORCEMENT, |
| 81 PrefHashFilter::TRACKING_STRATEGY_ATOMIC}, | 113 PrefHashFilter::TRACKING_STRATEGY_ATOMIC}, |
| 82 {1u, kProtectedAtomic, PrefHashFilter::ENFORCE_ON_LOAD, | 114 {1u, kProtectedAtomic, PrefHashFilter::ENFORCE_ON_LOAD, |
| 83 PrefHashFilter::TRACKING_STRATEGY_ATOMIC}}; | 115 PrefHashFilter::TRACKING_STRATEGY_ATOMIC}}; |
| 84 | 116 |
| 85 const size_t kExtraReportingId = 2u; | 117 const size_t kExtraReportingId = 2u; |
| 86 const size_t kReportingIdCount = 3u; | 118 const size_t kReportingIdCount = 3u; |
| 87 | 119 |
| 88 } // namespace | 120 } // namespace |
| 89 | 121 |
| 90 class ProfilePrefStoreManagerTest : public testing::Test { | 122 class ProfilePrefStoreManagerTest : public testing::Test { |
| 91 public: | 123 public: |
| 92 ProfilePrefStoreManagerTest() | 124 ProfilePrefStoreManagerTest() |
| 93 : configuration_(kConfiguration, | 125 : configuration_(kConfiguration, |
| 94 kConfiguration + arraysize(kConfiguration)), | 126 kConfiguration + arraysize(kConfiguration)), |
| 95 profile_pref_registry_(new user_prefs::PrefRegistrySyncable), | 127 profile_pref_registry_(new user_prefs::PrefRegistrySyncable), |
| 96 registry_verifier_(profile_pref_registry_.get()), | 128 registry_verifier_(profile_pref_registry_.get()), |
| 129 mock_validation_delegate_weak_factory_(&mock_validation_delegate_), |
| 97 seed_("seed"), | 130 seed_("seed"), |
| 98 reset_recorded_(false) {} | 131 reset_recorded_(false) {} |
| 99 | 132 |
| 100 void SetUp() override { | 133 void SetUp() override { |
| 101 ProfilePrefStoreManager::RegisterProfilePrefs(profile_pref_registry_.get()); | 134 ProfilePrefStoreManager::RegisterProfilePrefs(profile_pref_registry_.get()); |
| 102 for (const PrefHashFilter::TrackedPreferenceMetadata* it = kConfiguration; | 135 for (const PrefHashFilter::TrackedPreferenceMetadata* it = kConfiguration; |
| 103 it != kConfiguration + arraysize(kConfiguration); | 136 it != kConfiguration + arraysize(kConfiguration); |
| 104 ++it) { | 137 ++it) { |
| 105 if (it->strategy == PrefHashFilter::TRACKING_STRATEGY_ATOMIC) { | 138 if (it->strategy == PrefHashFilter::TRACKING_STRATEGY_ATOMIC) { |
| 106 profile_pref_registry_->RegisterStringPref(it->name, std::string()); | 139 profile_pref_registry_->RegisterStringPref(it->name, std::string()); |
| (...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 161 pref_service_factory.Create(profile_pref_registry_.get())); | 194 pref_service_factory.Create(profile_pref_registry_.get())); |
| 162 | 195 |
| 163 ProfilePrefStoreManager::ClearResetTime(pref_service.get()); | 196 ProfilePrefStoreManager::ClearResetTime(pref_service.get()); |
| 164 } | 197 } |
| 165 | 198 |
| 166 void InitializePrefs() { | 199 void InitializePrefs() { |
| 167 // According to the implementation of ProfilePrefStoreManager, this is | 200 // According to the implementation of ProfilePrefStoreManager, this is |
| 168 // actually a SegregatedPrefStore backed by two underlying pref stores. | 201 // actually a SegregatedPrefStore backed by two underlying pref stores. |
| 169 scoped_refptr<PersistentPrefStore> pref_store = | 202 scoped_refptr<PersistentPrefStore> pref_store = |
| 170 manager_->CreateProfilePrefStore( | 203 manager_->CreateProfilePrefStore( |
| 171 main_message_loop_.task_runner(), | 204 main_message_loop_.task_runner(), main_message_loop_.task_runner(), |
| 172 base::Bind(&ProfilePrefStoreManagerTest::RecordReset, | 205 base::Bind(&ProfilePrefStoreManagerTest::RecordReset, |
| 173 base::Unretained(this)), | 206 base::Unretained(this)), |
| 174 &mock_validation_delegate_); | 207 mock_validation_delegate_weak_factory_.GetWeakPtr()); |
| 175 InitializePrefStore(pref_store.get()); | 208 InitializePrefStore(pref_store.get()); |
| 176 pref_store = NULL; | 209 pref_store = NULL; |
| 177 base::RunLoop().RunUntilIdle(); | 210 base::RunLoop().RunUntilIdle(); |
| 178 } | 211 } |
| 179 | 212 |
| 180 void DestroyPrefStore() { | 213 void DestroyPrefStore() { |
| 181 if (pref_store_.get()) { | 214 if (pref_store_.get()) { |
| 182 ClearResetRecorded(); | 215 ClearResetRecorded(); |
| 183 // Force everything to be written to disk, triggering the PrefHashFilter | 216 // Force everything to be written to disk, triggering the PrefHashFilter |
| 184 // while our RegistryVerifier is watching. | 217 // while our RegistryVerifier is watching. |
| 185 pref_store_->CommitPendingWrite(); | 218 pref_store_->CommitPendingWrite(); |
| 186 base::RunLoop().RunUntilIdle(); | 219 base::RunLoop().RunUntilIdle(); |
| 187 | 220 |
| 188 pref_store_->RemoveObserver(®istry_verifier_); | 221 pref_store_->RemoveObserver(®istry_verifier_); |
| 189 pref_store_ = NULL; | 222 pref_store_ = NULL; |
| 190 // Nothing should have to happen on the background threads, but just in | 223 // Nothing should have to happen on the background threads, but just in |
| 191 // case... | 224 // case... |
| 192 base::RunLoop().RunUntilIdle(); | 225 base::RunLoop().RunUntilIdle(); |
| 193 } | 226 } |
| 194 } | 227 } |
| 195 | 228 |
| 196 void InitializePrefStore(PersistentPrefStore* pref_store) { | 229 void InitializePrefStore(PersistentPrefStore* pref_store) { |
| 197 pref_store->AddObserver(®istry_verifier_); | 230 pref_store->AddObserver(®istry_verifier_); |
| 198 PersistentPrefStore::PrefReadError error = pref_store->ReadPrefs(); | 231 PrefStoreReadObserver read_observer(pref_store); |
| 232 PersistentPrefStore::PrefReadError error = read_observer.Read(); |
| 199 EXPECT_EQ(PersistentPrefStore::PREF_READ_ERROR_NO_FILE, error); | 233 EXPECT_EQ(PersistentPrefStore::PREF_READ_ERROR_NO_FILE, error); |
| 200 pref_store->SetValue(kTrackedAtomic, | 234 pref_store->SetValue(kTrackedAtomic, |
| 201 base::MakeUnique<base::StringValue>(kFoobar), | 235 base::MakeUnique<base::StringValue>(kFoobar), |
| 202 WriteablePrefStore::DEFAULT_PREF_WRITE_FLAGS); | 236 WriteablePrefStore::DEFAULT_PREF_WRITE_FLAGS); |
| 203 pref_store->SetValue(kProtectedAtomic, | 237 pref_store->SetValue(kProtectedAtomic, |
| 204 base::MakeUnique<base::StringValue>(kHelloWorld), | 238 base::MakeUnique<base::StringValue>(kHelloWorld), |
| 205 WriteablePrefStore::DEFAULT_PREF_WRITE_FLAGS); | 239 WriteablePrefStore::DEFAULT_PREF_WRITE_FLAGS); |
| 206 pref_store->SetValue(kUnprotectedPref, | 240 pref_store->SetValue(kUnprotectedPref, |
| 207 base::MakeUnique<base::StringValue>(kFoobar), | 241 base::MakeUnique<base::StringValue>(kFoobar), |
| 208 WriteablePrefStore::DEFAULT_PREF_WRITE_FLAGS); | 242 WriteablePrefStore::DEFAULT_PREF_WRITE_FLAGS); |
| 209 pref_store->RemoveObserver(®istry_verifier_); | 243 pref_store->RemoveObserver(®istry_verifier_); |
| 210 pref_store->CommitPendingWrite(); | 244 pref_store->CommitPendingWrite(); |
| 211 base::RunLoop().RunUntilIdle(); | 245 base::RunLoop().RunUntilIdle(); |
| 212 } | 246 } |
| 213 | 247 |
| 214 void LoadExistingPrefs() { | 248 void LoadExistingPrefs() { |
| 215 DestroyPrefStore(); | 249 DestroyPrefStore(); |
| 216 pref_store_ = manager_->CreateProfilePrefStore( | 250 pref_store_ = manager_->CreateProfilePrefStore( |
| 217 main_message_loop_.task_runner(), | 251 main_message_loop_.task_runner(), main_message_loop_.task_runner(), |
| 218 base::Bind(&ProfilePrefStoreManagerTest::RecordReset, | 252 base::Bind(&ProfilePrefStoreManagerTest::RecordReset, |
| 219 base::Unretained(this)), | 253 base::Unretained(this)), |
| 220 NULL); | 254 NULL); |
| 221 pref_store_->AddObserver(®istry_verifier_); | 255 pref_store_->AddObserver(®istry_verifier_); |
| 222 pref_store_->ReadPrefs(); | 256 PrefStoreReadObserver read_observer(pref_store_); |
| 257 read_observer.Read(); |
| 223 } | 258 } |
| 224 | 259 |
| 225 void ReplaceStringInPrefs(const std::string& find, | 260 void ReplaceStringInPrefs(const std::string& find, |
| 226 const std::string& replace) { | 261 const std::string& replace) { |
| 227 base::FileEnumerator file_enum(profile_dir_.GetPath(), true, | 262 base::FileEnumerator file_enum(profile_dir_.GetPath(), true, |
| 228 base::FileEnumerator::FILES); | 263 base::FileEnumerator::FILES); |
| 229 | 264 |
| 230 for (base::FilePath path = file_enum.Next(); !path.empty(); | 265 for (base::FilePath path = file_enum.Next(); !path.empty(); |
| 231 path = file_enum.Next()) { | 266 path = file_enum.Next()) { |
| 232 // Tamper with the file's contents | 267 // Tamper with the file's contents |
| (...skipping 26 matching lines...) Expand all Loading... |
| 259 ADD_FAILURE() << "No validation observed for preference: " << pref_path; | 294 ADD_FAILURE() << "No validation observed for preference: " << pref_path; |
| 260 } | 295 } |
| 261 | 296 |
| 262 base::MessageLoop main_message_loop_; | 297 base::MessageLoop main_message_loop_; |
| 263 std::vector<PrefHashFilter::TrackedPreferenceMetadata> configuration_; | 298 std::vector<PrefHashFilter::TrackedPreferenceMetadata> configuration_; |
| 264 base::ScopedTempDir profile_dir_; | 299 base::ScopedTempDir profile_dir_; |
| 265 TestingPrefServiceSimple local_state_; | 300 TestingPrefServiceSimple local_state_; |
| 266 scoped_refptr<user_prefs::PrefRegistrySyncable> profile_pref_registry_; | 301 scoped_refptr<user_prefs::PrefRegistrySyncable> profile_pref_registry_; |
| 267 RegistryVerifier registry_verifier_; | 302 RegistryVerifier registry_verifier_; |
| 268 MockValidationDelegate mock_validation_delegate_; | 303 MockValidationDelegate mock_validation_delegate_; |
| 304 base::WeakPtrFactory<TrackedPreferenceValidationDelegate> |
| 305 mock_validation_delegate_weak_factory_; |
| 269 std::unique_ptr<ProfilePrefStoreManager> manager_; | 306 std::unique_ptr<ProfilePrefStoreManager> manager_; |
| 270 scoped_refptr<PersistentPrefStore> pref_store_; | 307 scoped_refptr<PersistentPrefStore> pref_store_; |
| 271 | 308 |
| 272 std::string seed_; | 309 std::string seed_; |
| 273 | 310 |
| 274 private: | 311 private: |
| 275 void RecordReset() { | 312 void RecordReset() { |
| 276 // As-is |reset_recorded_| is only designed to remember a single reset, make | 313 // As-is |reset_recorded_| is only designed to remember a single reset, make |
| 277 // sure none was previously recorded (or that ClearResetRecorded() was | 314 // sure none was previously recorded (or that ClearResetRecorded() was |
| 278 // called). | 315 // called). |
| (...skipping 201 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 480 | 517 |
| 481 // Trigger the logic that migrates it back to the unprotected preferences | 518 // Trigger the logic that migrates it back to the unprotected preferences |
| 482 // file. | 519 // file. |
| 483 pref_store_->SetValue(kProtectedAtomic, | 520 pref_store_->SetValue(kProtectedAtomic, |
| 484 base::WrapUnique(new base::StringValue(kGoodbyeWorld)), | 521 base::WrapUnique(new base::StringValue(kGoodbyeWorld)), |
| 485 WriteablePrefStore::DEFAULT_PREF_WRITE_FLAGS); | 522 WriteablePrefStore::DEFAULT_PREF_WRITE_FLAGS); |
| 486 LoadExistingPrefs(); | 523 LoadExistingPrefs(); |
| 487 ExpectStringValueEquals(kProtectedAtomic, kGoodbyeWorld); | 524 ExpectStringValueEquals(kProtectedAtomic, kGoodbyeWorld); |
| 488 VerifyResetRecorded(false); | 525 VerifyResetRecorded(false); |
| 489 } | 526 } |
| OLD | NEW |