Pass rendering state on swapping in a prerendered page.

chrome/browser/sessions/session_restore_android.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/sessions/session_restore.h"
 
 #include <vector>
 
 #include "chrome/browser/android/tab_android.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 24 matching lines @@
       content::WebContents::CreateParams(context));
   int selected_index = session_tab.normalized_navigation_index();
   new_web_contents->GetController().Restore(
       selected_index,
       content::NavigationController::RESTORE_LAST_SESSION_EXITED_CLEANLY,
       &entries);
 
   TabAndroid* current_tab = TabAndroid::FromWebContents(web_contents);
   DCHECK(current_tab);
   if (disposition == CURRENT_TAB) {
-    current_tab->SwapTabContents(web_contents, new_web_contents);
+    current_tab->SwapTabContents(web_contents, new_web_contents, false);

[Ted C, 2014/01/28 18:11:44]

Previously, this wasn't resulting in a page load started event being triggered
in the downstream code.  You might want to double check that when restoring tabs
nothing bad happens.

We might need to add a distinct call for this action that only does the old
functionality.

[Shuhei Takahashi (google), 2014/01/29 05:58:10]

Added another boolean flag didStartLoad to distinguish two cases.

     delete web_contents;
   } else {
     DCHECK(disposition == NEW_FOREGROUND_TAB ||
         disposition == NEW_BACKGROUND_TAB);
     tab_model->CreateTab(new_web_contents, current_tab->GetAndroidId());
   }
   return new_web_contents;
 }
 
 // static
 std::vector<Browser*> SessionRestore::RestoreForeignSessionWindows(
     Profile* profile,
     chrome::HostDesktopType host_desktop_type,
     std::vector<const SessionWindow*>::const_iterator begin,
     std::vector<const SessionWindow*>::const_iterator end) {
   NOTREACHED();
   return std::vector<Browser*>();
 }