Make page popups work under OOPIF

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 17e4875da9c03244cd432c0eb68f07a05241f8d9..a4b6333396d6ddeae0819cef40e60d25dfa9b6d0 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -2225,6 +2225,15 @@ void WebContentsImpl::SendScreenRects() {
   RenderWidgetHostImpl::From(GetRenderViewHost()->GetWidget())
       ->SendScreenRects();
 
+  for (FrameTreeNode* node : frame_tree_.Nodes()) {
+    // TODO(kenrb): We skip the root, because it is already covered by the
+    // RenderViewHost call above. When RenderViewHost is no longer a
+    // RenderWidgetHost, this can be used to update all RenderWidgetHosts.

[nasko, 2016/03/17 17:25:45]

RenderViewHost is no longer RenderWidgetHost, right? Why not remove the above
call and just iterate through all the nodes?
Also, even if they did inherit, why does that matter?

[kenrb, 2016/03/17 18:59:48]

Thanks for the tip, I didn't realize the top-level RenderFrameHost's widget
pointer points to the main RenderWidgetHost now. Fixed.

+    if (node != frame_tree_.root() &&
+        node->current_frame_host()->GetRenderWidgetHost())
+      node->current_frame_host()->GetRenderWidgetHost()->SendScreenRects();
+  }
+
   RenderWidgetHostViewBase* rwhv =
       static_cast<RenderWidgetHostViewBase*>(GetRenderWidgetHostView());
   if (rwhv) {