Fix static type warnings in test.dart.

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 44 matching lines @@
   ReceivePort receiveTestName;
   TestExpectations testExpectations;
 
   CCTestSuite(Map this.configuration,
               String this.suiteName,
               String runnerName,
               List<String> this.statusFilePaths) {
     runnerPath = TestUtils.buildDir(configuration) + '/' + runnerName;
   }
 
-  void complexStatusMatching() => false;
+  bool complexStatusMatching() => false;
 
   void testNameHandler(String testName, ignore) {
     if (testName == "") {
       receiveTestName.close();
       doDone(true);
     } else {
       // Only run the tests that match the pattern. Use the name
       // "suiteName/testName" for cc tests.
       RegExp pattern = configuration['selectors'][suiteName];
       String constructedName = '$suiteName/$testName';
@@ skipping 78 matching lines @@
   TestExpectations testExpectations;
   List<TestInformation> cachedTests;
   final String pathSeparator;
 
   StandardTestSuite(Map this.configuration,
                     String this.suiteName,
                     String this.directoryPath,
                     List<String> this.statusFilePaths)
     : pathSeparator = new Platform().pathSeparator();
 
-  void isTestFile(String filename) => filename.endsWith("Test.dart");
+  bool isTestFile(String filename) => filename.endsWith("Test.dart");
 
-  void listRecursively() => false;
+  bool listRecursively() => false;
 
-  void complexStatusMatching() => false;
+  bool complexStatusMatching() => false;
 
   String shellPath() => TestUtils.dartShellFileName(configuration);
 
   List<String> additionalOptions() => [];
 
   void forEachTest(Function onTest, Map testCache, [Function onDone = null]) {
     doTest = onTest;
     doDone = (onDone != null) ? onDone : (() => null);
 
     var filesRead = 0;
@@ skipping 322 matching lines @@
         return 'application/dart';
       case 'chromium':
       case 'frogium':
         return 'text/javascript';
       default:
         Expect.fail('Unimplemented component scriptType');
         return null;
     }
   }
 
-  String get scriptName() {
-    switch (configuration['component']) {
-      case 'dartium':
-        return tempDir.path + 'test.dart';
-      case 'chromium':
-      case 'frogium':
-        return tempDir.path + 'test.js';
-      default:
-        Expect.fail('Unimplemented component scriptType');
-        return null;
-    }
-  }
-
   String getHtmlName(String filename) {
     return filename.replaceAll('/', '_') + configuration['component'] + '.html';
   }
 
   String get dumpRenderTreeFilename() {
     if (new Platform().operatingSystem() == 'macos') {
       return 'client/tests/drt/DumpRenderTree.app/Contents/' +
           'MacOS/DumpRenderTree';
     } 
     return 'client/tests/drt/DumpRenderTree';
@@ skipping 142 matching lines @@
   DartcCompilationTestSuite(Map configuration,
                             String suiteName,
                             String directoryPath,
                             List<String> this._testDirs,
                             List<String> expectations)
       : super(configuration,
               suiteName,
               directoryPath,
               expectations);
 
-  void activityStarted() => ++activityCount;
+  void activityStarted() { ++activityCount; }
 
   void activityCompleted() {
     if (--activityCount == 0) {
       directoryListingDone(true);
     }
   }
 
   String shellPath() => TestUtils.compilerPath(configuration);
 
   List<String> additionalOptions() {
@@ skipping 38 matching lines @@
   List<String> testClasses;
   Function doTest;
   Function doDone;
   TestExpectations testExpectations;
 
   JUnitTestSuite(Map this.configuration,
                  String this.suiteName,
                  String this.directoryPath,
                  String this.statusFilePath);
 
-  void isTestFile(String filename) => filename.endsWith("Tests.java") &&
+  bool isTestFile(String filename) => filename.endsWith("Tests.java") &&
       !filename.contains('com/google/dart/compiler/vm') &&
       !filename.contains('com/google/dart/corelib/SharedTests.java');
 
   void forEachTest(Function onTest,
                    Map testCacheIgnored,
                    [Function onDone = null]) {
     doTest = onTest;
     doDone = (onDone != null) ? onDone : (() => null);
     
     if (configuration['component'] != 'dartc') {
@@ skipping 220 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);
   }
 }