DevTools: process events from all threads in TimelineTraceEventBindings

DevTools: process events from all threads in TimelineTraceEventBindings

We used to process only main thread events in both TraceEventBindings
and TimelineFrameModel. This drops the filtering by thread from TracingModel
and lets its clients process all events from the inspected target, performing
filtering on their own.
Also, this forks the enum for thread event names from that of timeline model --
they may occasionally overlap, but essentially these are two different enums.

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

[yurys, 2014-05-27 15:41:32]

lgtm

https://codereview.chromium.org/301593002/diff/20001/Source/devtools/front_en...
File Source/devtools/front_end/timeline/TracingModel.js (right):

https://codereview.chromium.org/301593002/diff/20001/Source/devtools/front_en...
Source/devtools/front_end/timeline/TracingModel.js:91: if
(this._needToSortInspectedTargetEvents) {
Can we just sort them once after all events have been processed?

https://codereview.chromium.org/301593002/diff/20001/Source/devtools/front_en...
Source/devtools/front_end/timeline/TracingModel.js:288: this.thread =
payload.tid;
thread -> tid? or we could store a reference to the owning Thread instead.

[caseq, 2014-05-28 08:48:40]

On 2014/05/27 15:41:32, yurys wrote:
> lgtm
> 
>
https://codereview.chromium.org/301593002/diff/20001/Source/devtools/front_en...
> File Source/devtools/front_end/timeline/TracingModel.js (right):
> 
>
https://codereview.chromium.org/301593002/diff/20001/Source/devtools/front_en...
> Source/devtools/front_end/timeline/TracingModel.js:91: if
> (this._needToSortInspectedTargetEvents) {
> Can we just sort them once after all events have been processed?

Done, though this may possibly lead to us returning inconsistent (i.e. unsorted)
events if the method is called before we finish.
> 
>
https://codereview.chromium.org/301593002/diff/20001/Source/devtools/front_en...
> Source/devtools/front_end/timeline/TracingModel.js:288: this.thread =
> payload.tid;
> thread -> tid? or we could store a reference to the owning Thread instead.

Done.

[caseq, 2014-05-28 08:54:05]

The CQ bit was checked by caseq@chromium.org

[commit-bot: I haz the power, 2014-05-28 08:54:13]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/caseq@chromium.org/301593002/40001

[commit-bot: I haz the power, 2014-05-28 10:14:21]

Change committed as 174955