OOPIF: Replicate allowFullscreen flag.

content/browser/frame_host/render_frame_proxy_host.cc

 // Copyright 2014 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/browser/frame_host/render_frame_proxy_host.h"
 
 #include <utility>
 
 #include "base/lazy_instance.h"
 #include "content/browser/bad_message.h"
@@ skipping 176 matching lines @@
   Send(new FrameMsg_NewFrameProxy(routing_id_,
                                   frame_tree_node_->frame_tree()
                                       ->GetRenderViewHost(site_instance_.get())
                                       ->GetRoutingID(),
                                   opener_routing_id,
                                   parent_routing_id,
                                   frame_tree_node_
                                       ->current_replication_state()));
 
   render_frame_proxy_created_ = true;
+
+  // For subframes, initialize the proxy's WebFrameOwnerProperties only if they
+  // differ from default values.
+  bool should_send_properties = frame_tree_node_->frame_owner_properties() !=
+                                blink::WebFrameOwnerProperties();
+  if (frame_tree_node_->parent() && should_send_properties) {

[alexmos, 2016/05/11 00:15:09]

Just a note that after discussing this with Charlie last week, I went ahead and
added this optimization in PS6 to only send WebFrameOwnerProperties to new
proxies when they're different from default values.  No need for this IPC in the
default case, as WebRemoteFrameImpl::createRemoteChild already initializes the
FrameOwner with default values.

[Charlie Reis, 2016/05/13 21:18:42]

Acknowledged.

+    Send(new FrameMsg_SetFrameOwnerProperties(
+        routing_id_, frame_tree_node_->frame_owner_properties()));
+  }
+
   return true;
 }
 
 void RenderFrameProxyHost::UpdateOpener() {
   // Another frame in this proxy's SiteInstance may reach the new opener by
   // first reaching this proxy and then referencing its window.opener.  Ensure
   // the new opener's proxy exists in this case.
   if (frame_tree_node_->opener()) {
     frame_tree_node_->opener()->render_manager()->CreateOpenerProxies(
         GetSiteInstance(), frame_tree_node_);
@@ skipping 151 matching lines @@
   target_rfh->Send(new FrameMsg_AdvanceFocus(target_rfh->GetRoutingID(), type,
                                              source_proxy_routing_id));
 }
 
 void RenderFrameProxyHost::OnFrameFocused() {
   frame_tree_node_->frame_tree()->SetFocusedFrame(frame_tree_node_,
                                                   GetSiteInstance());
 }
 
 }  // namespace content