Refactor VM service IDs

runtime/bin/vmservice/client/lib/src/observatory_elements/script_ref.dart

+// Copyright (c) 2013, 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.
+
+library script_ref_element;
+
+import 'package:polymer/polymer.dart';
+import 'observatory_element.dart';
+
+@CustomTag('script-ref')
+class ScriptRefElement extends ObservatoryElement {
+  @published Map ref;
+  @published String scriptURL = '';
+  ScriptRefElement.created() : super.created();
+
+  void refChanged(oldValue) {
+    notifyPropertyChange(#url, "", url);
+    notifyPropertyChange(#name, [], name);
+  }
+
+  void scriptURLChanged(oldValue) {
+    notifyPropertyChange(#url, "", url);
+    notifyPropertyChange(#name, [], name);
+  }
+
+  String get url {
+    print('$app $ref');

[turnidge, 2013/12/19 20:39:29]

Do you mean to have this print here?

[Cutch, 2013/12/19 21:53:58]

Removed.

+    if (app != null && ref != null) {
+      return app.locationManager.currentIsolateRelativeLink(ref['id']);
+    } else if (app != null) {
+      return app.locationManager.currentIsolateScriptLink(scriptURL);
+    }

[turnidge, 2013/12/19 20:39:29]

So sometimes we build a script-ref from a 'ref' and sometimes from a
'scriptURL'.  Is this just because a frame doesn't return a ref yet?  If this is
temporary can we add a TODO to remove it?

[Cutch, 2013/12/19 21:53:58]

Yes, it's just because 'frame' outputs a 'url' property instead of a 'script': {
'type': '@Script', ... }.

Added TODO.

+    return '';
+  }
+
+  String get name {
+    return ref != null ? ref['name'] : scriptURL;
+  }
+}