More gracefully handle browser errors.

lib/src/runner/browser/safari.dart

 // Copyright (c) 2015, 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.runner.browser.safari;
 
 import 'dart:async';
 import 'dart:convert';
 import 'dart:io';
 
 import 'package:path/path.dart' as p;
+import 'package:stack_trace/stack_trace.dart';
 
 import '../../util/io.dart';
+import '../../utils.dart';
+import '../application_exception.dart';
 import 'browser.dart';
 
 /// A class for running an instance of Safari.
 ///
 /// Any errors starting or running the process are reported through [onExit].
 class Safari implements Browser {
   /// The underlying process.
   Process _process;
 
   Future get onExit => _onExitCompleter.future;
@@ skipping 28 matching lines @@
 
       return Process.start(executable, [redirect]).then((process) {
         _process = process;
         _onProcessStartedCompleter.complete();
 
         // TODO(nweiz): the browser's standard output is almost always useless
         // noise, but we should allow the user to opt in to seeing it.
         return _process.exitCode;
       });
     }).then((exitCode) {
-      if (exitCode != 0) throw "Safari failed with exit code $exitCode.";
-    }).then(_onExitCompleter.complete)
-        .catchError(_onExitCompleter.completeError);
+      if (exitCode == 0) return null;
+
+      return UTF8.decodeStream(_process.stderr).then((error) {
+        throw new ApplicationException(
+            "Safari failed with exit code $exitCode:\n$error");
+      });
+    }).then(_onExitCompleter.complete).catchError((error, stackTrace) {
+      if (stackTrace == null) stackTrace = new Trace.current();
+      _onExitCompleter.completeError(
+          new ApplicationException(
+              "Failed to start Safari: ${getErrorMessage(error)}."),
+          stackTrace);
+    });
   }
 
   Future close() {
     _onProcessStarted.then((_) => _process.kill());
 
     // Swallow exceptions. The user should explicitly use [onExit] for these.
     return onExit.catchError((_) {});
   }
 }