Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3150)

Unified Diff: chrome/browser/sessions/session_restore.h

Issue 21022018: Sessions API - previously Session Restore API. Supports restoring currently open foreign windows an… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added assert true to test Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sessions/session_restore.h
diff --git a/chrome/browser/sessions/session_restore.h b/chrome/browser/sessions/session_restore.h
index cda431a8807c2adb0fe2551eb65182ed8fd1e0f5..aff16ed259d20f5e7e6828971f16c20e19c7d9b5 100644
--- a/chrome/browser/sessions/session_restore.h
+++ b/chrome/browser/sessions/session_restore.h
@@ -52,10 +52,11 @@ class SessionRestore {
uint32 behavior,
const std::vector<GURL>& urls_to_open);
- // Specifically used in the restoration of a foreign session. This method
+ // Specifically used in the restoration of a foreign session. This function
// restores the given session windows to multiple browsers all of which
- // will be created on the desktop specified by |host_desktop_type|.
- static void RestoreForeignSessionWindows(
+ // will be created on the desktop specified by |host_desktop_type|. The
+ // created Browsers are returned to identify the restored Windows.
sky 2013/08/01 00:05:43 You don't need the last part: 'Returns the created
Kristen Dwan 2013/08/02 20:54:19 Done.
+ static std::vector<Browser*> RestoreForeignSessionWindows(
Profile* profile,
chrome::HostDesktopType host_desktop_type,
std::vector<const SessionWindow*>::const_iterator begin,
@@ -63,8 +64,9 @@ class SessionRestore {
// Specifically used in the restoration of a foreign session. This method
// restores the given session tab to the browser of |source_web_contents| if
- // the disposition is not NEW_WINDOW.
- static void RestoreForeignSessionTab(
+ // the disposition is not NEW_WINDOW. Returns the WebContents corresponding
+ // to the restored tab.
+ static content::WebContents* RestoreForeignSessionTab(
content::WebContents* source_web_contents,
const SessionTab& tab,
WindowOpenDisposition disposition);

Powered by Google App Engine
This is Rietveld 408576698