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

content/shell/webkit_test_controller.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_SHELL_WEBKIT_TEST_CONTROLLER_H_
 #define CONTENT_SHELL_WEBKIT_TEST_CONTROLLER_H_
 
 #include <ostream>
 #include <string>
 
@@ skipping 142 matching lines @@
   int current_pid_;
 
   // True if we should set the test configuration to the next RenderViewHost
   // created.
   bool send_configuration_to_next_host_;
 
   // True if we are currently running a layout test, and false during the setup
   // 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_;
 
   // Per test config.
   bool enable_pixel_dumping_;
   std::string expected_pixel_hash_;
   gfx::Size initial_size_;
   GURL test_url_;
 
   // True if the WebPreferences of newly created RenderViewHost should be
   // overridden with prefs_.
   bool should_override_prefs_;
   WebPreferences prefs_;
 
   base::CancelableClosure watchdog_;
 
   NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(WebKitTestController);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_SHELL_WEBKIT_TEST_CONTROLLER_H_