Detach old frame on WebFrame::swap.

Source/core/page/PrintContextTest.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/page/PrintContext.h"
 
 #include "core/dom/Document.h"
 #include "core/frame/FrameHost.h"
 #include "core/frame/FrameView.h"
@@ skipping 316 matching lines @@
 
     printSinglePage(canvas);
 
     const Vector<MockCanvas::Operation>& operations = canvas.recordedOperations();
     ASSERT_EQ(2u, operations.size());
     EXPECT_EQ(MockCanvas::DrawRect, operations[0].type);
     EXPECT_SKRECT_EQ(250, 260, 170, 180, operations[0].rect);
     EXPECT_EQ(MockCanvas::DrawRect, operations[1].type);
     EXPECT_SKRECT_EQ(350, 360, 270, 280, operations[1].rect);
 
-    subframe->detach();
+    subframe->detach(FrameDetachType::Remove);
     static_cast<SingleChildFrameLoaderClient*>(document().frame()->client())->setChild(nullptr);
     document().frame()->host()->decrementSubframeCount();
 }
 
 TEST_F(PrintContextFrameTest, WithScrolledSubframe)
 {
     MockCanvas canvas;
     document().setBaseURLOverride(KURL(ParsedURLString, "http://a.com/"));
     setBodyInnerHTML("<style>::-webkit-scrollbar { display: none }</style>"
         "<iframe id='frame' src='http://b.com/' width='500' height='500'"
@@ skipping 22 matching lines @@
 
     const Vector<MockCanvas::Operation>& operations = canvas.recordedOperations();
     ASSERT_EQ(3u, operations.size());
     EXPECT_EQ(MockCanvas::DrawRect, operations[0].type);
     EXPECT_SKRECT_EQ(50, 60, 70, 80, operations[0].rect); // FIXME: the rect should be clipped.
     EXPECT_EQ(MockCanvas::DrawRect, operations[1].type);
     EXPECT_SKRECT_EQ(150, 160, 170, 180, operations[1].rect);
     EXPECT_EQ(MockCanvas::DrawRect, operations[2].type);
     EXPECT_SKRECT_EQ(250, 260, 270, 280, operations[2].rect);
 
-    subframe->detach();
+    subframe->detach(FrameDetachType::Remove);
     static_cast<SingleChildFrameLoaderClient*>(document().frame()->client())->setChild(nullptr);
     document().frame()->host()->decrementSubframeCount();
 }
 
 } // namespace blink