Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2361)

Unified Diff: chrome/browser/sessions/session_service.cc

Issue 1131373003: [Session restore] Add MRU logic to loading of background pages. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove unnecessary change in tab_restore_service.h Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sessions/session_service.cc
diff --git a/chrome/browser/sessions/session_service.cc b/chrome/browser/sessions/session_service.cc
index 99de7204e61bdf7d52e078b0e979d175b29f58eb..4bf7f25cdd01b6a5b4379383e937a86fed6896a4 100644
--- a/chrome/browser/sessions/session_service.cc
+++ b/chrome/browser/sessions/session_service.cc
@@ -487,6 +487,16 @@ void SessionService::SetTabExtensionAppID(
tab_id, extension_app_id).Pass());
}
+void SessionService::SetLastActiveTime(const SessionID& window_id,
+ const SessionID& tab_id,
+ base::Time last_active_time) {
+ if (!ShouldTrackChangesToWindow(window_id))
+ return;
+
+ ScheduleCommand(
+ sessions::CreateLastActiveTimeCommand(tab_id, last_active_time).Pass());
+}
+
base::CancelableTaskTracker::TaskId SessionService::GetLastSession(
const SessionCallback& callback,
base::CancelableTaskTracker* tracker) {
@@ -727,6 +737,10 @@ void SessionService::BuildCommandsForTab(const SessionID& window_id,
sessions::CreatePinnedStateCommand(session_id, true));
}
+ base_session_service_->AppendRebuildCommand(
sky 2015/05/18 15:53:28 Only if SessionRestore::SMART_RESTORE_MODE_MRU?
Georges Khalil 2015/05/19 19:32:15 Done.
+ sessions::CreateLastActiveTimeCommand(session_id,
+ tab->GetLastActiveTime()));
+
extensions::TabHelper* extensions_tab_helper =
extensions::TabHelper::FromWebContents(tab);
if (extensions_tab_helper->extension_app()) {

Powered by Google App Engine
This is Rietveld 408576698