Support devFS writes via HTTP PUT

Support devFS writes via HTTP PUT

- [x] Allow DevFS writes over HTTP PUT with a gzip compressed payload.
- [x] Add service test for http put writes.

BUG=
R=rmacnak@google.com

Committed: https://github.com/dart-lang/sdk/commit/a67b9c82ee7cc74ca9623843887412768d1eed28

[Cutch, 2016-08-08 15:02:28]

Description was changed from

==========
Support devFS writes via HTTP PUT

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

to

==========
Support devFS writes via HTTP PUT

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

[Cutch, 2016-08-08 15:02:28]

johnmccutchan@google.com changed reviewers:
+ rmacnak@google.com

[Cutch, 2016-08-08 15:47:24]

Description was changed from

==========
Support devFS writes via HTTP PUT

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

to

==========
Support devFS writes via HTTP PUT

- [x] Allow DevFS writes over HTTP PUT with a gzip compressed payload.
- [x] Add service test for http put writes.

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

[rmacnak, 2016-08-08 16:33:42]

https://codereview.chromium.org/2225583002/diff/60001/runtime/bin/vmservice/s...
File runtime/bin/vmservice/server.dart (right):

https://codereview.chromium.org/2225583002/diff/60001/runtime/bin/vmservice/s...
runtime/bin/vmservice/server.dart:169: Object fsNameList;
var or List: fsNameList[0] below is a static warning.

https://codereview.chromium.org/2225583002/diff/60001/runtime/bin/vmservice/s...
runtime/bin/vmservice/server.dart:188: } catch (e) { /* ignore */ }
A write error (out of disk space) would end up here, no?

[Cutch, 2016-08-08 17:22:06]

https://codereview.chromium.org/2225583002/diff/60001/runtime/bin/vmservice/s...
File runtime/bin/vmservice/server.dart (right):

https://codereview.chromium.org/2225583002/diff/60001/runtime/bin/vmservice/s...
runtime/bin/vmservice/server.dart:169: Object fsNameList;
On 2016/08/08 16:33:42, rmacnak wrote:
> var or List: fsNameList[0] below is a static warning.

Done.

https://codereview.chromium.org/2225583002/diff/60001/runtime/bin/vmservice/s...
runtime/bin/vmservice/server.dart:188: } catch (e) { /* ignore */ }
On 2016/08/08 16:33:42, rmacnak wrote:
> A write error (out of disk space) would end up here, no?

DevFS currently doesn't report write errors or out of disk space errors. I could
remove this try and let the service isolate die (like it does when I/O errors
occur in other DevFS code paths).

[rmacnak, 2016-08-08 17:23:06]

lgtm

[Cutch, 2016-08-08 17:35:34]

Description was changed from

==========
Support devFS writes via HTTP PUT

- [x] Allow DevFS writes over HTTP PUT with a gzip compressed payload.
- [x] Add service test for http put writes.

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

to

==========
Support devFS writes via HTTP PUT

- [x] Allow DevFS writes over HTTP PUT with a gzip compressed payload.
- [x] Add service test for http put writes.

BUG=
R=rmacnak@google.com

Committed:
https://github.com/dart-lang/sdk/commit/a67b9c82ee7cc74ca9623843887412768d1eed28
==========

[Cutch, 2016-08-08 17:35:36]

Committed patchset #5 (id:80001) manually as
a67b9c82ee7cc74ca9623843887412768d1eed28 (presubmit successful).