DevTools: Timeline: speed-up records processing.

Source/devtools/front_end/timeline/TimelinePanel.js

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2012 Intel 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.
@@ skipping 796 matching lines @@
                 this._lazyFrameModel.addTraceEvents(this._tracingTimelineModel.inspectedTargetEvents(), this._tracingModel.sessionId());
             else
                 this._lazyFrameModel.addRecords(this._model.records());
         }
         if (this._tracingTimelineModel) {
             this.requestWindowTimes(this._tracingTimelineModel.minimumRecordTime(), this._tracingTimelineModel.maximumRecordTime());
             this._refreshViews();
         }
         this._hideProgressPane();
         this._overviewPane.update();
-    },
-
-    _onRecordAdded: function(event)
-    {
-        this._addRecord(/** @type {!WebInspector.TimelineModel.Record} */(event.data));
+        this._updateSearchHighlight(false, true);
     },
 
     /**
-     * @param {!WebInspector.TimelineModel.Record} record
+     * @param {!WebInspector.Event} event
      */
-    _addRecord: function(record)
+    _onRecordAdded: function(event)
     {
+        /** @type {!WebInspector.TimelineModel.Record} */

[apavlov, 2014/11/06 09:22:47]

You should not need this @type annotation, since you already have the @type-cast
for event.data

[loislo, 2014/11/06 09:41:58]

Done.

+        var record = /** @type {!WebInspector.TimelineModel.Record} */ (event.data);
         if (this._lazyFrameModel && !this._tracingModel)
             this._lazyFrameModel.addRecord(record);
         for (var i = 0; i < this._currentViews.length; ++i)
             this._currentViews[i].addRecord(record);
-        this._updateSearchHighlight(false, true);
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _willReloadPage: function(event)
     {
         if (this._operationInProgress || this._userInitiatedRecording || !this.isShowing())
             return;
         this._startRecording(false);
@@ skipping 676 matching lines @@
 
 WebInspector.TimelinePanelFactory.prototype = {
     /**
      * @return {!WebInspector.Panel}
      */
     createPanel: function()
     {
         return WebInspector.TimelinePanel._instance();
     }
 }