Add MemoryCoordinator::SetClientMemoryState and GetClientMemoryState

content/browser/memory/memory_coordinator.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 "content/browser/memory/memory_coordinator.h"
 
 #include "base/memory/memory_coordinator_client_registry.h"
 #include "content/public/common/content_features.h"
 
 namespace content {
@@ skipping 23 matching lines @@
 };
 
 // static
 MemoryCoordinator* MemoryCoordinator::GetInstance() {
   if (!base::FeatureList::IsEnabled(features::kMemoryCoordinator))
     return nullptr;
   return base::Singleton<MemoryCoordinator,
                          base::LeakySingletonTraits<MemoryCoordinator>>::get();
 }
 
-MemoryCoordinator::MemoryCoordinator() {}
+MemoryCoordinator::MemoryCoordinator()
+  : client_memory_state_(base::MemoryState::NORMAL) {}
 
 MemoryCoordinator::~MemoryCoordinator() {}
 
 void MemoryCoordinator::CreateHandle(
     int render_process_id,
     mojom::MemoryCoordinatorHandleRequest request) {
   std::unique_ptr<MemoryCoordinatorHandleImpl> handle(
       new MemoryCoordinatorHandleImpl(std::move(request)));
   handle->binding().set_connection_error_handler(
       base::Bind(&MemoryCoordinator::OnConnectionError, base::Unretained(this),
@@ skipping 19 matching lines @@
   // A nop doesn't need to be sent, but is considered successful.
   if (iter->second.memory_state == memory_state)
     return true;
 
   // Update the internal state and send the message.
   iter->second.memory_state = memory_state;
   iter->second.handle->child()->OnStateChange(memory_state);
   return true;
 }
 
+void MemoryCoordinator::SetClientMemoryState(base::MemoryState memory_state) {

[haraken, 2016/10/12 08:11:45]

Who are you assuming calls this method?

Also Chris' proposal is to introduce MemoryCoordinatorProxy, right?

I might want to see a proof-of-concept CL that implements MCP and changes MC and
CMC using MCP. Also we need to add tests.

+  client_memory_state_ = memory_state;
+  base::MemoryCoordinatorClientRegistry::GetInstance()->Notify(memory_state);
+}
+
 mojom::MemoryState MemoryCoordinator::GetMemoryState(
     int render_process_id) const {
   auto iter = children_.find(render_process_id);
   if (iter == children_.end())
     return mojom::MemoryState::UNKNOWN;
   return iter->second.memory_state;
 }
 
+base::MemoryState MemoryCoordinator::GetClientMemoryState() const {
+  return client_memory_state_;
+}
+
 void MemoryCoordinator::AddChildForTesting(
     int dummy_render_process_id, mojom::ChildMemoryCoordinatorPtr child) {
   mojom::MemoryCoordinatorHandlePtr mch;
   auto request = mojo::GetProxy(&mch);
   std::unique_ptr<MemoryCoordinatorHandleImpl> handle(
       new MemoryCoordinatorHandleImpl(std::move(request)));
   handle->AddChild(std::move(child));
   CreateChildInfoMapEntry(dummy_render_process_id, std::move(handle));
 }
 
@@ skipping 14 matching lines @@
 
 MemoryCoordinator::ChildInfo::ChildInfo() {}
 
 MemoryCoordinator::ChildInfo::ChildInfo(const ChildInfo& rhs) {
   // This is a nop, but exists for compatibility with STL containers.
 }
 
 MemoryCoordinator::ChildInfo::~ChildInfo() {}
 
 }  // namespace content