Ensure History > Recent Tabs restore preserves window disposition.

chrome/browser/ui/android/tab_model/android_live_tab_context.cc

 // Copyright 2016 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/android/tab_android.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/android/tab_model/android_live_tab_context.h"
 #include "chrome/browser/ui/android/tab_model/tab_model.h"
 #include "chrome/browser/ui/android/tab_model/tab_model_list.h"
 #include "components/sessions/content/content_live_tab.h"
@@ skipping 41 matching lines @@
     return nullptr;
 
   return sessions::ContentLiveTab::GetForWebContents(web_contents);
 }
 
 // Not supported by android.
 bool AndroidLiveTabContext::IsTabPinned(int index) const {
   return false;
 }
 
+// Not supported by android.

[sky, 2017/05/12 02:42:06]

supported->applicable to?

[chrisha, 2017/06/28 18:30:42]

I was copy-pasting similar comments from in this file, but agreed "not
applicable to" is more precise. Changed this and the other comments.

+const gfx::Rect AndroidLiveTabContext::GetBounds() const {
+  return gfx::Rect();
+}
+
+// Not supported by android.
+ui::WindowShowState AndroidLiveTabContext::GetShowState() const {
+  return ui::SHOW_STATE_NORMAL;
+}
+
+// Not supported by android.
+std::string AndroidLiveTabContext::GetWorkspace() const {
+  return "";

[sky, 2017/05/12 02:42:06]

"" -> std::string()

[chrisha, 2017/06/28 18:30:42]

Done.

+}
+
 sessions::LiveTab* AndroidLiveTabContext::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 57 matching lines @@
     for (auto it = TabModelList::begin(); it != TabModelList::end(); ++it) {
       TabModel* model = *it;
       if (!model->IsOffTheRecord()) {
         return model->GetLiveTabContext();
       }
     }
   }
 
   return tab_model ? tab_model->GetLiveTabContext() : nullptr;
 }