statistics: do resizing in UI thread, not render thread

The updatePaintNode() function, which is run on the render
thread detected a geometry change and initiated recalculation
of the chart layout.

This means that plotAreaChanged() was called in two different
thread contexts, which is questionable. Instead, hook into
the geometryChanged() function and recalculate the chart items
there.

This fixes a rendering bug, because the old code would first
delete unneeded items and then rerender the chart. Thus, old
grid and tick items were still visible.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 2 insertions, 2 deletions

diff --git a/stats/statsview.h b/stats/statsview.h
index 0af91b382..3623c01eb 100644
--- a/stats/statsview.h
+++ b/stats/statsview.h
@@ -64,12 +64,12 @@ public:
 private slots:
 	void replotIfVisible();
 private:
-	bool resetChart;
-
 	// QtQuick related things
+	bool backgroundDirty;
 	QRectF plotRect;
 	QSGNode *updatePaintNode(QSGNode *oldNode, QQuickItem::UpdatePaintNodeData *updatePaintNodeData) override;
 
+	void geometryChanged(const QRectF &newGeometry, const QRectF &oldGeometry) override;
 	void plotAreaChanged(const QSizeF &size);
 	void reset(); // clears all series and axes
 	void setAxes(StatsAxis *x, StatsAxis *y);