Handle vmservice messages while at breakpoint.

runtime/vm/debugger.h

Index: runtime/vm/debugger.h
===================================================================
--- runtime/vm/debugger.h	(revision 31070)
+++ runtime/vm/debugger.h	(working copy)
@@ -36,12 +36,16 @@
   RawString* SourceUrl();
   intptr_t LineNumber();
 
-  void GetCodeLocation(Library* lib, Script* script, intptr_t* token_pos);
+  void GetCodeLocation(Library* lib,
+                       Script* script,
+                       intptr_t* token_pos) const;
 
   void Enable();
   void Disable();
   bool IsEnabled() const { return is_enabled_; }
 
+  void PrintToJSONStream(JSONStream* stream) const;

[hausner, 2013/12/17 21:04:33]

Should you import the header that defines JSONStream, or do a forward
declaration. Apparently we are including it indirectly somehow, but I think it's
cleaner if the header is self-contained.

[turnidge, 2013/12/17 21:50:01]

Header is now included in the .cc file.  I have put forward class declarations
in this file.

+
  private:
   void VisitObjectPointers(ObjectPointerVisitor* visitor);
 
@@ -353,6 +357,8 @@
 
   uword GetPatchedStubAddress(uword breakpoint_address);
 
+  void PrintBreakpointsToJSONArray(JSONArray* jsarr) const;

[hausner, 2013/12/17 21:04:33]

ditto

[turnidge, 2013/12/17 21:50:01]

Done.

+
   static bool IsDebuggable(const Function& func);
 
  private: