[TabManager] Start function refactoring.

chrome/browser/memory/tab_manager.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_MEMORY_TAB_MANAGER_H_
 #define CHROME_BROWSER_MEMORY_TAB_MANAGER_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 59 matching lines @@
 
   TabManager();
   ~TabManager() override;
 
   // Number of discard events since Chrome started.
   int discard_count() const { return discard_count_; }
 
   // See member comment.
   bool recent_tab_discard() const { return recent_tab_discard_; }
 
+  // Check if tabs can be discarded multiple times.
+  bool CheckMultipleDiscards();

[Georges Khalil, 2016/05/16 18:41:22]

This should be private (don't forget to reorder the implementation as well).

nit: Change function name to AllowMultipleDiscards.
nit: Change comment to "Return true is tabs can be discarded more than once."

[Anderson Silva, 2016/05/16 19:39:06]

Acknowledged.

+
   // Start/Stop the Tab Manager.
   void Start();
   void Stop();
 
   // Returns the list of the stats for all renderers. Must be called on the UI
   // thread. The returned list is sorted by reversed importance.
   TabStatsList GetTabStats();
 
   // Returns a sorted list of renderers, from most important to least important.
   std::vector<content::RenderProcessHost*> GetOrderedRenderers();
@@ skipping 236 matching lines @@
 
   // Weak pointer factory used for posting delayed tasks to task_runner_.
   base::WeakPtrFactory<TabManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TabManager);
 };
 
 }  // namespace memory
 
 #endif  // CHROME_BROWSER_MEMORY_TAB_MANAGER_H_