Fix test.dart to have the same behavior as test.py for --frog and --froglib.

tools/testing/dart/test_suite.dart

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #library("test_suite");
 
 #import("status_file_parser.dart");
 #import("test_runner.dart");
 #import("multitest.dart");
 
@@ skipping 426 matching lines @@
           }
           compilerArgs.addAll(vmOptions);
           compilerArgs.add('--ignore-unrecognized-flags');
           compilerArgs.add('--out');
           compilerArgs.add(compiledDartWrapperFilename);
           compilerArgs.add(dartWrapperFilename);
           // TODO(whesse): Add --fatal-type-errors if needed.
           break;
         case 'frogium':
         case 'webdriver':
-          compilerArgs.addAll(['--libdir=$dartDir/frog/lib',
+          String libdir = configuration['froglib'];
+          if (libdir == '') {
+            libdir = '$dartDir/frog/lib';
+          }
+          compilerArgs.addAll(['--libdir=$libdir',
                                '--compile-only',
                                '--out=$compiledDartWrapperFilename']);
           compilerArgs.addAll(vmOptions);
           compilerArgs.add(dartWrapperFilename);
           break;
         case 'dartium':
           // No compilation phase.
           compilerExecutable = null;
           compilerArgs = null;
           break;
         default:
           Expect.fail('unimplemented component $component');
       }
 
       String executable = getFilename(dumpRenderTreeFilename);
       List<String> args;
       if (component == 'webdriver') {
+        // TODO(efortuna): These paths are not OS independent! 
         executable = '$dartDir/tools/testing/run_selenium.py';
         args = ['--out', htmlPath, '--browser', configuration['browser']];
       } else {
         args = ['--no-timeout'];
         if (component == 'dartium') {
           var dartFlags = ['--enable_asserts',
                            '--enable_type_checks',
                            '--ignore-unrecognized-flags'];
           dartFlags.addAll(vmOptions);
           args.add('--dart-flags=${Strings.join(dartFlags, " ")}');
@@ skipping 446 matching lines @@
     if (!(new File(name)).existsSync()) {
       throw "Executable '$name' does not exist";
     }
     return name;
   }
 
   static String compilerPath(Map configuration) {
     if (configuration['component'] == 'dartium') {
       return null;  // No separate compiler for dartium tests.
     }
-    var name = '${buildDir(configuration)}/${compilerName(configuration)}';
+    var name = configuration['frog'];

[Bill Hesse, 2012/01/19 16:31:28]

I don't think we want --frog to override the dartc path if component =
'chromium' or 'dartc'.  This could happen with
test.dart -cchromium,frogium --frog=i_built/this/frogsh.
Could this be
if (configuration['frog'] != '' && (configuration['component'] == 'frogium' ||
... 'webkit')) {
 name = configuration['frog'];
} else {
 name = ....
}

+    if (name == '') {
+      name = '${buildDir(configuration)}/${compilerName(configuration)}';
+    }
     if (!(new File(name)).existsSync()) {
       throw "Executable '$name' does not exist";
     }
     return name;
   }
 
   static String outputDir(Map configuration) {
     var outputDir = '';
     var system = configuration['system'];
     if (system == 'linux') {
@@ skipping 74 matching lines @@
  * $noCrash tests are expected to be flaky but not crash
  * $pass tests are expected to pass
  * $failOk tests are expected to fail that we won't fix
  * $fail tests are expected to fail that we should fix
  * $crash tests are expected to crash that we should fix
  * $timeout tests are allowed to timeout
 """;
     print(report);
    }
 }