Add MemoryCoordinator

components/memory_coordinator/child/child_memory_coordinator_impl.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 "components/memory_coordinator/child/child_memory_coordinator_impl.h"
 
+#include "components/memory_coordinator/child/blink_memory_coordinator_client.h"
+
 namespace memory_coordinator {
 
+// static
+void ChildMemoryCoordinatorImpl::Create(
+    mojo::InterfaceRequest<mojom::ChildMemoryCoordinator> request) {
+  scoped_refptr<ClientList> clients(new ClientList);
+  new ChildMemoryCoordinatorImpl(std::move(request), clients);

[haraken, 2016/06/27 02:17:07]

Nit: new ChildMemoryCoordinatorImpl(std::move(request), (new ClientList)

[bashi, 2016/06/27 03:49:29]

Done.

+}
+
 ChildMemoryCoordinatorImpl::ChildMemoryCoordinatorImpl(
       mojo::InterfaceRequest<mojom::ChildMemoryCoordinator> request,
       scoped_refptr<ClientList> clients)
     : binding_(this, std::move(request)),
-      clients_(clients) {}
+      clients_(clients),
+      blink_client_(new BlinkMemoryCoordinatorClient) {

[haraken, 2016/06/27 02:17:07]

Do we need the member variable |blink_client_|?

[bashi, 2016/06/27 03:49:29]

Someone needs to have the ownership of BlinkMemoryCoordinatorClient because
ClientList doesn't take the ownership. I think ChildMemoryCoordinatorImpl is a
good place.

+  // TODO(bashi): Add more clients (e.g. skia).
+  clients_->AddObserver(blink_client_.get());
+}
 
 ChildMemoryCoordinatorImpl::~ChildMemoryCoordinatorImpl() {}
 
 void ChildMemoryCoordinatorImpl::OnStateChange(mojom::MemoryState state) {
+  state_ = state;
   clients_->Notify(FROM_HERE, &MemoryCoordinatorClient::OnMemoryStateChange,
                    state);
 }
 
+void ChildMemoryCoordinatorImpl::GetLastState(
+    const GetLastStateCallback& callback) {
+  callback.Run(state_);
+}
+
 }  // namespace memory_coordinator