[content shell] don't open new windows after printing the test results

content/shell/webkit_test_controller.h

Index: content/shell/webkit_test_controller.h
diff --git a/content/shell/webkit_test_controller.h b/content/shell/webkit_test_controller.h
index 8585d5c3899f8dd574bb685cba8563a54a0b3c5c..af7411f0b5e939ceeb1ab2773b4f6cea65d98fe3 100644
--- a/content/shell/webkit_test_controller.h
+++ b/content/shell/webkit_test_controller.h
@@ -160,6 +160,10 @@ class WebKitTestController : public base::NonThreadSafe,
   // phase between two layout tests.
   bool is_running_test_;
 
+  // True if we already printed results and are in the cleaning up phase of a
+  // test.
+  bool is_cleaning_up_;

[marja, 2013/05/28 11:16:35]

Would it make sense to have a "test phase" enum instead of these bools? It would
also list the test phases in the order they occur, and make the code simpler to
understand...

+
   // True if the currently running test is a compositing test.
   bool is_compositing_test_;