We weren't properly converting a path to URL format, which mostly broke
Windows but could also break other operating systems if weird characters
were in the path.

See #277

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1196483002.

Instead of providing all test suites to the Engine's constructor, the
engine now has a sink that can be used to add suites over time and
closed to indicate that no more suites are coming. This makes the use
pattern more flexible, and is a step towards supporting a live test
runner and to more memory efficiency.

Reporters no longer wrap the Engine. Instead, they expose a static
`watch` function which subscribes to the events the Engine emits.

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1187103004.

The cleanup logic in the server was waiting for the browser to actually
connect to the browser manager, which caused the browser to exit before
the test runner knew that exit was allowed.

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1179193003.

Closes #274

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1185203003.

This also ensures that a launcher that fails after being closed doesn't
break anything.

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1175163003.

Rather than trying to keep up with the latest and greatest
implementations of Function.toString, this just avoids verifying the
function string values at all.

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1180613008.

R=nweiz@google.com

Review URL: https://codereview.chromium.org//1182333002.

This captures some syntax errors that were previously falling by the wayside,
and cleans up some edge cases in displaying syntax errors in general.

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1172213002.

This reduces a bit of the duplication between the compact and expanded
reporters, and sets the stage for better loading in the future.

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1175753002.

Add support for checked mode

Platform.executable doesn't include the ".exe" extension on Windows,
which causes symlink resolution for it to fail.

Closes #133

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1165313002.

It turns out this was already happening, since "pub run" defaults to
checked mode for all non-global executables. However, this won't always
be the case; see dart-lang/pub#1230.

Closes #130

Since we don't load isolates from code being generated by the main
isolate, there's no need to work around dart-lang/sdk#12617. This
dramatically decreases load times for VM tests.

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1169473005

R=nweiz@google.com

Review URL: https://codereview.chromium.org//1158873005

R=nweiz@google.com

Review URL: https://codereview.chromium.org//1133603005

Polymer tests use a special wrapper that we need to be aware of.

Closes #128

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1152083008

Closes #132

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1148723005

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1158463003

Closes #131

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1162703003

R=kevmoo@google.com, rnystrom@google.com

Review URL: https://codereview.chromium.org//1159923010

This is useful for the same reason as waitForOutstandingCallbacks.

R=kevmoo@google.com, rnystrom@google.com

Review URL: https://codereview.chromium.org//1152083003

Closes #10

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1156493010

This also gives scheduled_test and similar systems a hook for tracking
outstanding callbacks without having to reinvent the wheel.

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1161873007

See http://github.com/dart-lang/sdk/commit/0358f722ba56294f83fa516dc875939c9f518372

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1163593004

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1156603002

Fix a bunch of broken tests.

Closes #120

Convert most Future-based code to async/await

Now that we only support Dart >=1.9, all supported Dart versions support
Isolate.kill().

Now that we use async/await, we're no longer compatible with Dart 1.8.

This moves the logic of starting the server and extracting its port
number to a utility function.

This makes it much easier to implement asynchronous methods and memoize
them or make them reentrant.

Convert JS stack traces into Dart stack traces.

Closes #21

This helps avoid issues where "localhost" means different things on
different platforms.

Closes #100

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1145483003

Don't wait for every suite to load at once—this basically ensures that
once there are enough, later ones will time out. Instead, only load
eight suites at once.

Closes #102

R=kevmoo@google.com

Review URL: https://codereview.chromium.org//1140033003