Ensure History > Recent Tabs restore preserves window disposition.

chrome/browser/ui/browser_live_tab_context.cc

 // 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.
 
 #include "chrome/browser/ui/browser_live_tab_context.h"
 
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_tabrestore.h"
@@ skipping 39 matching lines @@
 
 sessions::LiveTab* BrowserLiveTabContext::GetActiveLiveTab() const {
   return sessions::ContentLiveTab::GetForWebContents(
       browser_->tab_strip_model()->GetActiveWebContents());
 }
 
 bool BrowserLiveTabContext::IsTabPinned(int index) const {
   return browser_->tab_strip_model()->IsTabPinned(index);
 }
 
+const gfx::Rect BrowserLiveTabContext::GetBounds() const {
+  return browser_->window()->GetRestoredBounds();
+}
+
+ui::WindowShowState BrowserLiveTabContext::GetShowState() const {
+  return browser_->window()->GetRestoredState();
+}
+
+std::string BrowserLiveTabContext::GetWorkspace() const {
+  return browser_->window()->GetWorkspace();
+}
+
 sessions::LiveTab* BrowserLiveTabContext::AddRestoredTab(
     const std::vector<sessions::SerializedNavigationEntry>& navigations,
     int tab_index,
     int selected_navigation,
     const std::string& extension_app_id,
     bool select,
     bool pin,
     bool from_last_session,
     const sessions::PlatformSpecificTabData* tab_platform_data,
     const std::string& user_agent_override) {
@@ skipping 50 matching lines @@
   return sessions::ContentLiveTab::GetForWebContents(web_contents);
 }
 
 void BrowserLiveTabContext::CloseTab() {
   chrome::CloseTab(browser_);
 }
 
 // static
 sessions::LiveTabContext* BrowserLiveTabContext::Create(
     Profile* profile,
-    const std::string& app_name) {
-  Browser* browser;
+    const std::string& app_name,
+    const gfx::Rect& bounds,
+    ui::WindowShowState show_state,
+    const std::string& workspace) {
+  std::unique_ptr<Browser::CreateParams> create_params;
   if (app_name.empty()) {
-    browser = new Browser(Browser::CreateParams(profile, true));
+    create_params = base::MakeUnique<Browser::CreateParams>(
+        Browser::CreateParams(profile, true));
+    create_params->initial_bounds = bounds;
   } else {
     // Only trusted app popup windows should ever be restored.
-    browser = new Browser(Browser::CreateParams::CreateForApp(
-        app_name, true /* trusted_source */, gfx::Rect(), profile, true));
+    create_params = base::MakeUnique<Browser::CreateParams>(
+        Browser::CreateParams::CreateForApp(app_name, true /* trusted_source */,
+                                            bounds, profile,
+                                            true /* user_gesture */));
   }
-  if (browser)
-    return browser->live_tab_context();
-  else
-    return NULL;
+  create_params->initial_show_state = show_state;
+  create_params->initial_workspace = workspace;
+  Browser* browser = new Browser(*create_params.get());
+  return browser->live_tab_context();
 }
 
 // static
 sessions::LiveTabContext* BrowserLiveTabContext::FindContextForWebContents(
     const WebContents* contents) {
   Browser* browser = chrome::FindBrowserWithWebContents(contents);
   return browser ? browser->live_tab_context() : nullptr;
 }
 
 // static
 sessions::LiveTabContext* BrowserLiveTabContext::FindContextWithID(
     SessionID::id_type desired_id) {
   Browser* browser = chrome::FindBrowserWithID(desired_id);
   return browser ? browser->live_tab_context() : nullptr;
 }