Changes session restore callback to always happen when tabs are created

chrome/browser/sessions/session_restore.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_SESSIONS_SESSION_RESTORE_H_
 #define CHROME_BROWSER_SESSIONS_SESSION_RESTORE_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 73 matching lines @@
       content::WebContents* source_web_contents,
       const sessions::SessionTab& tab,
       WindowOpenDisposition disposition);
 
   // Returns true if we're in the process of restoring |profile|.
   static bool IsRestoring(const Profile* profile);
 
   // Returns true if synchronously restoring a session.
   static bool IsRestoringSynchronously();
 
-  // Register callbacks for session restore events. These callbacks are stored
-  // in |on_session_restored_callbacks_|.
-  // The callback is supplied an integer arg representing a tab count. The exact
-  // meaning and timing depend upon the restore type:
-  // - SessionRestore::SYNCHRONOUS: the parameter is the number of tabs that
-  // were created. Additionally the callback is invoked immediately after the
-  // tabs have been created. That is, the tabs are not necessarily loading.
-  // - For all other restore types the parameter is the number of tabs that were
-  // restored and is sent after all tabs have started loading. Additionally if a
-  // request to restore tabs comes in while a previous request to restore tabs
-  // has not yet completed (loading tabs is throttled), then the callback is
-  // only notified once both sets of tabs have started loading and with the
-  // total number of tabs for both restores.
+  // Registers a callback that is notified every time session restore completes.
+  // Note that 'complete' means all the browsers and tabs have been created but
+  // have not necessarily finished loading. The integer supplied to the callback
+  // indicates the number of tabs that were created.
   static CallbackSubscription RegisterOnSessionRestoredCallback(
       const base::Callback<void(int)>& callback);
 
   // The max number of non-selected tabs SessionRestore loads when restoring
   // a session. A value of 0 indicates all tabs are loaded at once.
   static size_t num_tabs_to_load_;
 
  private:
   SessionRestore();
 
   // Accessor for |*on_session_restored_callbacks_|. Creates a new object the
   // first time so that it always returns a valid object.
   static CallbackList* on_session_restored_callbacks() {
     if (!on_session_restored_callbacks_)
       on_session_restored_callbacks_ = new CallbackList();
     return on_session_restored_callbacks_;
   }
 
   // Contains all registered callbacks for session restore notifications.
   static CallbackList* on_session_restored_callbacks_;
 
   DISALLOW_COPY_AND_ASSIGN(SessionRestore);
 };
 
 #endif  // CHROME_BROWSER_SESSIONS_SESSION_RESTORE_H_