Refactor frame navigation/detach state cleanup to be more sane.

Source/web/tests/FrameTestHelpers.cpp

Index: Source/web/tests/FrameTestHelpers.cpp
diff --git a/Source/web/tests/FrameTestHelpers.cpp b/Source/web/tests/FrameTestHelpers.cpp
index 9c278750566274d261444fe66b79f38832e1f33b..c2cf8433db4bf28e3a3ab1215068d14bc03adaf7 100644
--- a/Source/web/tests/FrameTestHelpers.cpp
+++ b/Source/web/tests/FrameTestHelpers.cpp
@@ -40,6 +40,7 @@
 #include "public/platform/WebURLRequest.h"
 #include "public/platform/WebURLResponse.h"
 #include "public/platform/WebUnitTestSupport.h"
+#include "public/web/WebRemoteFrame.h"
 #include "public/web/WebSettings.h"
 #include "public/web/WebViewClient.h"
 #include "web/WebLocalFrameImpl.h"
@@ -320,6 +321,18 @@ void TestWebFrameClient::waitForLoadToComplete()
     }
 }
 
+TestWebRemoteFrameClient::TestWebRemoteFrameClient()
+    : m_frame(WebRemoteFrame::create(this))
+{
+}
+
+void TestWebRemoteFrameClient::frameDetached()
+{
+    if (m_frame->parent())
+        m_frame->parent()->removeChild(m_frame);
+    m_frame->close();
+}
+
 void TestWebViewClient::initializeLayerTreeView()
 {
     m_layerTreeView = adoptPtr(Platform::current()->unitTestSupport()->createLayerTreeViewForTesting());