DevTools: cleanup debugger tests.

LayoutTests/http/tests/inspector/inspector-test.js

 var initialize_InspectorTest = function() {
 
 var results = [];
 var resultsSynchronized = false;
 
 function consoleOutputHook(messageType)
 {
     InspectorTest.addResult(messageType + ": " + Array.prototype.slice.call(arguments, 1));
 }
 
@@ skipping 678 matching lines @@
     requestAnimationFrame(runTest.bind(this, enableWatchDogWhileDebugging));
 }
 
 function didEvaluateForTestInFrontend(callId)
 {
     if (callId !== completeTestCallId)
         return;
     delete window.completeTestCallId;
     if (outputElement && window.quietUntilDone)
         outputElementParent.appendChild(outputElement);
-    // Close inspector asynchrously to allow caller of this
-    // function send response before backend dispatcher and frontend are destroyed.
-    setTimeout(closeInspectorAndNotifyDone, 0);
+    closeInspectorAndNotifyDone();

[vsevik, 2014/06/06 11:51:23]

I don't really understand this code, but why did this change? Is this necessary
to do this change in this CL ?
This might cause some unexpected flakiness and it would be better to make this
change separately.

 }
 
 function closeInspectorAndNotifyDone()
 {
     if (window._watchDogTimer)
         clearTimeout(window._watchDogTimer);
 
     testRunner.closeWebInspector();
     setTimeout(function() {
         testRunner.notifyDone();
@@ skipping 58 matching lines @@
     evaluateInWebInspector: function(callId, script)
     {
         window.opener.postMessage(["evaluateInWebInspector", callId, script], "*");
     },
 
     display: function() { }
 }
 
 if (!window.testRunner && window.opener)
     window.testRunner = new StandaloneTestRunnerStub();