OOPIF: Send page-level focus messages to all processes rendering a page.

content/browser/frame_host/render_frame_host_manager.cc

Index: content/browser/frame_host/render_frame_host_manager.cc
diff --git a/content/browser/frame_host/render_frame_host_manager.cc b/content/browser/frame_host/render_frame_host_manager.cc
index e19b9fa3212da70629e2d0e8f8c9ed8b1fc4e978..94ca8d238cfab28145e6e2dac5b026151c37d65e 100644
--- a/content/browser/frame_host/render_frame_host_manager.cc
+++ b/content/browser/frame_host/render_frame_host_manager.cc
@@ -32,6 +32,7 @@
 #include "content/browser/webui/web_ui_controller_factory_registry.h"
 #include "content/browser/webui/web_ui_impl.h"
 #include "content/common/frame_messages.h"
+#include "content/common/input_messages.h"
 #include "content/common/site_isolation_policy.h"
 #include "content/common/view_messages.h"
 #include "content/public/browser/content_browser_client.h"
@@ -83,6 +84,12 @@ bool OpenerForFrameTreeNode(
   return true;
 }
 
+bool CollectSiteInstances(std::set<SiteInstance*>* set,

[Charlie Reis, 2015/10/16 18:56:09]

nit: Add comment about what this is useful for (e.g., SiteInstances involved in
rendering the page, which is a subset of those in proxy_hosts_, thanks to
openers).

[alexmos, 2015/10/16 22:56:05]

Done.

+                          FrameTreeNode* node) {
+  set->insert(node->current_frame_host()->GetSiteInstance());
+  return true;
+}
+
 }  // namespace
 
 // A helper class to hold all frame proxies and register as a
@@ -1992,6 +1999,28 @@ void RenderFrameHostManager::SetRWHViewForInnerContents(
   GetProxyToOuterDelegate()->SetChildRWHView(child_rwhv);
 }
 
+void RenderFrameHostManager::ReplicatePageFocus(bool is_focused) {
+  CHECK(frame_tree_node_->IsMainFrame());
+
+  std::set<SiteInstance*> frame_tree_site_instances;
+  frame_tree_node_->frame_tree()->ForEach(
+      base::Bind(&CollectSiteInstances, &frame_tree_site_instances));
+
+  for (const auto& instance : frame_tree_site_instances) {
+    if (instance == render_frame_host_->GetSiteInstance())
+      continue;
+
+    // Main frame should have proxies in all SiteInstances of other frames in

[Charlie Reis, 2015/10/16 18:56:09]

nit: Notify the proxies for all SiteInstances of other frames in this FrameTree.
 Note that the main frame might also know about proxies in...

[alexmos, 2015/10/16 22:56:05]

Done.

+    // this FrameTree. Note that it might also contain proxies in SiteInstances
+    // for a frame in a different FrameTree (e.g., for window.open), so we
+    // can't just iterate over proxy_hosts_.

[Charlie Reis, 2015/10/16 18:56:09]

This comment should go above line 2005.

[alexmos, 2015/10/16 22:56:05]

Done.

+    RenderFrameProxyHost* proxy = GetRenderFrameProxyHost(instance);
+    CHECK(proxy);
+
+    proxy->Send(new InputMsg_SetFocus(proxy->GetRoutingID(), is_focused));
+  }
+}
+
 bool RenderFrameHostManager::InitRenderView(
     RenderViewHostImpl* render_view_host,
     int proxy_routing_id) {