WIP: Add MemoryCoordinator::GetCurrentMemoryState

content/browser/memory/memory_coordinator.h

 // 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.
 
 #ifndef CONTENT_BROWSER_MEMORY_MEMORY_COORDINATOR_H_
 #define CONTENT_BROWSER_MEMORY_MEMORY_COORDINATOR_H_
 
 #include "base/memory/memory_coordinator_client_registry.h"
 #include "base/process/process_handle.h"
 #include "content/common/content_export.h"
@@ skipping 34 matching lines @@
   bool SetMemoryState(
       int render_process_id, mojom::MemoryState memory_state);
 
   // Returns the memory state of the specified render process. Returns UNKNOWN
   // if the process is not tracked by this coordinator.
   mojom::MemoryState GetMemoryState(int render_process_id) const;
 
   // Called when ChildMemoryCoordinator calls AddChild().
   virtual void OnChildAdded(int render_process_id) {}
 
+  virtual base::MemoryState GetCurrentState() const = 0;

[haraken, 2016/10/13 09:27:20]

GetCurrentMemoryState

[bashi, 2016/10/13 23:16:34]

nit: I'd prefer to return UNKNOWN so that TestMemoryCoordinator donesn't have to
override it.

[hajimehoshi, 2016/10/14 07:37:47]

Done.

+
  protected:
   // Constructor. Protected as this is a singleton, but accessible for
   // unittests.
   MemoryCoordinator();
 
   // Adds the given ChildMemoryCoordinator as a child of this coordinator.
   void AddChildForTesting(int dummy_render_process_id,
                           mojom::ChildMemoryCoordinatorPtr child);
 
   // Callback invoked by mojo when the child connection goes down. Exposed
@@ skipping 26 matching lines @@
   // MemoryCoordinator and removed automatically when an underlying binding is
   // disconnected.
   ChildInfoMap children_;
 
   DISALLOW_COPY_AND_ASSIGN(MemoryCoordinator);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_MEMORY_MEMORY_COORDINATOR_H_