Disable applyAuthorStyles

runtime/bin/vmservice/client/lib/src/elements/isolate_profile.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 isolate_profile_element;
 
 import 'dart:html';
 import 'observatory_element.dart';
 import 'package:logging/logging.dart';
 import 'package:observatory/service.dart';
 import 'package:observatory/app.dart';
 import 'package:polymer/polymer.dart';
 
 class ProfileCodeTrieNodeTreeRow extends TableTreeRow {
   final ServiceMap profile;
   @reflectable final CodeTrieNode root;
   @reflectable final CodeTrieNode node;
   @reflectable Code get code => node.code;
 
-  static String formatPercent(num a, num total) {
-    var percent = 100.0 * (a / total);
-    return '${percent.toStringAsFixed(2)}%';
-  }
+  @reflectable String tipKind = '';
+  @reflectable String tipTicks = '';
+  @reflectable String tipTime = '';

[turnidge, 2014/04/17 20:18:54]

Is the popup tip something that should be in a custom element?  Just asking.

[Cutch, 2014/04/17 22:12:39]

I'd like that at some point.

 
   ProfileCodeTrieNodeTreeRow(this.profile, this.root, this.node,
                              ProfileCodeTrieNodeTreeRow parent)
       : super(parent) {
     assert(root != null);
     assert(node != null);
     var totalSamples = root.count;
-    // When the row is created, fill out the columns.
-    if (parent == null) {
-      columns.add(formatPercent(node.count, root.count));
+    tipTicks = '${node.count}';
+    var period = profile['period'];
+    var MICROSECONDS_PER_SECOND = 1000000.0;
+    var seconds = (period * node.count) / MICROSECONDS_PER_SECOND; // seconds
+    tipTime = Utils.formatTimePrecise(seconds);
+    if (code.kind == CodeKind.Tag) {
+      columns.add(Utils.formatPercent(node.count, root.count));
+      columns.add(Utils.formatPercent(node.count, root.count));
+      tipKind = 'Tag (category)';
     } else {
-      columns.add(formatPercent(node.count, parent.node.count));
+      if (code.kind == CodeKind.Collected) {
+        tipKind = 'Code is gone (collected)';
+      } else if (code.kind == CodeKind.Reused) {
+        tipKind = 'Code is gone (collected then overwritten)';
+      } else {
+        tipKind = '${code.kind} (Function)';
+      }
+      if (parent == null) {
+        columns.add(Utils.formatPercent(node.count, root.count));
+      } else {
+        columns.add(Utils.formatPercent(node.count, parent.node.count));
+      }
+      columns.add(Utils.formatPercent(node.code.exclusiveTicks, totalSamples));
     }
-    columns.add(formatPercent(node.code.exclusiveTicks, totalSamples));
   }
 
   bool shouldDisplayChild(CodeTrieNode childNode, double threshold) {
     return ((childNode.count / node.count) > threshold) ||
             ((childNode.code.exclusiveTicks / root.count) > threshold);
   }
 
   void onShow() {
     var threshold = profile['threshold'];
     if (children.length > 0) {
       // Child rows already created.
       return;
     }
     for (var childNode in node.children) {
       if (!shouldDisplayChild(childNode, threshold)) {
         continue;
       }
       var row = new ProfileCodeTrieNodeTreeRow(profile, root, childNode, this);
       children.add(row);
     }
   }
 
   void onHide() {
   }
 }
 
-class ProfileCallerTreeRow extends TableTreeRow {
-  final ServiceMap profile;
-  @reflectable final Code code;
-
-  static String formatPercent(num a, num total) {
-    var percent = 100.0 * (a / total);
-    return '${percent.toStringAsFixed(2)}%';
-  }
-
-  ProfileCallerTreeRow(this.profile, this.code, ProfileCallerTreeRow parent) :
-      super(parent) {
-    assert(profile != null);
-    assert(code != null);
-    var totalSamples = profile['samples'];
-    // When the row is created, fill out the columns.
-    if (parent == null) {
-      var root = profile.isolate.tagRoot();
-      var totalAttributedCalls = root.callersCount(code);
-      var totalParentCalls = root.sumCallersCount();
-      columns.add(formatPercent(totalAttributedCalls, totalParentCalls));
-    } else {
-      var totalAttributedCalls = parent.code.callersCount(code);
-      var totalParentCalls = parent.code.sumCallersCount();
-      columns.add(formatPercent(totalAttributedCalls, totalParentCalls));
-    }
-    columns.add(formatPercent(code.exclusiveTicks, totalSamples));
-  }
-
-  bool shouldDisplayChild(CodeCallCount childNode, totalSamples,
-                          double threshold) {
-    var callerPercent = code.callersCount(childNode.code) /
-                        code.sumCallersCount();
-    return (callerPercent > threshold) ||
-            ((childNode.code.exclusiveTicks / totalSamples) > threshold);
-  }
-
-  void onShow() {
-    var threshold = profile['threshold'];
-    var totalSamples = profile['samples'];
-    if (children.length > 0) {
-      // Child rows already created.
-      return;
-    }
-    for (var codeCaller in code.callers) {
-      if (!shouldDisplayChild(codeCaller, totalSamples, threshold)) {
-        continue;
-      }
-      var row = new ProfileCallerTreeRow(profile, codeCaller.code, this);
-      children.add(row);
-    }
-  }
-
-  void onHide() {
-  }
-}
-
 /// Displays an IsolateProfile
 @CustomTag('isolate-profile')
 class IsolateProfileElement extends ObservatoryElement {
   IsolateProfileElement.created() : super.created();
   @published ServiceMap profile;
-  @observable bool callGraphChecked;
   @observable bool hideTagsChecked;
   @observable String sampleCount = '';
   @observable String refreshTime = '';
   @observable String sampleRate = '';
   @observable String sampleDepth = '';
   @observable String displayCutoff = '';
-  @reflectable double displayThreshold = 0.0001; // 0.5%.
+  @observable String timeSpan = '';
+  @reflectable double displayThreshold = 0.0002; // 0.02%.
 
   final _id = '#tableTree';
   TableTree tree;
 
   static const MICROSECONDS_PER_SECOND = 1000000.0;
 
   void profileChanged(oldValue) {
     if (profile == null) {
       return;
     }
     var totalSamples = profile['samples'];
     var now = new DateTime.now();
     sampleCount = totalSamples.toString();
     refreshTime = now.toString();
     sampleDepth = profile['depth'].toString();
     var period = profile['period'];
     sampleRate = (MICROSECONDS_PER_SECOND / period).toStringAsFixed(0);
+    timeSpan = formatTime(profile['timeSpan']);
     displayCutoff = '${(displayThreshold * 100.0).toString()}%';
     profile.isolate.processProfile(profile);
     profile['threshold'] = displayThreshold;
     _update();
   }
 
-  void callGraphCheckedChanged(oldValue) {
-    _update();
-  }
-
 
   void enteredView() {
     tree = new TableTree();
     _update();
   }
 
   void hideTagsCheckedChanged(oldValue) {
     refresh(null);
   }
 
   void refresh(var done) {
     var request = 'profile';
     if ((hideTagsChecked != null) && hideTagsChecked) {
       request += '?tags=hide';
     }
     profile.isolate.get(request).then((ServiceMap m) {
       // Assert we got back the a profile.
       assert(m.serviceType == 'Profile');
       profile = m;
     }).whenComplete(done);
   }
 
   void _update() {
     if (profile == null) {
       return;
     }
     _buildTree();
   }
 
-  void _buildCallersTree() {
-    assert(profile != null);
-    var root = profile.isolate.tagRoot();
-    if (root == null) {
-      Logger.root.warning('No profile root tag.');
-    }
-    try {
-      tree.initialize(new ProfileCallerTreeRow(profile, root, null));
-    } catch (e, stackTrace) {
-      Logger.root.warning('_buildCallersTree', e, stackTrace);
-    }
-
-    notifyPropertyChange(#tree, null, tree);
-  }
-
   void _buildStackTree() {
     var root = profile.isolate.profileTrieRoot;
     if (root == null) {
-      Logger.root.warning('No profile trie root.');
+      return;
     }
     try {
       tree.initialize(
           new ProfileCodeTrieNodeTreeRow(profile, root, root, null));
     } catch (e, stackTrace) {
       Logger.root.warning('_buildStackTree', e, stackTrace);
     }
     notifyPropertyChange(#tree, null, tree);
   }
 
   void _buildTree() {
-    if ((callGraphChecked) != null && callGraphChecked) {
-      _buildCallersTree();
-    } else {
-      _buildStackTree();
-    }
+    _buildStackTree();
   }
 
   @observable String padding(TableTreeRow row) {
     return 'padding-left: ${row.depth * 16}px;';
   }
 
   @observable String coloring(TableTreeRow row) {
-    const colors = const ['active', 'success', 'warning', 'danger', 'info'];
-    var index = row.depth % colors.length;
+    const colors = const ['color0', 'color1', 'color2', 'color4', 'color5',

[turnidge, 2014/04/17 20:18:54]

What happened to color3? 

We should probably name these styles something more specific... maybe rowColor0,
rowColor1... dunno.

Also, if the numbers were known dense, you could avoid the lookup array:

const int maxColor = X;
var d = row.depth - 1;
return "rowColor${d % maxColor}";

[Cutch, 2014/04/17 22:12:39]

Fixed.
Cheaper to index into an array of constant strings than to create a new one.

+                          'color6', 'color7', 'color8'];
+    var d = row.depth - 1;
+    var index = d % colors.length;
     return colors[index];
   }
 
   @observable void toggleExpanded(Event e, var detail, Element target) {
     // We only want to expand a tree row if the target of the click is
     // the table cell (passed in as target) or the span containing the
     // expander symbol (#expand).
     if ((e.target.id != 'expand') && (e.target != target)) {
       // Target of click was not the expander span or the table cell.
       return;
     }
     var row = target.parent;
     if (row is TableRowElement) {
       // Subtract 1 to get 0 based indexing.
-      tree.toggle(row.rowIndex - 1);
+      try {
+        tree.toggle(row.rowIndex - 1);
+      }  catch (e, stackTrace) {
+        Logger.root.warning('toggleExpanded', e, stackTrace);
+      }
     }
   }
 }