Add many response types

Add many response types

BUG=
R=turnidge@google.com

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

[turnidge, 2014-08-21 00:33:01]

LGTM w/ comments.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
File runtime/vm/service/protocol.md (right):

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:146: | error | [Error](#Error) | (optional) Error
encountered during class finalization
I have been using ? for optional properties.  What do you think?

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:171: | owningLibrary | [@Library](#atLibrary) |
Set for non-top level functions
optional.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:172: | owningClass | [@Class](#atClass) | Set for
non-top level functions
optional.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:184: | owningClass | [@Class](#atClass) | Set for
non-top level functions
ditto these two.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:207: | value | Instance | (optional) value
associated with static field <-- do we want to include this in a field
reference?
Good catch, probably not.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:209: | owner | [@Class](#atClass) | Owning class
<-- handling of owner is inconsistent with Function
I have been thinking of using a comma when multiple types are allowed:

| owner | [@Library](#atLibrary), [@Class](#atClass)

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:304: | 7 | int | Bytes allocated since last
accumulator reset |
You should do the same thing with TokenLine down below -- say the type is a List
of int and then break out the format separately.

[Cutch, 2014-08-26 20:29:28]

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
File runtime/vm/service/protocol.md (right):

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:146: | error | [Error](#Error) | (optional) Error
encountered during class finalization
On 2014/08/21 00:33:01, turnidge wrote:
> I have been using ? for optional properties.  What do you think?

Done.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:171: | owningLibrary | [@Library](#atLibrary) |
Set for non-top level functions
On 2014/08/21 00:33:01, turnidge wrote:
> optional.

Done.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:172: | owningClass | [@Class](#atClass) | Set for
non-top level functions
On 2014/08/21 00:33:01, turnidge wrote:
> optional.

Done.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:184: | owningClass | [@Class](#atClass) | Set for
non-top level functions
On 2014/08/21 00:33:01, turnidge wrote:
> ditto these two.

Done.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:207: | value | Instance | (optional) value
associated with static field <-- do we want to include this in a field
reference?
On 2014/08/21 00:33:01, turnidge wrote:
> Good catch, probably not.

Acknowledged.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:209: | owner | [@Class](#atClass) | Owning class
<-- handling of owner is inconsistent with Function
On 2014/08/21 00:33:01, turnidge wrote:
> I have been thinking of using a comma when multiple types are allowed:
> 
> | owner | [@Library](#atLibrary), [@Class](#atClass)

Done.

https://codereview.chromium.org/492093002/diff/20001/runtime/vm/service/proto...
runtime/vm/service/protocol.md:304: | 7 | int | Bytes allocated since last
accumulator reset |
On 2014/08/21 00:33:01, turnidge wrote:
> You should do the same thing with TokenLine down below -- say the type is a
List
> of int and then break out the format separately.

Done.

[Cutch, 2014-08-27 16:08:28]

Committed patchset #3 (id:40001) manually as 39601 (presubmit successful).