DevTools: Timeline: speed-up records processing.

DevTools: Timeline: speed-up records processing.

It was awfully slow because we updated selection and calculated aggregates for every added record.
So the complexity was at least n^2.
I moved the _updateSearchHighlight call to _onRecordingStopped

BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184909

[loislo, 2014-11-06 09:20:22]

loislo@chromium.org changed reviewers:
+ alph@chromium.org, caseq@chromium.org, pfeldman@chromium.org,
yurys@chromium.org

[apavlov, 2014-11-06 09:22:46]

apavlov@chromium.org changed reviewers:
+ apavlov@chromium.org

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

https://codereview.chromium.org/703113003/diff/1/Source/devtools/front_end/ti...
File Source/devtools/front_end/timeline/TimelinePanel.js (right):

https://codereview.chromium.org/703113003/diff/1/Source/devtools/front_end/ti...
Source/devtools/front_end/timeline/TimelinePanel.js:825: /** @type
{!WebInspector.TimelineModel.Record} */
You should not need this @type annotation, since you already have the @type-cast
for event.data

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

https://codereview.chromium.org/703113003/diff/1/Source/devtools/front_end/ti...
File Source/devtools/front_end/timeline/TimelinePanel.js (right):

https://codereview.chromium.org/703113003/diff/1/Source/devtools/front_end/ti...
Source/devtools/front_end/timeline/TimelinePanel.js:825: /** @type
{!WebInspector.TimelineModel.Record} */
On 2014/11/06 09:22:47, apavlov wrote:
> You should not need this @type annotation, since you already have the
@type-cast
> for event.data

Done.

[vsevik, 2014-11-06 09:59:37]

vsevik@chromium.org changed reviewers:
+ vsevik@chromium.org

[vsevik, 2014-11-06 09:59:37]

lgtm

[loislo, 2014-11-06 10:07:15]

The CQ bit was checked by loislo@chromium.org

[commit-bot: I haz the power, 2014-11-06 10:08:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/703113003/20001

[commit-bot: I haz the power, 2014-11-06 12:01:14]

Committed patchset #2 (id:20001) as 184909