Add browser path options to testing scripts.

tools/testing/dart/utils.dart

 // Copyright (c) 2013, 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 utils;
 
 import 'dart:async';
 import 'dart:io';
 import 'dart:math' show min;
 import 'dart:convert';
@@ skipping 108 matching lines @@
 }
 
 // TODO(kustermann,ricow): As soon we have a debug log we should log
 // invalid utf8-encoded input to the log.
 // Currently invalid bytes will be replaced by a replacement character.
 String decodeUtf8(List<int> bytes) {
   return UTF8.decode(bytes, allowMalformed: true);
 }
 
 class Locations {
-  static String getDartiumLocation(Map globalConfiguration) {
-    var dartium = globalConfiguration['dartium'];
-    if (dartium != null && dartium != '') {
-      return dartium;
+  static String getBrowserLocation(String browserName,
+                                   Map globalConfiguration) {
+    var location = globalConfiguration[browserName];
+    if (location != null && location != '') {
+      return location;
     }
-    if (Platform.operatingSystem == 'macos') {
-      return new Path('client/tests/dartium/Chromium.app/Contents/'
-          'MacOS/Chromium').toNativePath();
+    final browserLocations = const {
+        'firefox': const {
+          'windows': 'C:\\Program Files (x86)\\Mozilla Firefox\\firefox.exe',
+          'linux': 'firefox',
+          'macos': '/Applications/Firefox.app/Contents/MacOS/firefox'
+        },
+        'chrome': const {
+          'windows':
+            'C:\\Program Files (x86)\\Google\\Chrome\\Application\\chrome.exe',
+          'macos':
+            '/Applications/Google Chrome.app/Contents/MacOS/Google Chrome',
+          'linux': 'google-chrome'
+        },
+        'dartium': const {
+          'windows': 'client\\tests\\dartium\\chrome.exe',
+          'macos': 'client/tests/dartium/Chromium.app/Contents/MacOS/Chromium',
+          'linux': 'client/tests/dartium/chrome'
+        },
+        'safari': const {
+          'macos': '/Applications/Safari.app/Contents/MacOS/Safari'
+        },
+        'ie9': const {
+          'windows': 'C:\\Program Files\\Internet Explorer\\iexplore.exe'
+        },
+        'ie10': const {
+          'windows': 'C:\\Program Files\\Internet Explorer\\iexplore.exe'
+        }};
+
+    assert(browserLocations[browserName] != null);
+    location = browserLocations[browserName][Platform.operatingSystem];
+    if (location != null) {
+      return location;
+    } else {
+      throw '$browserName not supported on ${Platform.operatingSystem}';
     }
-    return new Path('client/tests/dartium/chrome').toNativePath();
   }
 }
 
 // This function is pretty stupid and only puts quotes around an argument if
 // it the argument contains a space.
 String escapeCommandLineArgument(String argument) {
   if (argument.contains(' ')) {
     return '"$argument"';
   }
   return argument;
@@ skipping 11 matching lines @@
 
 class UniqueObject {
   static int _nextId = 1;
   final int _hashCode;
 
   int get hashCode => _hashCode;
   operator==(other) => other is UniqueObject && _hashCode == other._hashCode;
 
   UniqueObject() : _hashCode = ++_nextId;
 }