bots: include webdriver tests in bots.

tools/testing/dart/test_suite.dart

Index: tools/testing/dart/test_suite.dart
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index f9878213ea734860798fdfe20a016170f90d4bc6..da0aa53b148a0a250b18026e28d4f9da052073ae 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -463,27 +463,7 @@ class StandardTestSuite implements TestSuite {
       List<String> args;
       if (component == 'webdriver') {
         executable = '$dartDir/tools/testing/run_selenium.py';
-        String browserFlag = 'chrome';
-        if (configuration['flag'] != null) {
-          for (var flag in configuration['flag'].split(',')) {
-            switch (flag) {
-              case 'ff':
-              case 'firefox':
-                browserFlag = 'ff';
-                break;
-              case 'ie':
-              case 'explorer':
-              case 'internet-explorer':
-                browserFlag = 'ie';
-                break;
-              case 'safari':
-                browserFlag = 'safari';
-                break;
-            }
-          }
-        }
-        args = ['--out', htmlPath, '--browser', browserFlag];
-
+        args = ['--out', htmlPath, '--browser', configuration['browser']];

[Emily Fortuna, 2012/01/11 18:59:45]

What if the browser flag isn't specified? Previously, we defaulted to chrome if
we specified webdriver without a flag. But throwing an error is also valid. 

[Siggi Cherem (dart-lang), 2012/01/11 19:55:22]

Turns out I made the 'browser' flag in test_options.dart have 'chrome' as the
default.

[Emily Fortuna, 2012/01/11 20:29:06]

Gotcha! I forgot about that. Carry on.

On 2012/01/11 19:55:22, sigmund wrote:

       } else {
         args = ['--no-timeout'];
         if (component == 'dartium') {