Make page popups work under OOPIF

content/browser/frame_host/render_widget_host_view_child_frame.cc

Index: content/browser/frame_host/render_widget_host_view_child_frame.cc
diff --git a/content/browser/frame_host/render_widget_host_view_child_frame.cc b/content/browser/frame_host/render_widget_host_view_child_frame.cc
index 7c60d9c1126d8c62a45720fb220c2f4b111bd7f1..71318cf7dfafcabf01057604013e67ee361f148d 100644
--- a/content/browser/frame_host/render_widget_host_view_child_frame.cc
+++ b/content/browser/frame_host/render_widget_host_view_child_frame.cc
@@ -68,6 +68,11 @@ void RenderWidgetHostViewChildFrame::SetSize(const gfx::Size& size) {
 
 void RenderWidgetHostViewChildFrame::SetBounds(const gfx::Rect& rect) {
   SetSize(rect.size());
+
+  if (rect != last_screen_rect_) {
+    last_screen_rect_ = rect;
+    host_->SendScreenRects();
+  }

[lfg, 2016/03/18 15:46:00]

I don't think this is enough, we need to send the screen rects to all the
descendant local roots in the frame tree.

[kenrb, 2016/03/18 17:46:10]

I don't think I understand this. This should be called independently for each
local root, since each CrossProcessFrameConnector should get an IPC with the
rect in it.

[lfg, 2016/03/18 18:53:53]

Let's say you have a FrameTree like:

A
 |
B
 |
C

If in the renderer for A the position of B is changed, we should get an IPC for
the change in the frame rect. However, the location of C also changed, but I
don't think renderer B will send an IPC for the location change.

[kenrb, 2016/03/18 20:03:19]

This is a really good point, thanks for catching it. Given time constraints, I
am going to propose that (if this is the only issue), this CL goes in as is and
I will file a bug to fix next week.

[lfg, 2016/03/18 20:06:20]

Sounds good. lgtm.

 }
 
 void RenderWidgetHostViewChildFrame::Focus() {