Make WI.StackedLineChart a WI.View subclass It can eliminate its own needsLayout/updateLayout logic and participate in normal view drawing. Note: - LineChart just moved as well.
Created attachment 360796 [details] [PATCH] Proposed Fix
Comment on attachment 360796 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=360796&action=review rs=me, not sure why this won't apply 🤔, but it looks good to me > Source/WebInspectorUI/UserInterface/Views/LineChart.js:91 > + super.layout(); Oops. Forgot that :P > Source/WebInspectorUI/UserInterface/Views/MemoryTimelineOverviewGraph.js:43 > + this._chart.size = new WI.Size(0, this.height); I don't think this is necessary, given that you're making the width `0`. Not setting a `size` means we don't draw, so it should have the same effect as setting the width to `0`.
https://trac.webkit.org/changeset/240871/webkit
<rdar://problem/47747265>