Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1651)

Unified Diff: Source/devtools/front_end/timeline/TimelinePanel.js

Issue 320523004: Timeline: fill frames according to CPU time spent per event category (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebased Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/devtools/front_end/timeline/TimelinePanel.js
diff --git a/Source/devtools/front_end/timeline/TimelinePanel.js b/Source/devtools/front_end/timeline/TimelinePanel.js
index 2864956fe19ff27d23a67c639816bca91c2931ca..f3a7ec6e6c9ce3eb7905e1da9ca3deea9b41137a 100644
--- a/Source/devtools/front_end/timeline/TimelinePanel.js
+++ b/Source/devtools/front_end/timeline/TimelinePanel.js
@@ -38,6 +38,7 @@ importScript("TimelineModelImpl.js");
importScript("TimelineJSProfile.js");
importScript("TimelineOverviewPane.js");
importScript("TimelinePresentationModel.js");
+importScript("TracingTimelineModel.js");
importScript("TimelineFrameModel.js");
importScript("TimelineEventOverview.js");
importScript("TimelineFrameOverview.js");
@@ -50,7 +51,6 @@ importScript("TimelineView.js");
importScript("TimelineTracingView.js");
importScript("TimelineLayersView.js");
importScript("TracingModel.js");
-importScript("TracingTimelineModel.js");
importScript("TracingTimelineUIUtils.js");
importScript("TransformController.js");
@@ -681,13 +681,13 @@ WebInspector.TimelinePanel.prototype = {
_onTracingComplete: function()
{
+ if (WebInspector.experimentsSettings.timelineOnTraceEvents.isEnabled())
+ this._model.didStopRecordingTraceEvents(this._tracingTimelineModel.mainThreadEvents());
if (this._lazyFrameModel) {
this._lazyFrameModel.reset();
this._lazyFrameModel.addTraceEvents(this._tracingTimelineModel.inspectedTargetEvents(), this._tracingModel.sessionId());
this._overviewPane.update();
}
- if (WebInspector.experimentsSettings.timelineOnTraceEvents.isEnabled())
- this._model.didStopRecordingTraceEvents(this._tracingTimelineModel.mainThreadEvents());
this._refreshViews();
},
« no previous file with comments | « Source/devtools/front_end/timeline/TimelineFrameModel.js ('k') | Source/devtools/front_end/timeline/TracingModel.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698