Discardable property support on TabManager

chrome/browser/memory/tab_manager.h

Index: chrome/browser/memory/tab_manager.h
diff --git a/chrome/browser/memory/tab_manager.h b/chrome/browser/memory/tab_manager.h
index cd5c3480417bf76976c8a3849ae4d0e5fe6c5057..99e8daec6b8ffaebf7e3f8d7ebaf042e0a013c8b 100644
--- a/chrome/browser/memory/tab_manager.h
+++ b/chrome/browser/memory/tab_manager.h
@@ -129,6 +129,7 @@ class TabManager : public TabStripModelObserver {
   void RemoveObserver(TabManagerObserver* observer);
 
  private:
+  FRIEND_TEST_ALL_PREFIXES(TabManagerObserverTest, OnDiscardableStateChange);

[Georges Khalil, 2016/07/21 20:51:51]

Add Is/Set TabDiscardable functions, and no need for friend anymore

[Anderson Silva, 2016/07/21 21:14:49]

Done.

   FRIEND_TEST_ALL_PREFIXES(TabManagerTest, CanOnlyDiscardOnce);
   FRIEND_TEST_ALL_PREFIXES(TabManagerTest, ChildProcessNotifications);
   FRIEND_TEST_ALL_PREFIXES(TabManagerTest, Comparator);
@@ -151,6 +152,11 @@ class TabManager : public TabStripModelObserver {
   void OnDiscardedStateChange(content::WebContents* contents,
                               bool is_discarded);
 
+  // Called by WebContentsData whenever the discardable state of a WebContents
+  // changes, so that observers can be informed.
+  void OnDiscardableStateChange(content::WebContents* contents,
+                                bool is_discardable);
+
   // The time that a renderer is given to react to a memory pressure
   // notification before another renderer is also notified. This prevents all
   // renderers from receiving and acting upon notifications simultaneously,