Preserve page resize information when main frame is remote

third_party/WebKit/Source/web/tests/WebFrameTest.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 7003 matching lines @@
     FrameTestHelpers::loadFrame(localFrame, m_baseURL + "subframe-hello.html");
     std::string content = localFrame->contentAsText(1024).utf8();
     EXPECT_EQ("hello", content);
 
     // Manually reset to break WebViewHelper's dependency on the stack allocated
     // TestWebFrameClient.
     reset();
     remoteFrame->close();
 }
 
+TEST_F(WebFrameSwapTest, ValidateSizeOnRemoteToLocalMainFrameSwap)
+{
+    WebSize size(111, 222);
+
+    WebRemoteFrame* remoteFrame = WebRemoteFrame::create(WebTreeScopeType::Document, nullptr);
+    mainFrame()->swap(remoteFrame);
+
+    remoteFrame->view()->resize(size);
+
+    FrameTestHelpers::TestWebFrameClient client;
+    WebLocalFrame* localFrame = WebLocalFrame::createProvisional(&client, remoteFrame, WebSandboxFlags::None, WebFrameOwnerProperties());
+    remoteFrame->swap(localFrame);
+
+    // Verify that the size that was set with a remote main frame is correct
+    // after swapping to a local frame.
+    WebFloatSize floatSize = localFrame->view()->visualViewportSize();

[bokan, 2016/01/11 18:18:58]

This actually returns the scaled size of the viewport. It works here since you
don't change the scale but I think it'd be more correct to get a ref to the
VisualViewport and test it's size() method explicitly. (e.g. depending on the
loaded page and how much initialization we go through here, on Android this will
fail since it initializes to minimum scale).

[kenrb, 2016/01/11 20:50:26]

Done.

+    EXPECT_EQ(size.width, floatSize.width);
+    EXPECT_EQ(size.height, floatSize.height);
+
+    // Manually reset to break WebViewHelper's dependency on the stack allocated
+    // TestWebFrameClient.
+    reset();
+    remoteFrame->close();
+}
+
 namespace {
 
 class SwapMainFrameWhenTitleChangesWebFrameClient : public FrameTestHelpers::TestWebFrameClient {
 public:
     SwapMainFrameWhenTitleChangesWebFrameClient()
         : m_remoteFrame(nullptr)
     {
     }
 
     ~SwapMainFrameWhenTitleChangesWebFrameClient() override
@@ skipping 1317 matching lines @@
 
 TEST_F(WebFrameTest, CallbackOrdering)
 {
     registerMockedHttpURLLoad("foo.html");
     FrameTestHelpers::WebViewHelper webViewHelper;
     CallbackOrderingWebFrameClient client;
     webViewHelper.initializeAndLoad(m_baseURL + "foo.html", true, &client);
 }
 
 } // namespace blink