Changeset 112403 in webkit


Ignore:
Timestamp:
Mar 28, 2012 8:36:03 AM (12 years ago)
Author:
caseq@chromium.org
Message:

Web Inspector: nuke HeapGraph.setSize(), do it automagically on update
https://bugs.webkit.org/show_bug.cgi?id=82471

Reviewed by Yury Semikhatsky.

  • inspector/front-end/TimelineOverviewPane.js:

(WebInspector.TimelineOverviewPane.prototype.update):
(WebInspector.HeapGraph.prototype.update):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r112402 r112403  
     12012-03-28  Andrey Kosyakov  <caseq@chromium.org>
     2
     3        Web Inspector: nuke HeapGraph.setSize(), do it automagically on update
     4        https://bugs.webkit.org/show_bug.cgi?id=82471
     5
     6        Reviewed by Yury Semikhatsky.
     7
     8        * inspector/front-end/TimelineOverviewPane.js:
     9        (WebInspector.TimelineOverviewPane.prototype.update):
     10        (WebInspector.HeapGraph.prototype.update):
     11
    1122012-03-28  Pavel Podivilov  <podivilov@chromium.org>
    213
  • trunk/Source/WebCore/inspector/front-end/TimelineOverviewPane.js

    r111691 r112403  
    197197        WebInspector.TimelinePanel.forAllRecords(records, updateBoundaries.bind(this));
    198198
    199         if (this._heapGraph.visible) {
    200             this._heapGraph.setSize(this._overviewGrid.element.offsetWidth, 60);
     199        if (this._heapGraph.visible)
    201200            this._heapGraph.update(records);
    202         } else if (this._verticalOverview)
     201        else if (this._verticalOverview)
    203202            this._verticalOverview.update(records);
    204203        else
     
    711710    },
    712711
    713     setSize: function(w, h) {
    714         this._canvas.width = w;
    715         this._canvas.height = h - 5;
    716     },
    717 
    718712    update: function(records)
    719713    {
    720714        if (!records.length)
    721715            return;
     716
     717        const yPadding = 5;
     718        this._canvas.width = this.element.clientWidth;
     719        this._canvas.height = this.element.clientHeight - yPadding;
    722720
    723721        const lowerOffset = 3;
Note: See TracChangeset for help on using the changeset viewer.