Move selection labels to renderer from render items

Creating a new label on demand is fast enough, so we don't need to
store old labels. Storing labels unnecessarily leads to unchecked
use of graphics memory.

Optimized bar slices a bit by having separate class of render items
for slice items.

Change-Id: I5afdecfb3949393341b6c3eaefbc667c65da32e3
Reviewed-by: Tomi Korpipää <tomi.korpipaa@digia.com>

1 files changed, 4 insertions, 3 deletions

diff --git a/src/datavisualization/engine/drawer.cpp b/src/datavisualization/engine/drawer.cpp
index b70d7c0e..55a2c2a5 100644
--- a/src/datavisualization/engine/drawer.cpp
+++ b/src/datavisualization/engine/drawer.cpp
@@ -24,6 +24,7 @@
 #include "q3dcamera.h"
 #include "utils_p.h"
 #include "texturehelper_p.h"
+#include "abstract3drenderer_p.h"
 
 #include <QtGui/QMatrix4x4>
 #include <QtCore/qmath.h>
@@ -349,10 +350,10 @@ void Drawer::drawLabel(const AbstractRenderItem &item, const LabelItem &labelIte
     drawObject(shader, object, labelItem.textureId());
 }
 
-void Drawer::generateSelectionLabelTexture(AbstractRenderItem *item)
+void Drawer::generateSelectionLabelTexture(Abstract3DRenderer *renderer)
 {
-    LabelItem &labelItem = item->selectionLabelItem();
-    generateLabelItem(labelItem, item->selectionLabel());
+    LabelItem &labelItem = renderer->selectionLabelItem();
+    generateLabelItem(labelItem, renderer->selectionLabel());
 }
 
 void Drawer::generateLabelItem(LabelItem &item, const QString &text, int widestLabel)