DevTools: Make functions on timeline tree view point to the right source location

Source/devtools/front_end/timeline/TimelineModel.js

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 1466 matching lines @@
      * @return {boolean}
      */
     function filter(e)
     {
         if (!e.endTime && e.phase !== WebInspector.TracingModel.Phase.Instant)
             return false;
         if (e.endTime <= startTime || e.startTime >= endTime)
             return false;
         if (WebInspector.TracingModel.isAsyncPhase(e.phase))
             return false;
-        for (var filter of filters) {
-            if (!filter.accept(e))
+        for (var i = 0, l = filters.length; i < l; ++i) {

[yurys, 2015/08/29 00:44:18]

did it prove to be slow?

[alph, 2015/09/01 21:38:55]

Yes, for-of still disables optimization and the filter function is very hot.

+            if (!filters[i].accept(e))
                 return false;
         }
         return true;
     }
 
     /**
      * @param {!WebInspector.TracingModel.Event} e
      */
     function onStartEvent(e)
     {
@@ skipping 864 matching lines @@
         /** @type {!Object.<string, !Array.<!WebInspector.InvalidationTrackingEvent>>} */
         this._invalidations = {};
         /** @type {!Object.<number, !Array.<!WebInspector.InvalidationTrackingEvent>>} */
         this._invalidationsByNodeId = {};
 
         this._lastRecalcStyle = undefined;
         this._lastPaintWithLayer = undefined;
         this._didPaint = false;
     }
 }