Pref service: Filter updates from read-only pref stores.

services/preferences/public/cpp/pref_store_impl.cc

 // Copyright 2017 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 "services/preferences/public/cpp/pref_store_impl.h"
 
 #include <memory>
+#include <unordered_set>
 
+#include "base/stl_util.h"
 #include "base/values.h"
 
 namespace prefs {
 
+class PrefStoreImpl::Observer {
+ public:
+  Observer(mojom::PrefStoreObserverPtr observer,
+           std::unordered_set<std::string> prefs)
+      : observer_(std::move(observer)), prefs_(std::move(prefs)) {}
+
+  Observer(Observer&& other) = default;
+  Observer& operator=(Observer&& other) = default;
+
+  void OnInitializationCompleted(bool succeeded) {
+    observer_->OnInitializationCompleted(succeeded);
+  }
+
+  void OnPrefChanged(const std::string& key, const base::Value& value) {
+    if (!base::ContainsKey(prefs_, key))
+      return;
+
+    observer_->OnPrefChanged(key, value.CreateDeepCopy());
+  }
+
+  void OnPrefRemoved(const std::string& key) {
+    if (!base::ContainsKey(prefs_, key))
+      return;
+
+    observer_->OnPrefChanged(key, nullptr);
+  }
+
+ private:
+  mojom::PrefStoreObserverPtr observer_;
+  const std::unordered_set<std::string> prefs_;
+
+  DISALLOW_COPY_AND_ASSIGN(Observer);
+};
+
 PrefStoreImpl::PrefStoreImpl(scoped_refptr<::PrefStore> pref_store,
                              mojom::PrefStoreRequest request)
     : backing_pref_store_(std::move(pref_store)),
       backing_pref_store_initialized_(false),
       binding_(this, std::move(request)) {
   DCHECK(backing_pref_store_);
   if (backing_pref_store_->IsInitializationComplete())
     OnInitializationCompleted(true);
   backing_pref_store_->AddObserver(this);
 }
 
 PrefStoreImpl::~PrefStoreImpl() {
   backing_pref_store_->RemoveObserver(this);
 }
 
 // static
 std::unique_ptr<PrefStoreImpl> PrefStoreImpl::Create(
-    mojom::PrefStoreRegistryPtr registry_ptr,
+    mojom::PrefStoreRegistry* registry_ptr,
     scoped_refptr<::PrefStore> pref_store,
     PrefValueStore::PrefStoreType type) {
   mojom::PrefStorePtr ptr;
   auto impl = base::MakeUnique<PrefStoreImpl>(std::move(pref_store),
                                               mojo::MakeRequest(&ptr));
   registry_ptr->Register(type, std::move(ptr));
   return impl;
 }
 
 void PrefStoreImpl::OnPrefValueChanged(const std::string& key) {
   auto dictionary = base::MakeUnique<base::DictionaryValue>();
   const base::Value* value = nullptr;
   if (backing_pref_store_->GetValue(key, &value)) {
-    for (const auto& observer : observers_)
-      observer->OnPrefChanged(key, value->CreateDeepCopy());
+    for (auto& observer : observers_)

[tibell, 2017/03/27 03:37:56]

const

[Sam McNally, 2017/03/27 06:44:40]

Done.

+      observer.OnPrefChanged(key, *value);
   } else {
-    for (const auto& observer : observers_)
-      observer->OnPrefChanged(key, nullptr);
+    for (auto& observer : observers_)

[tibell, 2017/03/27 03:37:56]

const

[Sam McNally, 2017/03/27 06:44:40]

Done.

+      observer.OnPrefRemoved(key);
   }
 }
 
 void PrefStoreImpl::OnInitializationCompleted(bool succeeded) {
   // Some pref stores call this more than once. We just ignore all calls after
   // the first.
   if (backing_pref_store_initialized_)
     DCHECK(succeeded);
   backing_pref_store_initialized_ = succeeded;
-  for (const auto& observer : observers_)
-    observer->OnInitializationCompleted(succeeded);
+  for (auto& observer : observers_)

[tibell, 2017/03/27 03:37:56]

const

[Sam McNally, 2017/03/27 06:44:40]

Done.

+    observer.OnInitializationCompleted(succeeded);
 }
 
-void PrefStoreImpl::AddObserver(const AddObserverCallback& callback) {
+void PrefStoreImpl::AddObserver(
+    const std::vector<std::string>& prefs_to_observe,
+    const AddObserverCallback& callback) {
   mojom::PrefStoreObserverPtr observer_ptr;
   auto request = mojo::MakeRequest(&observer_ptr);
-  observers_.push_back(std::move(observer_ptr));
+  observers_.emplace_back(
+      std::move(observer_ptr),
+      std::unordered_set<std::string>(prefs_to_observe.begin(),
+                                      prefs_to_observe.end()));
   callback.Run(mojom::PrefStoreConnection::New(
       std::move(request), backing_pref_store_->GetValues(),
       backing_pref_store_->IsInitializationComplete()));
 }
 
 }  // namespace prefs