[Session restore] Add MRU logic to loading of background pages.

chrome/browser/sessions/session_restore.cc

 // Copyright 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 <algorithm>
 #include <list>
 #include <set>
 #include <string>
@@ skipping 563 matching lines @@
 
     // Associate sessionStorage (if any) to the restored tab.
     scoped_refptr<content::SessionStorageNamespace> session_storage_namespace;
     if (!tab.session_storage_persistent_id.empty()) {
       session_storage_namespace =
           content::BrowserContext::GetDefaultStoragePartition(profile_)
               ->GetDOMStorageContext()
               ->RecreateSessionStorage(tab.session_storage_persistent_id);
     }
 
     WebContents* web_contents = chrome::AddRestoredTab(

[sky, 2015/05/18 15:53:28]

Might this set last active time if the tab is selected?

[Georges Khalil, 2015/05/19 19:32:15]

True, if the tab is selected, we do not want to override the last active time.
Added a check at line 598. Also added a check to make sure time is not in the
future.

         browser, tab.navigations, tab_index, selected_index,
         tab.extension_app_id,
         false,  // select
         tab.pinned, true, session_storage_namespace.get(),
         tab.user_agent_override);
     // Regression check: check that the tab didn't start loading right away. The
     // focused tab will be loaded by Browser, and TabLoader will load the rest.
     DCHECK(web_contents->GetController().NeedsReload());
 
+    // Restore the saved last active time. By default, the last active time of a
+    // WebContent is initially set to the creation time of the tab, which is not
+    // necessarly the same as the loading time. This makes sure that the state
+    // is preserved between restores.
+    web_contents->SetLastActiveTime(tab.last_active_time);
+
     return web_contents;
   }
 
   Browser* CreateRestoredBrowser(Browser::Type type,
                                  gfx::Rect bounds,
                                  ui::WindowShowState show_state,
                                  const std::string& app_name) {
     Browser::CreateParams params(type, profile_, host_desktop_type_);
     if (!app_name.empty()) {
       const bool trusted_source = true;  // We only store trusted app windows.
@@ skipping 221 matching lines @@
   if (prioritize_tabs == "mru")
     return SMART_RESTORE_MODE_MRU;
   if (prioritize_tabs == "simple")
     return SMART_RESTORE_MODE_SIMPLE;
   return SMART_RESTORE_MODE_OFF;
 }
 
 // static
 base::CallbackList<void(int)>*
     SessionRestore::on_session_restored_callbacks_ = nullptr;