Stop using callbacks in MemoryCoordinatorProxy

content/browser/memory/memory_coordinator.h

Index: content/browser/memory/memory_coordinator.h
diff --git a/content/browser/memory/memory_coordinator.h b/content/browser/memory/memory_coordinator.h
index 9ee0512172fc74fdc04d380115b0093323f2330d..5f42aab99ab73b2943e126afd8602d476047b575 100644
--- a/content/browser/memory/memory_coordinator.h
+++ b/content/browser/memory/memory_coordinator.h
@@ -6,6 +6,7 @@
 #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"
@@ -59,13 +60,7 @@ class CONTENT_EXPORT MemoryCoordinator {
   // 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