Profile dynamic calls.

tool/input_sdk/private/ddc_runtime/operations.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.
 
 /// This library defines runtime operations on objects used by the code
 /// generator.
 part of dart._runtime;
 
 _canonicalFieldName(obj, name, args, displayName) => JS('', '''(() => {
   $name = $canonicalMember($obj, $name);
   if ($name) return $name;
   // TODO(jmesserly): in the future we might have types that "overlay" Dart
   // methods while also exposing the full native API, e.g. dart:html vs
   // dart:dom. To support that we'd need to fall back to the normal name
   // if an extension method wasn't found.
   $throwNoSuchMethodFunc($obj, $displayName, $args);
 })()''');
 
 dload(obj, field) => JS('', '''(() => {
+  $_trackCall(obj, $field, null);
   $field = $_canonicalFieldName($obj, $field, [], $field);
   if ($hasMethod($obj, $field)) {
     return $bind($obj, $field);
   }
   // TODO(vsm): Implement NSM robustly.  An 'in' check breaks on certain
   // types.  hasOwnProperty doesn't chase the proto chain.
   // Also, do we want an NSM on regular JS objects?
   // See: https://github.com/dart-lang/dev_compiler/issues/169
   let result = $obj[$field];
   return result;
 })()''');
 
 dput(obj, field, value) => JS('', '''(() => {
   $field = $_canonicalFieldName($obj, $field, [$value], $field);
+  $_trackCall(obj, $field, null);
+
   // TODO(vsm): Implement NSM and type checks.
   // See: https://github.com/dart-lang/dev_compiler/issues/170
   $obj[$field] = $value;
   return $value;
 })()''');
 
 /// Check that a function of a given type can be applied to
 /// actuals.
 _checkApply(type, actuals) => JS('', '''(() => {
   if ($actuals.length < $type.args.length) return false;
@@ skipping 37 matching lines @@
 throwNoSuchMethod(obj, name, pArgs, nArgs, extras) => JS('', '''(() => {
   $throw_(new $NoSuchMethodError($obj, $_dartSymbol($name), $pArgs, $nArgs, $extras));
 })()''');
 
 throwNoSuchMethodFunc(obj, name, pArgs, opt_func) => JS('', '''(() => {
   if ($obj === void 0) $obj = $opt_func;
   $throwNoSuchMethod($obj, $name, $pArgs);
 })()''');
 
 _checkAndCall(f, ftype, obj, typeArgs, args, name) => JS('', '''(() => {
+  $_trackCall(obj, name, f.toString());
+
   let originalFunction = $f;
   if (!($f instanceof Function)) {
     // We're not a function (and hence not a method either)
     // Grab the `call` method if it's not a function.
     if ($f != null) {
       $ftype = $getMethodType($f, 'call');
       $f = $f.call;
     }
     if (!($f instanceof Function)) {
       $throwNoSuchMethodFunc($obj, $name, $args, originalFunction);
@@ skipping 49 matching lines @@
   $throwNoSuchMethodFunc($obj, $name, $args, originalFunction);
 })()''');
 
 dcall(f, @rest args) => _checkAndCall(
     f, _getRuntimeType(f), JS('', 'void 0'), null, args, 'call');
 
 
 dgcall(f, typeArgs, @rest args) => _checkAndCall(
     f, _getRuntimeType(f), JS('', 'void 0'), typeArgs, args, 'call');
 
+Map<String, int> callMethodStats = new Map();

[Jacob, 2016/06/01 01:23:17]

this one is my fault but fix it anyway :)
change this to _callMethodStats.
You should be able to use the refactor support in atom to fix all occurences.

[priscillalee, 2016/06/01 20:38:09]

Acknowledged.

+
+class ProfileEntry {

[Jacob, 2016/06/01 01:23:17]

you can make all fields in this class final.
E.g.
final String key;
instead of 
String key.

[priscillalee, 2016/06/01 20:38:09]

Acknowledged.

+  String key;
+  num count;
+  num total_pct;

[Jacob, 2016/06/01 01:23:17]

See the dart style guide for naming members.
https://www.dartlang.org/effective-dart/style/

for example this member could be totalPercentage
instead of total_pct.
For this particular case we should actually just remove total_pct and
total_pct_div_n 
as they can be computed easily enough in the client application so we don't need
to include them here.
Btw, as a follow up cl I expect we will add more methods to this class such as
lists of the encountered types, etc so having a class for this is worthwhile.

[priscillalee, 2016/06/01 20:38:09]

Acknowledged.

+  num total_pct_div_n;
+
+  ProfileEntry(this.key, this.count, this.total_pct, this.total_pct_div_n);
+}
+
+dumpDynamicStats() {
+  List<ProfileEntry> ret = new List();
+  //StringBuffer sb = new StringBuffer();
+  //sb.write("Stats:");

[Jacob, 2016/06/01 01:23:17]

remove commented out lines.

[priscillalee, 2016/06/01 20:38:09]

Acknowledged.

+  var keys = callMethodStats.keys.toList();
+  num total = 0;

[Jacob, 2016/06/01 01:23:17]

you can remove total now that we are calculating it in a separate UI.

[priscillalee, 2016/06/01 20:38:09]

thanks, I also removed sum.

+  for (var val in callMethodStats.values) {
+    total += val;
+  }
+
+  keys.sort((a, b) => callMethodStats[b].compareTo(callMethodStats[a]));
+  num sum = 0;
+  //sb.write('name, count, total%, total%\n');
+  //sb.write('<all>, $total, 100.00%, 100.0%\n');
+  for (var key in keys) {
+    int count = callMethodStats[key];
+    sum += count;
+    ret.add(new ProfileEntry(key, count, count/total*100, sum/total*100));
+    //sb.write(
+        //"$key, $count, ${(count/total*100).toStringAsFixed(2)}%, ${(sum/total*100).toStringAsFixed(2)}%\n");
+  }
+  //print(sb.toString());
+  return ret;
+}
+
+clearDynamicStats() {
+  callMethodStats.clear();
+}
+
+bool _track_profile = false;

[Jacob, 2016/06/01 01:23:17]

this should be named _trackProfile.

[priscillalee, 2016/06/01 20:38:09]

Acknowledged.

+
+set trackProfile(bool b) {
+  _track_profile = b;

[Jacob, 2016/06/01 01:23:17]

nit: change this from bool v
to 
bool value

[priscillalee, 2016/06/01 20:38:09]

Acknowledged.

+}
+
+get trackProfile => _track_profile;
+
+_trackCall(obj, name, displayName) {
+  if (!_track_profile) return;
+
+  var actual = getReifiedType(obj);
+  String stackStr = JS('String', "new Error().stack");

[Jacob, 2016/06/01 01:23:17]

add
// TODO(jacobr): pass caller information in instead of using the stack trace.

[priscillalee, 2016/06/01 20:38:09]

Acknowledged.

+  var stack = stackStr.split('\n    at ');
+  String src = '';
+  for (int i = 2; i < stack.length; ++i) {
+    var frame = stack[i];
+    if (!frame.contains('dev_compiler/lib/runtime/dart_sdk.js')) {
+      src = frame;
+      break;
+    }
+  }
+
+  name = "${typeName(actual)}.$name <$src>";
+  if (callMethodStats.containsKey(name)) {
+    callMethodStats[name] = callMethodStats[name] + 1;
+  } else {
+    callMethodStats[name] = 1;
+  }
+}
 
 /// Shared code for dsend, dindex, and dsetindex.
 _callMethod(obj, name, typeArgs, args, displayName) {
+  _trackCall(obj, name, displayName);
   var symbol = _canonicalFieldName(obj, name, args, displayName);
   var f = obj != null ? JS('', '#[#]', obj, symbol) : null;
   var ftype = getMethodType(obj, symbol);
   return _checkAndCall(f, ftype, obj, typeArgs, args, displayName);
 }
 
 dsend(obj, method, @rest args) => _callMethod(obj, method, null, args, method);
 
 dgsend(obj, typeArgs, method, @rest args) =>
     _callMethod(obj, method, typeArgs, args, method);
@@ skipping 414 matching lines @@
     constructor(dartIterator) {
       this.dartIterator = dartIterator;
     }
     next() {
       let i = this.dartIterator;
       let done = !i.moveNext();
       return { done: done, value: done ? void 0 : i.current };
     }
   }
 ''');