Miscellaneous Observatory UI improvements

runtime/bin/vmservice/client/lib/src/elements/code_ref.dart

Index: runtime/bin/vmservice/client/lib/src/elements/code_ref.dart
diff --git a/runtime/bin/vmservice/client/lib/src/elements/code_ref.dart b/runtime/bin/vmservice/client/lib/src/elements/code_ref.dart
index 47103d3a3443bdda67623817ffa44fadd1a77541..25c85d4a96b63cd0c544386d56a1416b9d015de4 100644
--- a/runtime/bin/vmservice/client/lib/src/elements/code_ref.dart
+++ b/runtime/bin/vmservice/client/lib/src/elements/code_ref.dart
@@ -6,8 +6,19 @@ library code_ref_element;
 
 import 'package:polymer/polymer.dart';
 import 'service_ref.dart';
+import 'package:observatory/service.dart';
 
 @CustomTag('code-ref')
 class CodeRefElement extends ServiceRefElement {
+  refChanged(oldValue) {
+    super.refChanged(oldValue);
+    notifyPropertyChange(#isDartCode, 0, 1);
+  }
+  @observable bool get isDartCode {

[turnidge, 2014/04/01 19:42:48]

Could this predicate be put in the model object (Code) instead?

You could add:

    @observable Code code => ref;

and use "code.isDartCode" in the html file.  (Also would need to notify property
"code" when "ref" changes?).

Second choice: if for some reason using the model object directly doesn't work,
this getter could be renamed something ui-ish like "shouldShowLink" .

[Cutch, 2014/04/02 18:26:52]

Done.

+    if (ref is Code) {
+      return ref.kind == CodeKind.Dart;
+    }
+    return false;
+  }
   CodeRefElement.created() : super.created();
 }