TabManagerDelegate: Better prioritize ARC processes.

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..b4185c254d7e070a005328a36464e65dd828ec8a 100644
--- a/chrome/browser/memory/tab_manager.h
+++ b/chrome/browser/memory/tab_manager.h
@@ -128,6 +128,10 @@ class TabManager : public TabStripModelObserver {
   void AddObserver(TabManagerObserver* observer);
   void RemoveObserver(TabManagerObserver* observer);
 
+  // Returns true if |first| is considered less desirable to be killed than
+  // |second|.
+  static bool CompareTabStats(TabStats first, TabStats second);

[Yusuke Sato, 2016/07/15 00:55:05]

I know this is not your code, but isn't it better to use

 const TabStats& 

? The struct is large.

[cylee1, 2016/07/15 17:31:13]

Done.

+
  private:
   FRIEND_TEST_ALL_PREFIXES(TabManagerTest, CanOnlyDiscardOnce);
   FRIEND_TEST_ALL_PREFIXES(TabManagerTest, ChildProcessNotifications);
@@ -244,10 +248,6 @@ class TabManager : public TabStripModelObserver {
   // creating one if needed.
   WebContentsData* GetWebContentsData(content::WebContents* contents) const;
 
-  // Returns true if |first| is considered less desirable to be killed than
-  // |second|.
-  static bool CompareTabStats(TabStats first, TabStats second);
-
   // Returns either the system's clock or the test clock. See |test_tick_clock_|
   // for more details.
   base::TimeTicks NowTicks() const;