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

runtime/vm/service/service.md

 # Dart VM Service Protocol 1.0 (Draft 1)
 
 > Please post feedback to the [observatory-discuss group][discuss-list]
 
 This document describes _draft 1_ of _version 1.0_ of the Dart VM
 Service Protocol.  This protocol is used to communicate with a running
 Dart Virtual Machine.
 
 To use the Service Protocol, start the VM with the *--observe* flag.
 The VM will start a webserver which services protocol requests via WebSocket.
@@ skipping 818 matching lines @@
   string name;
 
   // What kind of code object is this?
   CodeKind kind;
 }
 ```
 
 _@Code_ is a reference to a _Code_ object.
 
 ```
-class @Code extends @Object {
+class Code extends @Object {

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

This is a dup of the above definition. Should this have different fields?

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

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]

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]

Acknowledged.

   // A name for this code object.
   string name;
 
   // What kind of code object is this?
   CodeKind kind;
 }
 ```
 
 A _Code_ object represents compiled code in the Dart VM.
 
@@ skipping 279 matching lines @@
 }
 ```
 
 A _Flag_ represents a single VM command line flag.
 
 ### FlagList
 
 ```
 class FlagList extends Response {
   // A list of all flags which are set to default values.
-  unmodifiedFlags []Flag
+  Flag[] unmodifiedFlags;
 
   // A list of all flags which have been modified by the user.
-  modifiedFlags []Flag
+  Flag[] modifiedFlags;
 }
 ```
 
 A _FlagList_ represents the complete set of VM command line flags.
 
 ### Frame
 
 ```
 class Frame extends Response {
   int index;
@@ skipping 10 matching lines @@
 ```
 class @Function extends @Object {
   // The name of this function.
   string name;
 
   // The owner of this field, which can be a Library, Class, or a
   // Function.
   @Library|@Class|@Function owner;
 
   // Is this function static?
-  bool static
+  bool static;
 
   // Is this function const?
   bool const;
-
 }
 ```
 
 An _@Function_ is a reference to a _Function_.
 
 
 ```
-// A Dart language function.

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

Already doc'd below, and none of the other definitions use docs before the
class.

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

Acknowledged.

 class Function extends Object {
   // The name of this function.
   string name;
 
   // The owner of this field, which can be a Library, Class, or a
   // Function.
   @Library|@Class|@Function owner;
 
   // The location of this function in the source code.
   SourceLocation location [optional];
@@ skipping 66 matching lines @@
   //
   // Provided for instance kinds:
   //   TypeParameter
   @Class parameterizedClass [optional];
 
 
   // The pattern of a RegExp instance.
   //
   // Provided for instance kinds:
   //   RegExp
-  @String pattern [optional];
+  String pattern [optional];
 }
 ```
 
 _@Instance_ is a reference to an _Instance_.
 
 ```
 class Instance extends Object {
   // What kind of instance is this?
   InstanceKind kind;
 
@@ skipping 101 matching lines @@
   // The referent of a MirrorReference instance.
   //
   // Provided for instance kinds:
   //   MirrorReference
   @Instance mirrorReferent [optional];
 
   // The pattern of a RegExp instance.
   //
   // Provided for instance kinds:
   //   RegExp
-  @String pattern [optional];
-  
+  String pattern [optional];
+
   // The key for a WeakProperty instance.
   //
   // Provided for instance kinds:
   //   WeakProperty
   @Instance propertyKey [optional];
-  
+
   // The key for a WeakProperty instance.
   //
   // Provided for instance kinds:
   //   WeakProperty
   @Instance propertyValue [optional];
 
   // The type arguments for this type.
   //
   // Provided for instance kinds:
   //   Type
@@ skipping 27 matching lines @@
   //   TypeParameter
   @Instance bound [optional];
 }
 ```
 
 An _Instance_ represents an instance of the Dart language class _Object_.
 
 ### InstanceKind
 
 ```
-enum {
+enum InstanceKind {
   // A general instance of the Dart class Object.
   PlainInstance,
 
   // null instance.
   Null,
 
   // true or false.
   Bool,
 
   // An instance of the Dart class double.
@@ skipping 231 matching lines @@
 ```
 
 A _Null_ object represents the Dart language value null.
 
 ### Object
 
 ```
 class @Object extends Response {
   // A unique identifier for an Object. Passed to the
   // getObject RPC to load this Object.
-  string id
+  string id;
 }
 ```
 
 _@Object_ is a reference to a _Object_.
 
 ```
 class Object extends Response {
   // A unique identifier for an Object. Passed to the
   // getObject RPC to reload this Object.
   //
@@ skipping 113 matching lines @@
 tokenPos | line | column
 -------- | ---- | ------
 100 | 1 | 5
 101 | 1 | 8
 102 | 2 | 7
 
 ### SourceLocation
 
 ```
 class SourceLocation extends Response {
-  // The script contaiinging the source location.
+  // The script containing the source location.
   @Script script;
 
   // The first token of the location.
   int tokenPos;
 
   // The last token of the location if this is a range.
   int endTokenPos [optional];
 }
 ```
 
 The _SourceLocation_ class is used to designate a position or range in
 some script.
 
 ### Stack
 
 ```
-class Stack {
+class Stack extends Response {
   Frame[] frames;
   Message[] messages;
 }
 ```
 
 ### StepOption
 
 ```
 enum StepOption {
   Into,
@@ skipping 83 matching lines @@
 
   // The Dart VM version string.
   string version;
 
   // The process id for the VM.
   string pid;
 
   // The time that the VM started in milliseconds since the epoch.
   //
   // Suitable to pass to DateTime.fromMillisecondsSinceEpoch.
-  int startTime
+  int startTime;
 
   // A list of isolates running in the VM.
-  @Isolate[] isolates
+  @Isolate[] isolates;
 }
 ```
 
 ## Revision History
 
 version | comments
 ------- | --------
 1.0 draft 1 | initial revision
 
 
 [discuss-list]: https://groups.google.com/a/dartlang.org/forum/#!forum/observatory-discuss