Place all new baseline files in the most specific platform directory....

webkit/tools/layout_tests/run_webkit_tests.py

 #!/bin/env python
 # Copyright (c) 2006-2008 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.
 
 """Run layout tests using the test_shell.
 
 This is a port of the existing webkit test script run-webkit-tests.
 
 The TestRunner class runs a series of tests (TestType interface) against a set
@@ skipping 212 matching lines @@
     for i in xrange(int(self._options.num_test_shells)):
       shell_args = []
       test_args = test_type_base.TestArguments()
       if self._options.pixel_tests:
         png_path = os.path.join(self._options.results_directory,
                                 "png_result%s.png" % i)
         shell_args.append("--pixel-tests=" + png_path)
         test_args.png_path = png_path
 
       test_args.wdiff = self._options.wdiff
-
-      if self._options.new_baseline:
-        test_args.new_baseline = self._options.new_baseline
-        if not self._options.pixel_tests:
-          test_args.text_baseline = True
-
+      test_args.new_baseline = self._options.new_baseline
       test_args.show_sources = self._options.sources
 
       # Create separate TestTypes instances for each thread.
       test_types = []
       for t in self._test_types:
         test_types.append(t(self._options.platform,
                             self._options.results_directory))
 
       if self._options.startup_dialog:
         shell_args.append('--testshell-startup-dialog')
@@ skipping 257 matching lines @@
   try:
     test_shell_binary_path = path_utils.TestShellBinaryPath(options.target)
   except:
     print "\nERROR: test_shell is not found. Be sure that you have built it"
     print "and that you are using the correct build. This script will run the"
     print "Release one by default. Use --debug to use the Debug build.\n"
     sys.exit(1)
 
   logging.info("Using platform '%s'" % options.platform)
   logging.info("Placing test results in %s" % options.results_directory)
+  if options.new_baseline:
+    logging.info("Placing new baselines in %s" %
+                 os.path.join(path_utils.PlatformResultsDir(options.platform),
+                              options.platform))
   logging.info("Using %s build at %s" %
                (options.target, test_shell_binary_path))
   if options.pixel_tests:
     logging.info("Running pixel tests")
 
   if 'cygwin' == sys.platform:
     logging.warn("#" * 40)
     logging.warn("# UNEXPECTED PYTHON VERSION")
     logging.warn("# This script should be run using the version of python")
     logging.warn("# in third_party/python_24/")
@@ skipping 33 matching lines @@
   option_parser.add_option("", "--pixel-tests", action="store_true",
                            default=False,
                            help="enable pixel-to-pixel PNG comparisons")
   option_parser.add_option("", "--wdiff", action="store_true",
                            default=False,
                            help="enable word-by-word diffing")
   option_parser.add_option("", "--results-directory",
                            default="layout-test-results",
                            help="Output results directory source dir,"
                                 " relative to Debug or Release")
-  option_parser.add_option("", "--new-baseline", default=None, metavar="DIR",
-                           help="save results as new baselines into this "
-                                "directory (e.g. layout_test_results/v8), "
-                                "overwriting whatever's already there. "
-                                "If pixel tests are being run, only image "
-                                "baselines will be saved, not text.")
+  option_parser.add_option("", "--new-baseline", action="store_true",
+                           default=False,
+                           help="save all generated results as new baselines "
+                                "into the platform directory , overwriting "

[tony, 2008/11/14 04:45:42]

Nit: space before comma.

+                                "whatever's already there.")
   option_parser.add_option("", "--noshow-results", action="store_true",
                            default=False, help="don't launch the test_shell"
                            " with results after the tests are done")
   option_parser.add_option("", "--full-results-html", action="store_true",
                            default=False, help="show all failures in"
                            "results.html, rather than only regressions")
   option_parser.add_option("", "--nocompare-failures", action="store_true",
                            default=False,
                            help="Disable comparison to the last test run. "
                                 "When enabled, show stats on how many tests "
@@ skipping 22 matching lines @@
                            default=False, 
                            help="create a dialog on test_shell.exe startup")
   option_parser.add_option("", "--winhttp", action="store_true",
                            default=False, 
                            help="Use WinHTTP stack")
   option_parser.add_option("", "--test-list", action="append",
                            help="read list of tests to run from file",
                            metavar="FILE")
   options, args = option_parser.parse_args()
   main(options, args)