Enable resize and layout for frames with RemoteFrame ancestors

Source/web/PageWidgetDelegate.cpp

Index: Source/web/PageWidgetDelegate.cpp
diff --git a/Source/web/PageWidgetDelegate.cpp b/Source/web/PageWidgetDelegate.cpp
index e2735262e119fcb1bb184f62b343489c2ab03ffb..7e2bcf57183d5c73080040c99862a442c9ef01ea 100644
--- a/Source/web/PageWidgetDelegate.cpp
+++ b/Source/web/PageWidgetDelegate.cpp
@@ -69,11 +69,18 @@ void PageWidgetDelegate::animate(Page* page, double monotonicFrameBeginTime)
     page->animator().serviceScriptedAnimations(monotonicFrameBeginTime);
 }
 
-void PageWidgetDelegate::layout(Page* page)
+void PageWidgetDelegate::layout(Page* page, LocalFrame* rootFrame)
 {
-    if (!page || !page->mainFrame())
+    if (!page)

[eseidel, 2014/07/11 16:33:03]

Why is it valid to call this method with a null page?

[kenrb, 2014/07/11 20:03:33]

It was added explicitly in r169672. I am not certain, but it looks like it is
needed for some unit tests.

[dcheng, 2014/07/11 21:22:52]

Do you mind filing a bug and assigning it to me? This is kind of bizarre;
ideally we should fix those tests rather than adding test-specific handling
here.

         return;
-    page->animator().updateLayoutAndStyleForPainting();
+
+    if (!rootFrame) {

[eseidel, 2014/07/11 16:33:03]

Why not make this a default =0 argument then?  Or is the goal that eventually
teh Page argument will be gone from this function and thus rootFrame makes sense
to be required?

[kenrb, 2014/07/11 20:03:33]

Done.

+        if (!page->mainFrame() || !page->mainFrame()->isLocalFrame())

[eseidel, 2014/07/11 16:33:03]

How can a page not have a mainFrame?  Or a page's mainFrame be remote?

[kenrb, 2014/07/11 20:03:33]

I don't know about a null mainFrame (see above revision number). However,
mainFrames can be remote when we are rendering an out-of-process iframe. The
entire frame tree has to be replicated, and all frames from the original process
(including the mainFrame) become remoteFrames in the child frame's process.

+            return;
+        rootFrame = toLocalFrame(page->mainFrame());
+    }
+
+    page->animator().updateLayoutAndStyleForPainting(rootFrame);
 }
 
 void PageWidgetDelegate::paint(Page* page, PageOverlayList* overlays, WebCanvas* canvas, const WebRect& rect, CanvasBackground background)