Transfer user agent override info between browser and renderer

chrome/browser/prerender/prerender_contents.cc

 // Copyright (c) 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/prerender/prerender_contents.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/process_util.h"
@@ skipping 343 matching lines @@
       content::Source<WebContents>(new_contents));
 
   // Register for redirect notifications sourced from |this|.
   notification_registrar_.Add(
       this, content::NOTIFICATION_RESOURCE_RECEIVED_REDIRECT,
       content::Source<WebContents>(GetWebContents()));
 
   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 {
   CHECK(child_id);
   DCHECK_GE(child_id_, -1);
   *child_id = child_id_;
   return child_id_ != -1;
 }
 
 bool PrerenderContents::GetRouteId(int* route_id) const {
@@ skipping 328 matching lines @@
 bool PrerenderContents::IsCrossSiteNavigationPending() const {
   if (!prerender_contents_.get() || !prerender_contents_->web_contents())
     return false;
   const WebContents* web_contents = prerender_contents_->web_contents();
   return (web_contents->GetSiteInstance() !=
           web_contents->GetPendingSiteInstance());
 }
 
 
 }  // namespace prerender