vm/observatory: Clean up script access

runtime/bin/vmservice/client/lib/src/service/object.dart

Index: runtime/bin/vmservice/client/lib/src/service/object.dart
diff --git a/runtime/bin/vmservice/client/lib/src/service/object.dart b/runtime/bin/vmservice/client/lib/src/service/object.dart
index 044927d990c4bba7c702f6d49ce925c1d51c46a7..0353ae3f872fc0ebee04c993bc04b0c588a3d8f9 100644
--- a/runtime/bin/vmservice/client/lib/src/service/object.dart
+++ b/runtime/bin/vmservice/client/lib/src/service/object.dart
@@ -1338,6 +1338,7 @@ class Script extends ServiceObject with Coverage {
   @observable String kind;
   @observable int firstTokenPos;
   @observable int lastTokenPos;
+  @observable Library owningLibrary;
   bool get canCache => true;
   bool get immutable => true;
 
@@ -1360,6 +1361,7 @@ class Script extends ServiceObject with Coverage {
   Map _tokenToCol = {};
 
   void _update(ObservableMap map, bool mapIsRef) {
+    _upgradeCollection(map, isolate);
     kind = map['kind'];
     _url = map['name'];
     _shortUrl = _url.substring(_url.lastIndexOf('/') + 1);
@@ -1367,6 +1369,7 @@ class Script extends ServiceObject with Coverage {
     vmName = _url;
     _processSource(map['source']);
     _parseTokenPosTable(map['tokenPosTable']);

[Cutch, 2014/07/15 22:06:54]

There should be a "if (mapIsRef) { return; }" statement somewhere in here.

[Michael Lippautz (Google), 2014/07/15 22:40:42]

Done.

+    owningLibrary = map['owning_library'];
   }
 
   void _parseTokenPosTable(List<List<int>> table) {