Add MemoryCoordinatorDelegate

chrome/browser/memory/tab_manager.cc

Index: chrome/browser/memory/tab_manager.cc
diff --git a/chrome/browser/memory/tab_manager.cc b/chrome/browser/memory/tab_manager.cc
index 582b2e667099f6fa0085f6721c4b9c69a37709c8..9b91aa526398a6d0b9006f23db2f4e69a7a2a264 100644
--- a/chrome/browser/memory/tab_manager.cc
+++ b/chrome/browser/memory/tab_manager.cc
@@ -211,6 +211,7 @@ void TabManager::Start() {
     }
   }
 #endif
+  MemoryCoordinatorDelegate::Set(weak_ptr_factory_.GetWeakPtr());
 }
 
 void TabManager::Stop() {
@@ -418,6 +419,17 @@ void TabManager::SetTabAutoDiscardableState(content::WebContents* contents,
   GetWebContentsData(contents)->SetAutoDiscardableState(state);
 }
 
+bool TabManager::CanSuspendRenderer(int render_process_id) {
+  auto tab_stats = GetUnsortedTabStats();
+  for (auto& tab : tab_stats) {
+    if (tab.child_process_host_id != render_process_id)
+      continue;
+    if (!CanDiscardTab(tab.tab_contents_id))

[chrisha, 2016/10/10 23:25:08]

Discard and suspend have very different semantics. We should be able to suspend
anything that doesn't have a "foreground" presence (ie, casting, playing audio,
etc).

Take a look at tasak's first stab at this here:

https://codereview.chromium.org/2389683002/

[bashi, 2016/10/11 01:10:06]

Agreed. Changed to check IsMediaTab().

+      return false;
+  }
+  return true;
+}
+
 // static
 bool TabManager::CompareTabStats(const TabStats& first,
                                  const TabStats& second) {