Move all args to per-request for the worker, and add a test

Move all args to per-request for the worker, and add a test

BUG=
R=jmesserly@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/39d038366f9080bf00ab242202a19d212d5372f5

[jakemac, 2016-04-15 21:46:31]

jakemac@google.com changed reviewers:
+ jmesserly@google.com

[Jennifer Messerly, 2016-04-18 18:08:41]

LGTM, nice refactor :)

https://codereview.chromium.org/1896613002/diff/40001/bin/dartdevc.dart
File bin/dartdevc.dart (right):

https://codereview.chromium.org/1896613002/diff/40001/bin/dartdevc.dart#newco...
bin/dartdevc.dart:122: return 1;
BTW, this was probably a bug in my original code, but it'd be nice to give a
different exit code for this one.
No strong preference for what it is. Perhaps 70 for "internal software error":
http://stackoverflow.com/questions/1101957/are-there-any-standard-exit-status...

... that's also where the 64 comes from, I think.

(funny enough, while searching around found this:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=22600)

[jakemac, 2016-04-18 18:14:26]

https://codereview.chromium.org/1896613002/diff/40001/bin/dartdevc.dart
File bin/dartdevc.dart (right):

https://codereview.chromium.org/1896613002/diff/40001/bin/dartdevc.dart#newco...
bin/dartdevc.dart:122: return 1;
On 2016/04/18 18:08:41, John Messerly wrote:
> BTW, this was probably a bug in my original code, but it'd be nice to give a
> different exit code for this one.
> No strong preference for what it is. Perhaps 70 for "internal software error":
>
http://stackoverflow.com/questions/1101957/are-there-any-standard-exit-status...
> 
> ... that's also where the 64 comes from, I think.
> 
> (funny enough, while searching around found this:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=22600)

Done.

[jakemac, 2016-04-18 18:17:11]

Description was changed from

==========
Move all args to per-request for the worker, and add a test

BUG=
==========

to

==========
Move all args to per-request for the worker, and add a test

BUG=
R=jmesserly@google.com

Committed:
https://github.com/dart-lang/dev_compiler/commit/39d038366f9080bf00ab242202a1...
==========

[jakemac, 2016-04-18 18:17:13]

Committed patchset #4 (id:60001) manually as
39d038366f9080bf00ab242202a19d212d5372f5 (presubmit successful).