pkg/vm_service_client: add getSourceReport to VMServiceReference

lib/src/source_location.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 import 'breakpoint.dart';
 import 'scope.dart';
 import 'script.dart';
 
-VMSourceLocation newVMSourceLocation(Scope scope,
-    Map json) {
+VMSourceLocation newVMSourceLocation(Scope scope, Map json) {
   if (json == null) return null;
   assert(json["type"] == "SourceLocation");
-  return new VMSourceLocation._(scope, json);
+
+  var script = newVMScriptRef(scope, json["script"]);
+  return new VMSourceLocation._(script, json["tokenPos"], json["endTokenPos"]);
 }
 
+VMSourceLocation newVMSourceLocationFromLocation(

[nweiz, 2016/05/04 23:05:51]

Similarly, I'd call this "FromPositions".

[kevmoo, 2016/05/05 20:46:26]

Done.

+    VMScriptRef script, int tokenPos, int endTokenPos) =>

[nweiz, 2016/05/04 23:05:51]

Indent +4.

[kevmoo, 2016/05/05 20:46:26]

Done.

+    new VMSourceLocation._(script, tokenPos, endTokenPos);
+
 /// A location or span of code in a Dart script.
 class VMSourceLocation implements VMBreakpointLocation {
   /// The script containing the source location.
   final VMScriptRef script;
 
   /// The canonical source URI of [script].
   Uri get uri => script.uri;
 
   /// If this is a span, this represents the start of that span.
   final VMScriptToken token;
 
   /// The final token of the location, or `null` if this is not a span.
   final VMScriptToken end;
 
-  VMSourceLocation._(Scope scope, Map json)
-      : script = newVMScriptRef(scope, json["script"]),
-        token = newVMScriptToken(
-            scope.isolateId, json["script"]["id"], json["tokenPos"]),
-        end = newVMScriptToken(
-            scope.isolateId, json["script"]["id"], json["endTokenPos"]);
+  VMSourceLocation._(VMScriptRef script, int tokenPos, int endTokenPos)

[nweiz, 2016/05/04 23:05:51]

Rather than changing the calling convention for the main constructor, create a
new fromPositions constructor that the new top-level function forwards to.

[kevmoo, 2016/05/05 20:46:26]

Done.

+      : this.script = script,
+        token = newVMScriptTokenFromScript(script, tokenPos),
+        end = newVMScriptTokenFromScript(script, endTokenPos);
 
   String toString() =>
       end == null ? "$script at $token" : "$script from $token to $end";
 }