Use shelf handlers in ScheduledServer.

Use shelf handlers in ScheduledServer.

R=rnystrom@google.com

Committed: https://code.google.com/p/dart/source/detail?r=34713

[Bob Nystrom, 2014-04-02 23:55:30]

https://codereview.chromium.org/216373004/diff/1/pkg/scheduled_test/test/sche...
File pkg/scheduled_test/test/scheduled_server_test.dart (right):

https://codereview.chromium.org/216373004/diff/1/pkg/scheduled_test/test/sche...
pkg/scheduled_test/test/scheduled_server_test.dart:68: // Add a task's worth of
space to avoid hitting the heuristic of waiting
What does "a task's worth of space" mean here?

https://codereview.chromium.org/216373004/diff/1/pkg/scheduled_test/test/sche...
pkg/scheduled_test/test/scheduled_server_test.dart:88: schedule(() => new
Future.delayed(new Duration(seconds: 1)));
1s is pretty brutal. Can this be cranked down a bit?

https://codereview.chromium.org/216373004/diff/1/sdk/lib/_internal/pub/test/l...
File sdk/lib/_internal/pub/test/lish/utils.dart (right):

https://codereview.chromium.org/216373004/diff/1/sdk/lib/_internal/pub/test/l...
sdk/lib/_internal/pub/test/lish/utils.dart:33: headers: {'content-type':
'application/json'});
Does shelf have any explicit support for JSON responses? Plan to add any?

[nweiz, 2014-04-03 00:34:08]

https://codereview.chromium.org/216373004/diff/1/pkg/scheduled_test/test/sche...
File pkg/scheduled_test/test/scheduled_server_test.dart (right):

https://codereview.chromium.org/216373004/diff/1/pkg/scheduled_test/test/sche...
pkg/scheduled_test/test/scheduled_server_test.dart:68: // Add a task's worth of
space to avoid hitting the heuristic of waiting
On 2014/04/02 23:55:30, Bob Nystrom wrote:
> What does "a task's worth of space" mean here?

Clarified.

https://codereview.chromium.org/216373004/diff/1/pkg/scheduled_test/test/sche...
pkg/scheduled_test/test/scheduled_server_test.dart:88: schedule(() => new
Future.delayed(new Duration(seconds: 1)));
On 2014/04/02 23:55:30, Bob Nystrom wrote:
> 1s is pretty brutal. Can this be cranked down a bit?

Done.

https://codereview.chromium.org/216373004/diff/1/sdk/lib/_internal/pub/test/l...
File sdk/lib/_internal/pub/test/lish/utils.dart (right):

https://codereview.chromium.org/216373004/diff/1/sdk/lib/_internal/pub/test/l...
sdk/lib/_internal/pub/test/lish/utils.dart:33: headers: {'content-type':
'application/json'});
On 2014/04/02 23:55:30, Bob Nystrom wrote:
> Does shelf have any explicit support for JSON responses? Plan to add any?

It doesn't. Kevin and I haven't talked about a plan to add it; it's tough to
make something that works well with the existing constructors, since you could
return JSON for any status code.

[Bob Nystrom, 2014-04-03 16:46:32]

lgtm

[nweiz, 2014-04-03 18:24:23]

Committed patchset #2 manually as r34713 (presubmit successful).