| 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/supervised_user/supervised_user_settings_service.h" | 5 #include "chrome/browser/supervised_user/supervised_user_settings_service.h" |
| 6 | 6 |
| 7 #include <utility> | 7 #include <utility> |
| 8 | 8 |
| 9 #include "base/bind.h" | 9 #include "base/bind.h" |
| 10 #include "base/callback.h" | 10 #include "base/callback.h" |
| (...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 78 syncer::SUPERVISED_USER_SETTINGS, initial_sync_data, | 78 syncer::SUPERVISED_USER_SETTINGS, initial_sync_data, |
| 79 CreateSyncProcessor(), std::move(error_handler)); | 79 CreateSyncProcessor(), std::move(error_handler)); |
| 80 EXPECT_FALSE(result.error().IsSet()); | 80 EXPECT_FALSE(result.error().IsSet()); |
| 81 return result; | 81 return result; |
| 82 } | 82 } |
| 83 | 83 |
| 84 void UploadSplitItem(const std::string& key, const std::string& value) { | 84 void UploadSplitItem(const std::string& key, const std::string& value) { |
| 85 split_items_.SetStringWithoutPathExpansion(key, value); | 85 split_items_.SetStringWithoutPathExpansion(key, value); |
| 86 settings_service_.UploadItem( | 86 settings_service_.UploadItem( |
| 87 SupervisedUserSettingsService::MakeSplitSettingKey(kSplitItemName, key), | 87 SupervisedUserSettingsService::MakeSplitSettingKey(kSplitItemName, key), |
| 88 std::unique_ptr<base::Value>(new base::StringValue(value))); | 88 std::unique_ptr<base::Value>(new base::Value(value))); |
| 89 } | 89 } |
| 90 | 90 |
| 91 void UploadAtomicItem(const std::string& value) { | 91 void UploadAtomicItem(const std::string& value) { |
| 92 atomic_setting_value_.reset(new base::StringValue(value)); | 92 atomic_setting_value_.reset(new base::Value(value)); |
| 93 settings_service_.UploadItem( | 93 settings_service_.UploadItem( |
| 94 kAtomicItemName, | 94 kAtomicItemName, std::unique_ptr<base::Value>(new base::Value(value))); |
| 95 std::unique_ptr<base::Value>(new base::StringValue(value))); | |
| 96 } | 95 } |
| 97 | 96 |
| 98 void VerifySyncDataItem(syncer::SyncData sync_data) { | 97 void VerifySyncDataItem(syncer::SyncData sync_data) { |
| 99 const sync_pb::ManagedUserSettingSpecifics& supervised_user_setting = | 98 const sync_pb::ManagedUserSettingSpecifics& supervised_user_setting = |
| 100 sync_data.GetSpecifics().managed_user_setting(); | 99 sync_data.GetSpecifics().managed_user_setting(); |
| 101 base::Value* expected_value = nullptr; | 100 base::Value* expected_value = nullptr; |
| 102 if (supervised_user_setting.name() == kAtomicItemName) { | 101 if (supervised_user_setting.name() == kAtomicItemName) { |
| 103 expected_value = atomic_setting_value_.get(); | 102 expected_value = atomic_setting_value_.get(); |
| 104 } else { | 103 } else { |
| 105 EXPECT_TRUE(base::StartsWith(supervised_user_setting.name(), | 104 EXPECT_TRUE(base::StartsWith(supervised_user_setting.name(), |
| (...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 151 | 150 |
| 152 TEST_F(SupervisedUserSettingsServiceTest, ProcessAtomicSetting) { | 151 TEST_F(SupervisedUserSettingsServiceTest, ProcessAtomicSetting) { |
| 153 StartSyncing(syncer::SyncDataList()); | 152 StartSyncing(syncer::SyncDataList()); |
| 154 ASSERT_TRUE(settings_); | 153 ASSERT_TRUE(settings_); |
| 155 const base::Value* value = nullptr; | 154 const base::Value* value = nullptr; |
| 156 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); | 155 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); |
| 157 | 156 |
| 158 settings_.reset(); | 157 settings_.reset(); |
| 159 syncer::SyncData data = | 158 syncer::SyncData data = |
| 160 SupervisedUserSettingsService::CreateSyncDataForSetting( | 159 SupervisedUserSettingsService::CreateSyncDataForSetting( |
| 161 kSettingsName, base::StringValue(kSettingsValue)); | 160 kSettingsName, base::Value(kSettingsValue)); |
| 162 syncer::SyncChangeList change_list; | 161 syncer::SyncChangeList change_list; |
| 163 change_list.push_back( | 162 change_list.push_back( |
| 164 syncer::SyncChange(FROM_HERE, syncer::SyncChange::ACTION_ADD, data)); | 163 syncer::SyncChange(FROM_HERE, syncer::SyncChange::ACTION_ADD, data)); |
| 165 syncer::SyncError error = | 164 syncer::SyncError error = |
| 166 settings_service_.ProcessSyncChanges(FROM_HERE, change_list); | 165 settings_service_.ProcessSyncChanges(FROM_HERE, change_list); |
| 167 EXPECT_FALSE(error.IsSet()) << error.ToString(); | 166 EXPECT_FALSE(error.IsSet()) << error.ToString(); |
| 168 ASSERT_TRUE(settings_); | 167 ASSERT_TRUE(settings_); |
| 169 ASSERT_TRUE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); | 168 ASSERT_TRUE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); |
| 170 std::string string_value; | 169 std::string string_value; |
| 171 EXPECT_TRUE(value->GetAsString(&string_value)); | 170 EXPECT_TRUE(value->GetAsString(&string_value)); |
| (...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 215 ASSERT_TRUE(settings_); | 214 ASSERT_TRUE(settings_); |
| 216 const base::Value* value = nullptr; | 215 const base::Value* value = nullptr; |
| 217 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); | 216 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); |
| 218 | 217 |
| 219 settings_.reset(); | 218 settings_.reset(); |
| 220 | 219 |
| 221 { | 220 { |
| 222 syncer::SyncDataList sync_data; | 221 syncer::SyncDataList sync_data; |
| 223 // Adding 1 Atomic entry. | 222 // Adding 1 Atomic entry. |
| 224 sync_data.push_back(SupervisedUserSettingsService::CreateSyncDataForSetting( | 223 sync_data.push_back(SupervisedUserSettingsService::CreateSyncDataForSetting( |
| 225 kSettingsName, base::StringValue(kSettingsValue))); | 224 kSettingsName, base::Value(kSettingsValue))); |
| 226 // Adding 2 SplitSettings from dictionary. | 225 // Adding 2 SplitSettings from dictionary. |
| 227 base::DictionaryValue dict; | 226 base::DictionaryValue dict; |
| 228 dict.SetString("foo", "bar"); | 227 dict.SetString("foo", "bar"); |
| 229 dict.SetInteger("eaudecologne", 4711); | 228 dict.SetInteger("eaudecologne", 4711); |
| 230 for (base::DictionaryValue::Iterator it(dict); !it.IsAtEnd(); | 229 for (base::DictionaryValue::Iterator it(dict); !it.IsAtEnd(); |
| 231 it.Advance()) { | 230 it.Advance()) { |
| 232 sync_data.push_back( | 231 sync_data.push_back( |
| 233 SupervisedUserSettingsService::CreateSyncDataForSetting( | 232 SupervisedUserSettingsService::CreateSyncDataForSetting( |
| 234 SupervisedUserSettingsService::MakeSplitSettingKey(kSplitItemName, | 233 SupervisedUserSettingsService::MakeSplitSettingKey(kSplitItemName, |
| 235 it.key()), | 234 it.key()), |
| (...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 274 } | 273 } |
| 275 } | 274 } |
| 276 | 275 |
| 277 TEST_F(SupervisedUserSettingsServiceTest, SetLocalSetting) { | 276 TEST_F(SupervisedUserSettingsServiceTest, SetLocalSetting) { |
| 278 const base::Value* value = nullptr; | 277 const base::Value* value = nullptr; |
| 279 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); | 278 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); |
| 280 | 279 |
| 281 settings_.reset(); | 280 settings_.reset(); |
| 282 settings_service_.SetLocalSetting( | 281 settings_service_.SetLocalSetting( |
| 283 kSettingsName, | 282 kSettingsName, |
| 284 std::unique_ptr<base::Value>(new base::StringValue(kSettingsValue))); | 283 std::unique_ptr<base::Value>(new base::Value(kSettingsValue))); |
| 285 ASSERT_TRUE(settings_); | 284 ASSERT_TRUE(settings_); |
| 286 ASSERT_TRUE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); | 285 ASSERT_TRUE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); |
| 287 std::string string_value; | 286 std::string string_value; |
| 288 EXPECT_TRUE(value->GetAsString(&string_value)); | 287 EXPECT_TRUE(value->GetAsString(&string_value)); |
| 289 EXPECT_EQ(kSettingsValue, string_value); | 288 EXPECT_EQ(kSettingsValue, string_value); |
| 290 } | 289 } |
| 291 | 290 |
| 292 TEST_F(SupervisedUserSettingsServiceTest, UploadItem) { | 291 TEST_F(SupervisedUserSettingsServiceTest, UploadItem) { |
| 293 UploadSplitItem("foo", "bar"); | 292 UploadSplitItem("foo", "bar"); |
| 294 UploadSplitItem("blurp", "baz"); | 293 UploadSplitItem("blurp", "baz"); |
| (...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 358 // The uploaded items should not show up as settings. | 357 // The uploaded items should not show up as settings. |
| 359 const base::Value* value = nullptr; | 358 const base::Value* value = nullptr; |
| 360 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kAtomicItemName, &value)); | 359 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kAtomicItemName, &value)); |
| 361 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSplitItemName, &value)); | 360 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSplitItemName, &value)); |
| 362 | 361 |
| 363 // Restarting sync should not create any new changes. | 362 // Restarting sync should not create any new changes. |
| 364 settings_service_.StopSyncing(syncer::SUPERVISED_USER_SETTINGS); | 363 settings_service_.StopSyncing(syncer::SUPERVISED_USER_SETTINGS); |
| 365 StartSyncing(sync_data); | 364 StartSyncing(sync_data); |
| 366 ASSERT_EQ(0u, sync_processor_->changes().size()); | 365 ASSERT_EQ(0u, sync_processor_->changes().size()); |
| 367 } | 366 } |
| OLD | NEW |