Transfer user agent override info between browser and renderer

chrome/browser/prerender/prerender_contents.cc

Index: chrome/browser/prerender/prerender_contents.cc
diff --git a/chrome/browser/prerender/prerender_contents.cc b/chrome/browser/prerender/prerender_contents.cc
index 9f70bfe78854bd5748ac59d08d659704a7b21e18..d4ae79c5127912d274a3280ce5e9febf7b501d81 100644
--- a/chrome/browser/prerender/prerender_contents.cc
+++ b/chrome/browser/prerender/prerender_contents.cc
@@ -361,12 +361,17 @@ void PrerenderContents::StartPrerendering(
   DCHECK(load_start_time_.is_null());
   load_start_time_ = base::TimeTicks::Now();
 
-  new_contents->GetController().LoadURL(
+  // Transfer over the user agent override.
+  new_contents->SetUserAgentOverride(
+      prerender_manager_->config().user_agent_override);
+
+  new_contents->GetController().LoadURLWithUserAgentOverride(

[jochen (gone - plz use gerrit), 2012/05/31 14:09:16]

why is setting the override and then using LoadURL not enough?

[gone, 2012/05/31 21:43:25]

The override string is stored for all NavigationEntries owned by the
TabContents, while the NavigationEntry stores a bool saying whether or not it
needs the user agent to be overridden for it.  The relevant changes landed in
http://codereview.chromium.org/10170016/

       prerender_url_,
       referrer_,
       (origin_ == ORIGIN_OMNIBOX ? content::PAGE_TRANSITION_TYPED :
                                    content::PAGE_TRANSITION_LINK),
-      std::string());
+      std::string(),
+      prerender_manager_->config().is_overriding_user_agent);
 }
 
 bool PrerenderContents::GetChildId(int* child_id) const {