Carry over user-agent override when opening new windows

content/renderer/render_frame_impl.cc

 // Copyright 2013 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 "content/renderer/render_frame_impl.h"
 
 #include <map>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 2607 matching lines @@
   child_render_frame->in_frame_tree_ = true;
   child_render_frame->Initialize();
 
   return web_frame;
 }
 
 void RenderFrameImpl::didChangeOpener(blink::WebFrame* opener) {
   // Only a local frame should be able to update another frame's opener.
   DCHECK(!opener || opener->isWebLocalFrame());
 
+  // Carry over the user agent override flag, if it exists.
+  blink::WebView* webview = render_view_->webview();

[dcheng, 2016/03/31 05:24:18]

This doesn't get called at all when creating a new window:
RenderViewImpl::createView() and RenderFrameImpl::CreateMainFrame() is where
that all happens.

+  if (opener && !pending_navigation_params_.get() && webview &&
+      webview->mainFrame() && webview->mainFrame()->isWebLocalFrame() &&
+      webview->mainFrame()->dataSource()) {
+    if (webview->mainFrame()->dataSource()->request().url().isEmpty() &&
+        opener->dataSource()) {
+      DocumentState* old_document_state =
+          DocumentState::FromDataSource(opener->dataSource());
+      InternalDocumentStateData* internal_data =
+          InternalDocumentStateData::FromDocumentState(
+              DocumentState::FromDataSource(
+                  webview->mainFrame()->dataSource()));
+      InternalDocumentStateData* old_internal_data =
+          InternalDocumentStateData::FromDocumentState(old_document_state);
+      internal_data->set_is_overriding_user_agent(
+          old_internal_data->is_overriding_user_agent());
+    }
+  }
+
   int opener_routing_id = opener ?
       RenderFrameImpl::FromWebFrame(opener->toWebLocalFrame())->GetRoutingID() :
       MSG_ROUTING_NONE;
   Send(new FrameHostMsg_DidChangeOpener(routing_id_, opener_routing_id));
 }
 
 void RenderFrameImpl::frameDetached(blink::WebFrame* frame, DetachType type) {
   // NOTE: This function is called on the frame that is being detached and not
   // the parent frame.  This is different from createChildFrame() which is
   // called on the parent frame.
@@ skipping 3375 matching lines @@
                                     int match_count,
                                     int ordinal,
                                     const WebRect& selection_rect,
                                     bool final_status_update) {
   Send(new FrameHostMsg_Find_Reply(routing_id_, request_id, match_count,
                                    selection_rect, ordinal,
                                    final_status_update));
 }
 
 }  // namespace content