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

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

Issue 392293002: Set the web content's initial size only if the tab isn't selected in session restore. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sessions/session_restore.cc
diff --git a/chrome/browser/sessions/session_restore.cc b/chrome/browser/sessions/session_restore.cc
index ef0a860a15f4586524ea14cdef4ff7424f1a83cc..37b1f7e62865714132bd3e89128e8ab7767e6535 100644
--- a/chrome/browser/sessions/session_restore.cc
+++ b/chrome/browser/sessions/session_restore.cc
@@ -1044,14 +1044,14 @@ class SessionRestoreImpl : public content::NotificationObserver {
tab_index,
selected_index,
tab.extension_app_id,
- false, // select
+ !schedule_load, // select
oshima 2014/07/17 22:22:21 this value is computed from selected_tab_index her
sky 2014/07/18 15:42:10 I think selected would be clearer here as well as
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());
+ DCHECK(!schedule_load || web_contents->GetController().NeedsReload());
// Set up the file access rights for the selected navigation entry.
const int id = web_contents->GetRenderProcessHost()->GetID();
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698