OOPIF: Use the same uniqueName when swapping between local and 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 7455 matching lines @@
     localFrame->setCommittedFirstRealLoad();
     FrameTestHelpers::loadFrame(localFrame, m_baseURL + "subframe-hello.html");
     EXPECT_EQ(WebStandardCommit, client.historyCommitType());
 
     // Manually reset to break WebViewHelper's dependency on the stack allocated
     // TestWebFrameClient.
     reset();
     remoteFrame->close();
 }
 
+// The uniqueName should be preserved when swapping to a RemoteFrame and back,
+// whether the frame has a name or not.
+TEST_F(WebFrameSwapTest, UniqueNameAfterRemoteToLocalSwap)
+{
+    // Start with a named frame.
+    WebFrame* targetFrame = mainFrame()->firstChild();
+    ASSERT_TRUE(targetFrame);
+    WebString uniqueName = targetFrame->uniqueName();
+    EXPECT_EQ("frame1", uniqueName.utf8());
+
+    // Swap to a RemoteFrame.
+    FrameTestHelpers::TestWebRemoteFrameClient remoteFrameClient;
+    WebRemoteFrame* remoteFrame = WebRemoteFrame::create(WebTreeScopeType::Document, &remoteFrameClient);
+    targetFrame->swap(remoteFrame);
+    ASSERT_TRUE(mainFrame()->firstChild());
+    ASSERT_EQ(mainFrame()->firstChild(), remoteFrame);
+    EXPECT_EQ(uniqueName.utf8(), remoteFrame->uniqueName().utf8());
+
+    // Swap back to a LocalFrame.
+    RemoteToLocalSwapWebFrameClient client(remoteFrame);
+    WebLocalFrame* localFrame = WebLocalFrame::create(WebTreeScopeType::Document, &client);
+    localFrame->initializeToReplaceRemoteFrame(remoteFrame, "frame1", WebSandboxFlags::None, WebFrameOwnerProperties());
+    EXPECT_EQ(uniqueName.utf8(), localFrame->uniqueName().utf8());
+    FrameTestHelpers::loadFrame(localFrame, m_baseURL + "subframe-hello.html");
+    EXPECT_EQ(uniqueName.utf8(), localFrame->uniqueName().utf8());
+
+    // Repeat with no name on the frame.
+    localFrame->setName("");
+    WebString uniqueName2 = localFrame->uniqueName();
+    EXPECT_EQ("<!--framePath //<!--frame2-->-->", uniqueName2.utf8());
+
+    FrameTestHelpers::TestWebRemoteFrameClient remoteFrameClient2;
+    WebRemoteFrame* remoteFrame2 = WebRemoteFrame::create(WebTreeScopeType::Document, &remoteFrameClient2);
+    localFrame->swap(remoteFrame2);
+    ASSERT_TRUE(mainFrame()->firstChild());
+    ASSERT_EQ(mainFrame()->firstChild(), remoteFrame2);
+    EXPECT_EQ(uniqueName2.utf8(), remoteFrame2->uniqueName().utf8());
+
+    RemoteToLocalSwapWebFrameClient client2(remoteFrame2);
+    WebLocalFrame* localFrame2 = WebLocalFrame::create(WebTreeScopeType::Document, &client2);
+    localFrame2->initializeToReplaceRemoteFrame(remoteFrame2, "", WebSandboxFlags::None, WebFrameOwnerProperties());
+    EXPECT_EQ(uniqueName2.utf8(), localFrame2->uniqueName().utf8());
+    FrameTestHelpers::loadFrame(localFrame2, m_baseURL + "subframe-hello.html");
+    EXPECT_EQ(uniqueName2.utf8(), localFrame2->uniqueName().utf8());
+
+    // Manually reset to break WebViewHelper's dependency on the stack allocated
+    // TestWebFrameClient.
+    reset();
+    remoteFrame->close();
+    remoteFrame2->close();
+}
+
 class RemoteNavigationClient : public FrameTestHelpers::TestWebRemoteFrameClient {
 public:
     void navigate(const WebURLRequest& request, bool shouldReplaceCurrentEntry) override
     {
         m_lastRequest = request;
     }
 
     const WebURLRequest& lastRequest() const { return m_lastRequest; }
 
 private:
@@ skipping 837 matching lines @@
 
 TEST_F(WebFrameTest, CallbackOrdering)
 {
     registerMockedHttpURLLoad("foo.html");
     FrameTestHelpers::WebViewHelper webViewHelper;
     CallbackOrderingWebFrameClient client;
     webViewHelper.initializeAndLoad(m_baseURL + "foo.html", true, &client);
 }
 
 } // namespace blink