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

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

Issue 10827168: Fix user agent override restore pipeline (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Nit addressing Created 8 years, 4 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_restore.cc
diff --git a/chrome/browser/sessions/session_restore.cc b/chrome/browser/sessions/session_restore.cc
index 2906eec974ec5dbce4e54c8b362fc1957e744d8b..a25ce266d780eb6c298150c53151d98b46c9822c 100644
--- a/chrome/browser/sessions/session_restore.cc
+++ b/chrome/browser/sessions/session_restore.cc
@@ -593,7 +593,8 @@ class SessionRestoreImpl : public content::NotificationObserver {
selected_index,
true,
tab.extension_app_id,
- NULL);
+ NULL,
+ tab.user_agent_override);
} else {
int tab_index = use_new_window ? 0 : browser->active_index() + 1;
WebContents* web_contents = chrome::AddRestoredTab(
@@ -605,7 +606,8 @@ class SessionRestoreImpl : public content::NotificationObserver {
false, // selected
tab.pinned,
true,
- NULL);
+ NULL,
+ tab.user_agent_override);
// Start loading the tab immediately.
web_contents->GetController().LoadIfNecessary();
}
@@ -904,7 +906,8 @@ class SessionRestoreImpl : public content::NotificationObserver {
false, // select
tab.pinned,
true,
- session_storage_namespace.get());
+ 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());
« no previous file with comments | « no previous file | chrome/browser/sessions/session_restore_browsertest.cc » ('j') | chrome/browser/ui/tab_contents/tab_contents.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698