Chromium Code Reviews| Index: chrome/browser/managed_mode/managed_user_shared_settings_update.cc |
| diff --git a/chrome/browser/managed_mode/managed_user_shared_settings_update.cc b/chrome/browser/managed_mode/managed_user_shared_settings_update.cc |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..721ca224a0ead6c3d3c0a749b300b786b951774c |
| --- /dev/null |
| +++ b/chrome/browser/managed_mode/managed_user_shared_settings_update.cc |
| @@ -0,0 +1,40 @@ |
| +// Copyright 2013 The Chromium Authors. All rights reserved. |
| +// Use of this source code is governed by a BSD-style license that can be |
| +// found in the LICENSE file. |
| + |
| +#include "chrome/browser/managed_mode/managed_user_shared_settings_update.h" |
| + |
| +#include "base/bind.h" |
| +#include "base/values.h" |
| +#include "chrome/browser/managed_mode/managed_user_shared_settings_service.h" |
| + |
| +ManagedUserSharedSettingsUpdate::ManagedUserSharedSettingsUpdate( |
| + ManagedUserSharedSettingsService* service, |
| + const std::string& mu_id, |
| + const std::string& key, |
| + scoped_ptr<base::Value> value, |
| + const base::Callback<void(bool)>& success_callback) |
| + : service_(service), |
| + mu_id_(mu_id), |
| + key_(key), |
| + value_(value.Pass()), |
| + callback_(success_callback) { |
| + service->SetValueInternal(mu_id, key, *value_, false); |
| + subscription_ = service->Subscribe( |
| + base::Bind(&ManagedUserSharedSettingsUpdate::OnSettingChanged, |
| + base::Unretained(this))); |
| +} |
| + |
| +ManagedUserSharedSettingsUpdate::~ManagedUserSharedSettingsUpdate() {} |
| + |
| +void ManagedUserSharedSettingsUpdate::OnSettingChanged(const std::string& mu_id, |
| + const std::string& key) { |
| + if (mu_id != mu_id_) |
| + return; |
| + |
| + if (key != key_) |
| + return; |
| + |
| + const base::Value* value = service_->GetValue(mu_id, key); |
| + callback_.Run(value->Equals(value_.get())); |
|
Pam (message me for reviews)
2014/01/08 10:30:40
And also make sure acknowledged = true? Or what's
Bernhard Bauer
2014/01/08 11:42:04
Yes, I meant to DCHECK that in the service.
|
| +} |