Stop using callbacks in MemoryCoordinatorProxy

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/memory/memory_coordinator_proxy.h"
 #include "base/memory/memory_pressure_monitor.h"
 #include "base/process/process_handle.h"
 #include "content/common/content_export.h"
 #include "content/common/memory_coordinator.mojom.h"
 #include "content/public/browser/memory_coordinator_delegate.h"
 #include "mojo/public/cpp/bindings/binding.h"
 
 namespace content {
 
 // NOTE: Memory coordinator is under development and not fully working.
@@ skipping 33 matching lines @@
   // TODO(bashi): Remove this when MemoryPressureMonitor is retired.
   void RecordMemoryPressure(
       base::MemoryPressureMonitor::MemoryPressureLevel level);
 
   // Called when ChildMemoryCoordinator calls AddChild().
   virtual void OnChildAdded(int render_process_id) {}
 
   // Returns the global memory state.
   virtual base::MemoryState GetGlobalMemoryState() const;
 
-  // Returns the browser's current memory state. Note that the current state
-  // could be different from the global memory state as the browser won't be
-  // suspended.
-  virtual base::MemoryState GetCurrentMemoryState() const;
-
-  // Sets the global memory state for testing.
-  virtual void SetCurrentMemoryStateForTesting(base::MemoryState memory_state);
+  virtual base::WeakPtr<base::MemoryCoordinatorInterface> GetWeakPtr();

[chrisha, 2016/12/13 15:42:47]

Maybe call this GetBaseMemoryCoordinator?

GetWeakPtr would tend to imply that the type is WeakPtr<MemoryCoordinator>...

 
  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);
 
@@ skipping 50 matching lines @@
   ChildInfoMap children_;
 
   std::unique_ptr<MemoryCoordinatorDelegate> delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(MemoryCoordinator);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_MEMORY_MEMORY_COORDINATOR_H_