Make session restore forced tab load delay Finch configurable, and separate first tab loads from su…

chrome/browser/sessions/tab_loader.h

 // Copyright 2015 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_SESSIONS_TAB_LOADER_H_
 #define CHROME_BROWSER_SESSIONS_TAB_LOADER_H_
 
 #include <list>
 #include <set>
 
@@ skipping 51 matching lines @@
   ~TabLoader() override;
 
   // This is invoked once by RestoreTabs to start loading.
   void StartLoading(const std::vector<RestoredTab>& tabs);
 
   // Loads the next tab. If there are no more tabs to load this deletes itself,
   // otherwise |force_load_timer_| is restarted.
   void LoadNextTab();
 
   // Starts a timer to load load the next tab once expired before the current
-  // tab loading is finished.
+  // tab loading is finished. The first timer is used only once when loading the
+  // first batch of visible tabs.
+  void StartFirstTimer();

[gab, 2015/05/15 18:21:03]

I think it's more readable if we separate these two and their respective
comments.

[Nico, 2015/05/15 18:36:32]

+1

[chrisha, 2015/05/15 18:47:12]

Done.

   void StartTimer();
 
   // Removes the listeners from the specified tab and removes the tab from
   // the set of tabs to load and list of tabs we're waiting to get a load
   // from.
   void RemoveTab(content::NavigationController* controller);
 
   // Invoked from |force_load_timer_|. Doubles |force_load_delay_multiplier_|
   // and invokes |LoadNextTab| to load the next tab
   void ForceLoadTimerFired();
@@ skipping 42 matching lines @@
   // For keeping TabLoader alive while it's loading even if no
   // SessionRestoreImpls reference it.
   scoped_refptr<TabLoader> this_retainer_;
 
   static TabLoader* shared_tab_loader_;
 
   DISALLOW_COPY_AND_ASSIGN(TabLoader);
 };
 
 #endif  // CHROME_BROWSER_SESSIONS_TAB_LOADER_H_