Move memory_coordinator_client and its registry to base/memory

base/memory/memory_coordinator_client_registry.cc

+// Copyright 2016 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 "base/memory/memory_coordinator_client_registry.h"
+
+namespace base {
+
+// static
+MemoryCoordinatorClientRegistry*
+MemoryCoordinatorClientRegistry::GetInstance() {
+  return Singleton<
+      MemoryCoordinatorClientRegistry,
+      LeakySingletonTraits<MemoryCoordinatorClientRegistry>>::get();
+}
+
+MemoryCoordinatorClientRegistry::MemoryCoordinatorClientRegistry()
+    : clients_(new ClientList) {}

[dcheng, 2016/09/08 21:28:26]

Why do we need to store this in a scoped_refptr?

[bashi, 2016/09/08 23:00:08]

ObserverListThreadSafe is RefCountedThreadSafe. Can we put it in unique_ptr?

[dcheng, 2016/09/09 03:02:04]

Ah, I see... it's thread-safe so it can PostTask and do cross-thread
notifications.

+
+MemoryCoordinatorClientRegistry::~MemoryCoordinatorClientRegistry() {}
+
+void MemoryCoordinatorClientRegistry::Register(
+    MemoryCoordinatorClient* client) {
+  clients_->AddObserver(client);
+}
+
+void MemoryCoordinatorClientRegistry::Unregister(
+    MemoryCoordinatorClient* client) {
+  clients_->RemoveObserver(client);
+}
+
+}  // namespace base