Fixes to the service.md doc. Various minor changes discovered while

Fixes to the service.md doc. Various minor changes discovered while
parsing the file.

BUG=
R=turnidge@google.com

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

[devoncarew, 2015-06-29 18:38:19]

devoncarew@google.com changed reviewers:
+ johnmccutchan@google.com, turnidge@google.com

[devoncarew, 2015-06-29 18:38:20]

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service.md
File runtime/vm/service/service.md (left):

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service....
runtime/vm/service/service.md:1185: // A Dart language function.
Already doc'd below, and none of the other definitions use docs before the
class.

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service.md
File runtime/vm/service/service.md (right):

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service....
runtime/vm/service/service.md:839: class Code extends @Object {
This is a dup of the above definition. Should this have different fields?

[turnidge, 2015-06-29 22:07:01]

lgtm w/ 1 comment.

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service.md
File runtime/vm/service/service.md (left):

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service....
runtime/vm/service/service.md:1185: // A Dart language function.
On 2015/06/29 18:38:20, devoncarew wrote:
> Already doc'd below, and none of the other definitions use docs before the
> class.

Acknowledged.

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service.md
File runtime/vm/service/service.md (right):

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service....
runtime/vm/service/service.md:839: class Code extends @Object {
On 2015/06/29 18:38:20, devoncarew wrote:
> This is a dup of the above definition. Should this have different fields?

This one is a bit odd, since it only differs in its hidden fields.  We should
maintain the @Code/Code split in case we decide to expose the hidden fields. 
Does that work for you?

[devoncarew, 2015-06-29 22:22:22]

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service.md
File runtime/vm/service/service.md (right):

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service....
runtime/vm/service/service.md:839: class Code extends @Object {
On 2015/06/29 22:07:01, turnidge wrote:
> On 2015/06/29 18:38:20, devoncarew wrote:
> > This is a dup of the above definition. Should this have different fields?
> 
> This one is a bit odd, since it only differs in its hidden fields.  We should
> maintain the @Code/Code split in case we decide to expose the hidden fields. 
> Does that work for you?

Yup, totally. I just wanted to make sure that it was expected to have the two
definitions be the same (and that removing the @ on the 2nd definition was
correct). I'll leave my change then - removing the @.

[turnidge, 2015-06-29 22:55:36]

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service.md
File runtime/vm/service/service.md (right):

https://codereview.chromium.org/1220713008/diff/1/runtime/vm/service/service....
runtime/vm/service/service.md:839: class Code extends @Object {
On 2015/06/29 22:22:22, devoncarew wrote:
> On 2015/06/29 22:07:01, turnidge wrote:
> > On 2015/06/29 18:38:20, devoncarew wrote:
> > > This is a dup of the above definition. Should this have different fields?
> > 
> > This one is a bit odd, since it only differs in its hidden fields.  We
should
> > maintain the @Code/Code split in case we decide to expose the hidden fields.

> > Does that work for you?
> 
> Yup, totally. I just wanted to make sure that it was expected to have the two
> definitions be the same (and that removing the @ on the 2nd definition was
> correct). I'll leave my change then - removing the @.

Acknowledged.

[devoncarew, 2015-06-30 02:57:33]

Committed patchset #2 (id:20001) manually as
969d882e6e83cb8ee2f6adba79a8331b9ab21291 (presubmit successful).