Detach old frame on WebFrame::swap.

components/html_viewer/html_document.cc

Index: components/html_viewer/html_document.cc
diff --git a/components/html_viewer/html_document.cc b/components/html_viewer/html_document.cc
index bb7163763313ae216db3b56b7cddf7e395c96cf5..1358ba7653e2eca949f508e925d246c6a74c5335 100644
--- a/components/html_viewer/html_document.cc
+++ b/components/html_viewer/html_document.cc
@@ -273,6 +273,12 @@ blink::WebFrame* HTMLDocument::createChildFrame(
 }
 
 void HTMLDocument::frameDetached(blink::WebFrame* frame) {
+  frameDetached(frame, blink::WebDetachReason::Remove);

[dcheng, 2015/06/01 21:15:54]

Once this is defined inside WebFrameClient, we should be able to just say:
DetachReason::Remove.

+}
+
+void HTMLDocument::frameDetached(blink::WebFrame* frame,
+                                 blink::WebDetachReason reason) {
+  DCHECK(reason == blink::WebDetachReason::Remove);

[dcheng, 2015/06/01 21:15:54]

DCHECK_EQ

   if (frame->parent())
     frame->parent()->removeChild(frame);