Add crash dumps to service protocol and Observatory

runtime/observatory/lib/src/app/page.dart

Index: runtime/observatory/lib/src/app/page.dart
diff --git a/runtime/observatory/lib/src/app/page.dart b/runtime/observatory/lib/src/app/page.dart
index 7457b1d3828278ed168587fedbb028e3bea9cc55..e0fc2224eb8587bdb3fb180dc4049201a82102f1 100644
--- a/runtime/observatory/lib/src/app/page.dart
+++ b/runtime/observatory/lib/src/app/page.dart
@@ -113,7 +113,7 @@ class VMPage extends SimplePage {
         serviceElement.object = vm;
       }
     }).catchError((e, stack) {
-      Logger.root.severe('VMPage visit error: $e\n$stack');
+      Logger.root.severe('VMPage visit error: $e');
     });
   }
 }
@@ -274,7 +274,7 @@ class ErrorViewPage extends Page {
   }
 
   // TODO(turnidge): How to test this page?
-  bool canVisit(Uri uri) => uri.path.startsWith('error/');
+  bool canVisit(Uri uri) => uri.path.startsWith('error');

[turnidge, 2015/04/23 17:03:33]

We don't want to capture pages that look like this;  error-of-a-different-kind. 
Other canVisit methods have been updated to look like this:

bool canVisit(Uri uri) => uri.path == 'error';

[Cutch, 2015/04/24 00:05:32]

Done.

 }
 
 class VMConnectPage extends Page {
@@ -292,8 +292,7 @@ class VMConnectPage extends Page {
     assert(canVisit(uri));
   }
 
-  // TODO(turnidge): Update this to not have the trailing slash.
-  bool canVisit(Uri uri) => uri.path.startsWith('vm-connect/');
+  bool canVisit(Uri uri) => uri.path.startsWith('vm-connect');

[turnidge, 2015/04/23 17:03:33]

Ditto comment above.  If you get rid of the '/', you need to switch startsWith
to == and test that it still works.  Otherwise unintended name capture.

[Cutch, 2015/04/24 00:05:32]

Done.

 }
 
 class MetricsPage extends Page {