Chromium Code Reviews| Index: third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js |
| diff --git a/third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js b/third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js |
| index e85439fe65bfda124757908bc255fff80bf977ed..c76b09bbc43fdd2a00341e99dd112c6b0f79313a 100644 |
| --- a/third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js |
| +++ b/third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js |
| @@ -337,7 +337,8 @@ Timeline.TimelineFlameChartDataProvider = class { |
| var level = this._currentLevel + openEvents.length; |
| if (flowEventsEnabled) |
| this._appendFlowEvent(e, level); |
| - this._appendEvent(e, level); |
| + if (e.phase !== SDK.TracingModel.Phase.FlowEnd) |
| + this._appendEvent(e, level); |
| if (!isExtension && TimelineModel.TimelineModel.isMarkerEvent(e)) |
| this._timelineData.entryTotalTimes[this._entryData.length] = undefined; |
| @@ -755,6 +756,8 @@ Timeline.TimelineFlameChartDataProvider = class { |
| * @param {number} flowIndex |
| */ |
| function pushEndFlow(event, flowIndex) { |
| + if (flowIndex < 0) |
|
caseq
2017/03/14 02:09:43
How can this happen?
pfeldman
2017/03/14 02:29:24
That's a way to filter out things. Filtering in th
|
| + return; |
| timelineData.flowEndTimes[flowIndex] = event.startTime; |
| timelineData.flowEndLevels[flowIndex] = level; |
| } |