DevTools: [Timeline] Reset Memory Overview pane when clear button is clicked

Source/devtools/front_end/TimelineOverviewPane.js

Index: Source/devtools/front_end/TimelineOverviewPane.js
diff --git a/Source/devtools/front_end/TimelineOverviewPane.js b/Source/devtools/front_end/TimelineOverviewPane.js
index 40d9a6b04dc76e611789909866539231038edf7d..71907f7c270e7ad618f26ef3e782cd86a697ce84 100644
--- a/Source/devtools/front_end/TimelineOverviewPane.js
+++ b/Source/devtools/front_end/TimelineOverviewPane.js
@@ -463,12 +463,15 @@ WebInspector.TimelineMemoryOverview = function(model)
 WebInspector.TimelineMemoryOverview.prototype = {
     update: function()
     {
+        this._resetCanvas();
+
+        var ctx = this._context;
+        this._clear(ctx);

[caseq, 2013/07/24 09:07:06]

Why do we need _clear() at all? Seems like it's redundant given we call
_resetCanvas(). Can you drop it?

+
         var records = this._model.records;
         if (!records.length)
             return;
 
-        this._resetCanvas();
-
         const lowerOffset = 3;
         var maxUsedHeapSize = 0;
         var minUsedHeapSize = 100000000000;
@@ -494,9 +497,6 @@ WebInspector.TimelineMemoryOverview.prototype = {
             histogram[x] = Math.max(histogram[x] || 0, y);
         });
 
-        var ctx = this._context;
-        this._clear(ctx);
-
         height++; // +1 so that the border always fit into the canvas area.
 
         var y = 0;