Ensure History > Recent Tabs restore preserves window disposition.

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

Index: chrome/browser/ui/android/tab_model/android_live_tab_context.cc
diff --git a/chrome/browser/ui/android/tab_model/android_live_tab_context.cc b/chrome/browser/ui/android/tab_model/android_live_tab_context.cc
index 73740024ea894e4a9d2b34dda825fe2274f40e5f..37c849f28ad0f000135d443f45604f01726d1624 100644
--- a/chrome/browser/ui/android/tab_model/android_live_tab_context.cc
+++ b/chrome/browser/ui/android/tab_model/android_live_tab_context.cc
@@ -32,8 +32,8 @@ int AndroidLiveTabContext::GetSelectedIndex() const {
   return tab_model_->GetActiveIndex();
 }
 
-// Not supported by android.
 std::string AndroidLiveTabContext::GetAppName() const {
+  // Not applicable to android.
   return std::string();
 }
 
@@ -54,11 +54,26 @@ sessions::LiveTab* AndroidLiveTabContext::GetActiveLiveTab() const {
   return sessions::ContentLiveTab::GetForWebContents(web_contents);
 }
 
-// Not supported by android.
 bool AndroidLiveTabContext::IsTabPinned(int index) const {
+  // Not applicable to android.

[sky, 2017/07/11 20:28:09]

I like this better that you have the comment inside the function. Add
NOTREACHED() where applicable.

[chrisha, 2017/08/14 14:43:21]

Done.

   return false;
 }
 
+const gfx::Rect AndroidLiveTabContext::GetRestoredBounds() const {
+  // Not applicable to android.
+  return gfx::Rect();
+}
+
+ui::WindowShowState AndroidLiveTabContext::GetRestoredState() const {
+  // Not applicable to android.
+  return ui::SHOW_STATE_NORMAL;
+}
+
+std::string AndroidLiveTabContext::GetWorkspace() const {
+  // Not applicable to android.
+  return std::string();
+}
+
 sessions::LiveTab* AndroidLiveTabContext::AddRestoredTab(
     const std::vector<sessions::SerializedNavigationEntry>& navigations,
     int tab_index,