Timeline: nuke flow events experiment

Timeline: nuke flow events experiment

This removes timelineFlowEvents experiment that used to enable top-level
flow events and visualize these with arrows on flame chart.

Committed: https://crrev.com/f23c9c9ec07dcbdf68ee5033cc731274e38d0d50
Cr-Commit-Position: refs/heads/master@{#433359}

[caseq, 2016-11-18 19:57:06]

Description was changed from

==========
Timeline: nuke flow events experiment

This removes timelineFlowEvents experiment that used to enable top-level
flow events and visualize these with arrows.
==========

to

==========
Timeline: nuke flow events experiment

This removes timelineFlowEvents experiment that used to enable top-level
flow events and visualize these with arrows on flame chart.
==========

[caseq, 2016-11-18 19:57:06]

caseq@chromium.org changed reviewers:
+ alph@chromium.org, dgozman@chromium.org

[caseq, 2016-11-18 19:57:19]

ptal

[dgozman, 2016-11-18 20:07:08]

https://codereview.chromium.org/2517463005/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js
(left):

https://codereview.chromium.org/2517463005/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js:621:
timelineData.flowStartTimes.push(event.startTime);
Do you want to remove support from FlameChart as well?

[caseq, 2016-11-18 22:09:53]

Patchset #2 (id:20001) has been deleted

[caseq, 2016-11-18 22:10:08]

On 2016/11/18 20:07:08, dgozman wrote:
>
https://codereview.chromium.org/2517463005/diff/1/third_party/WebKit/Source/d...
> File
third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js
> (left):
> 
>
https://codereview.chromium.org/2517463005/diff/1/third_party/WebKit/Source/d...
>
third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js:621:
> timelineData.flowStartTimes.push(event.startTime);
> Do you want to remove support from FlameChart as well?

Ouch. Thanks for spotting. Done, ptal.

[dgozman, 2016-11-18 23:22:34]

lgtm

[caseq, 2016-11-18 23:22:53]

The CQ bit was checked by caseq@chromium.org

[commit-bot: I haz the power, 2016-11-18 23:23:55]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[alph, 2016-11-18 23:48:51]

lgtm

[commit-bot: I haz the power, 2016-11-19 00:57:26]

Description was changed from

==========
Timeline: nuke flow events experiment

This removes timelineFlowEvents experiment that used to enable top-level
flow events and visualize these with arrows on flame chart.
==========

to

==========
Timeline: nuke flow events experiment

This removes timelineFlowEvents experiment that used to enable top-level
flow events and visualize these with arrows on flame chart.
==========

[commit-bot: I haz the power, 2016-11-19 00:57:28]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-11-19 01:00:48]

Description was changed from

==========
Timeline: nuke flow events experiment

This removes timelineFlowEvents experiment that used to enable top-level
flow events and visualize these with arrows on flame chart.
==========

to

==========
Timeline: nuke flow events experiment

This removes timelineFlowEvents experiment that used to enable top-level
flow events and visualize these with arrows on flame chart.

Committed: https://crrev.com/f23c9c9ec07dcbdf68ee5033cc731274e38d0d50
Cr-Commit-Position: refs/heads/master@{#433359}
==========

[commit-bot: I haz the power, 2016-11-19 01:00:50]

Patchset 2 (id:??) landed as
https://crrev.com/f23c9c9ec07dcbdf68ee5033cc731274e38d0d50
Cr-Commit-Position: refs/heads/master@{#433359}