Sessions API - previously Session Restore API. Supports restoring currently open foreign windows an…

chrome/browser/sessions/tab_restore_service.h

Index: chrome/browser/sessions/tab_restore_service.h
diff --git a/chrome/browser/sessions/tab_restore_service.h b/chrome/browser/sessions/tab_restore_service.h
index e230deffff15d28719367ec4008b174bf2bf5ef9..ff2fd87ffe6aa30596425b1921a88ab9644b981c 100644
--- a/chrome/browser/sessions/tab_restore_service.h
+++ b/chrome/browser/sessions/tab_restore_service.h
@@ -155,8 +155,11 @@ class TabRestoreService : public BrowserContextKeyedService {
   // Restores the most recently closed entry. Does nothing if there are no
   // entries to restore. If the most recently restored entry is a tab, it is
   // added to |delegate|. If a new browser needs to be created for this entry,
-  // it will be created on the desktop specified by |host_desktop_type|.
-  virtual void RestoreMostRecentEntry(
+  // it will be created on the desktop specified by |host_desktop_type|. Returns
+  // the WebContents of the restored tab if entry is a Tab. In the case of
+  // restoring type Window, returns the WebContents of the last successfully

[sky, 2013/08/01 00:05:43]

Return value doesn't make sense here. SHouldn't this be a vector? If the caller
only cares about the last restored tab in the case of a window it can use
back().

[Kristen Dwan, 2013/08/02 20:54:19]

Done.

+  // restored tab in the window.
+  virtual content::WebContents* RestoreMostRecentEntry(
       TabRestoreServiceDelegate* delegate,
       chrome::HostDesktopType host_desktop_type) = 0;
 
@@ -170,10 +173,14 @@ class TabRestoreService : public BrowserContextKeyedService {
   // browser window) of the tab when it was closed will be respected if
   // disposition is UNKNOWN. If a new browser needs to be created for this
   // entry, it will be created on the desktop specified by |host_desktop_type|.
-  virtual void RestoreEntryById(TabRestoreServiceDelegate* delegate,
-                                SessionID::id_type id,
-                                chrome::HostDesktopType host_desktop_type,
-                                WindowOpenDisposition disposition) = 0;
+  // Returns the WebContents of the restored tab if entry is a Tab. In the case
+  // of restoring type Window, returns the WebContents of the last successfully

[sky, 2013/08/01 00:05:43]

Same comment.

[Kristen Dwan, 2013/08/02 20:54:19]

Done.

+  // restored tab in the window.
+  virtual content::WebContents* RestoreEntryById(
+      TabRestoreServiceDelegate* delegate,
+      SessionID::id_type id,
+      chrome::HostDesktopType host_desktop_type,
+      WindowOpenDisposition disposition) = 0;
 
   // Loads the tabs and previous session. This does nothing if the tabs
   // from the previous session have already been loaded.