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

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

Issue 185233002: DevTools: remove timeline presentation model dependency from record. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Review comments addressed. Created 6 years, 10 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
« no previous file with comments | « no previous file | Source/devtools/front_end/TimelinePresentationModel.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/TimelinePanel.js
diff --git a/Source/devtools/front_end/TimelinePanel.js b/Source/devtools/front_end/TimelinePanel.js
index 0f1276833e4d28b0f0c52630c075f81765bcc34c..055816751846ad233ee9a96c30e1e8d1a4cbde64 100644
--- a/Source/devtools/front_end/TimelinePanel.js
+++ b/Source/devtools/front_end/TimelinePanel.js
@@ -68,7 +68,7 @@ WebInspector.TimelinePanel = function()
this._model.addEventListener(WebInspector.TimelineModel.Events.RecordAdded, this._onRecordAdded, this);
// Create presentation model.
- this._presentationModel = new WebInspector.TimelinePresentationModel();
+ this._presentationModel = new WebInspector.TimelinePresentationModel(this._model);
this._durationFilter = new WebInspector.TimelineIsLongFilter();
this._windowFilter = new WebInspector.TimelineWindowFilter();
this._presentationModel.addFilter(this._windowFilter);
@@ -226,15 +226,15 @@ WebInspector.TimelinePanel.prototype = {
switch (mode) {
case WebInspector.TimelinePanel.Mode.Events:
views.overviewView = new WebInspector.TimelineEventOverview(this._model);
- views.mainViews = [new WebInspector.TimelineView(this, this._presentationModel, null)];
+ views.mainViews = [new WebInspector.TimelineView(this, this._model, this._presentationModel, null)];
break;
case WebInspector.TimelinePanel.Mode.Frames:
views.overviewView = new WebInspector.TimelineFrameOverview(this._model, this.frameModel());
- views.mainViews = [new WebInspector.TimelineView(this, this._presentationModel, this.frameModel())];
+ views.mainViews = [new WebInspector.TimelineView(this, this._model, this._presentationModel, this.frameModel())];
break;
case WebInspector.TimelinePanel.Mode.Memory:
views.overviewView = new WebInspector.TimelineMemoryOverview(this._model);
- var timelineView = new WebInspector.TimelineView(this, this._presentationModel, null);
+ var timelineView = new WebInspector.TimelineView(this, this._model, this._presentationModel, null);
views.mainViews = [timelineView];
var memoryStatistics = new WebInspector.CountersGraph(this, this._presentationModel);
views.mainViews.push(memoryStatistics);
« no previous file with comments | « no previous file | Source/devtools/front_end/TimelinePresentationModel.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698