Make HistoryBackend a client of memory coordinator

content/browser/browser_main_loop.cc

Index: content/browser/browser_main_loop.cc
diff --git a/content/browser/browser_main_loop.cc b/content/browser/browser_main_loop.cc
index 309e4eb4e5e5b5e14bfe5361916cab4e875f1020..736d6aefb727b1a3e2e1109787edb90fa0a33dd9 100644
--- a/content/browser/browser_main_loop.cc
+++ b/content/browser/browser_main_loop.cc
@@ -715,9 +715,8 @@ int BrowserMainLoop::PreCreateThreads() {
       parsed_command_line_));
 #endif
 
-  if (memory_coordinator::IsEnabled()) {
-    memory_coordinator_.reset(new memory_coordinator::MemoryCoordinator);
-  }
+  // Make sure memory coordinator is initialized.
+  memory_coordinator::MemoryCoordinator::GetInstance();

[jam, 2016/08/23 17:37:16]

why is this needed? the constructor of MC doesn't do anything, so why not just
have it instantiated by whoever first uses it?

[bashi, 2016/08/24 23:33:28]

Done.

 
 #if defined(ENABLE_PLUGINS)
   // Prior to any processing happening on the IO thread, we create the
@@ -1020,7 +1019,6 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() {
   }
 
   memory_pressure_monitor_.reset();
-  memory_coordinator_.reset();
 
 #if defined(OS_MACOSX)
   BrowserCompositorMac::DisableRecyclingForShutdown();