DevTools: fix for open-close-open.html test

content/shell/renderer/webkit_test_runner.cc

Index: content/shell/renderer/webkit_test_runner.cc
diff --git a/content/shell/renderer/webkit_test_runner.cc b/content/shell/renderer/webkit_test_runner.cc
index a6bd657859f79f924ce6c33d2c835c84b6bbcfb6..ece38507ec03b1b2ba12668dc3b80d7408834c1e 100644
--- a/content/shell/renderer/webkit_test_runner.cc
+++ b/content/shell/renderer/webkit_test_runner.cc
@@ -365,6 +365,9 @@ void WebKitTestRunner::showDevTools() {
 
 void WebKitTestRunner::closeDevTools() {
   Send(new ShellViewHostMsg_CloseDevTools(routing_id()));
+  WebDevToolsAgent* agent = render_view()->GetWebView()->devToolsAgent();
+  if (agent)
+    agent->detach();

[yurys, 2013/06/06 09:28:23]

How was it called before when front-end was closing? Don't we need to fix that
place too?

[yurys, 2013/06/06 09:35:41]

Looks like WebDevToolsAgentImpl::detach() should check if front-end is still
attached. Also it seems that there can be a race:
1. First front-end close is started
2. New front-end attached.
3. Detach event is received from the first front-end and current front-end (the
new one) is cleared
 or am I missing something?

 }
 
 void WebKitTestRunner::evaluateInWebInspector(long call_id,