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

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

Issue 180273023: DevTools: encapsulate presentation model in timeline view. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: For landing 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 | « Source/devtools/front_end/CountersGraph.js ('k') | Source/devtools/front_end/TimelineFlameChart.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/MemoryStatistics.js
diff --git a/Source/devtools/front_end/MemoryStatistics.js b/Source/devtools/front_end/MemoryStatistics.js
index 5e31d666abbc407369c38952610dfa8ccf144a5a..c6a2d1e362323f91c8aa1ade9410827df8fc8c3a 100644
--- a/Source/devtools/front_end/MemoryStatistics.js
+++ b/Source/devtools/front_end/MemoryStatistics.js
@@ -63,7 +63,7 @@ WebInspector.MemoryStatistics = function(delegate, model)
// Populate sidebar
this.sidebarElement().createChild("div", "sidebar-tree sidebar-tree-section").textContent = WebInspector.UIString("COUNTERS");
- this._createAllCounters();
+ this.createAllCounters();
}
/**
@@ -251,7 +251,7 @@ WebInspector.MemoryStatistics.prototype = {
throw new Error("Not implemented");
},
- _createAllCounters: function()
+ createAllCounters: function()
{
throw new Error("Not implemented");
},
@@ -399,6 +399,10 @@ WebInspector.MemoryStatistics.prototype = {
refreshRecords: function()
{
+ this.reset();
+ var records = this._model.records();
+ for (var i = 0; i < records.length; ++i)
+ this.addRecord(records[i]);
},
/**
« no previous file with comments | « Source/devtools/front_end/CountersGraph.js ('k') | Source/devtools/front_end/TimelineFlameChart.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698