QGraphicsScene: respect that items can override selection changes

QGraphicsItems may override itemChange to prevent certain attribute
changes. Overriding ItemSelectedChange this way is explicitly documented
to be allowed.

However QGraphicsScene::clearSelection did not test whether items were
in fact deselected after the call to setSelection, and always cleared
the stored set of selected items.

Fix this by checking the actual selected state of the item as we iterate
over them, and store those items that are still selected in a set that
becomes the new selectedItems set (which will be empty if no item
overrides, which is the default).

Add a test that also checks that clearing the selection emits the
selectionChanged signal correctly (and does not if all selected items
block being deselected).

Fixes: QTBUG-85474
Change-Id: I665afc132876e02e6e1061b7be37f4f6e4be418f
Reviewed-by: Axel Spoerl <axel.spoerl@qt.io>
Reviewed-by: Richard Moe Gustavsen <richard.gustavsen@qt.io>
(cherry picked from commit 2e12479e06d6869f568f0a7ee939453cda9afdbf)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>

1 files changed, 48 insertions, 0 deletions

diff --git a/tests/auto/widgets/graphicsview/qgraphicsscene/tst_qgraphicsscene.cpp b/tests/auto/widgets/graphicsview/qgraphicsscene/tst_qgraphicsscene.cpp
index cc92f3b65b..e639da3e7c 100644
--- a/tests/auto/widgets/graphicsview/qgraphicsscene/tst_qgraphicsscene.cpp
+++ b/tests/auto/widgets/graphicsview/qgraphicsscene/tst_qgraphicsscene.cpp
@@ -253,6 +253,7 @@ private slots:
     void focusItemChangedSignal();
     void minimumRenderSize();
     void focusOnTouch();
+    void clearSelection();
 
     // task specific tests below me
     void task139710_bspTreeCrash();
@@ -4834,6 +4835,53 @@ void tst_QGraphicsScene::focusOnTouch()
     QVERIFY(rect->hasFocus());
 }
 
+void tst_QGraphicsScene::clearSelection()
+{
+    class AlwaysSelectedItem : public QGraphicsRectItem
+    {
+    public:
+        using QGraphicsRectItem::QGraphicsRectItem;
+    protected:
+        QVariant itemChange(GraphicsItemChange change, const QVariant& value)
+        {
+            if (change == ItemSelectedChange)
+                return true;
+            return QGraphicsRectItem::itemChange(change, value);
+        }
+    };
+    QGraphicsScene scene;
+    QSignalSpy spy(&scene, &QGraphicsScene::selectionChanged);
+
+    QGraphicsRectItem *regularRect = new QGraphicsRectItem;
+    regularRect->setFlag(QGraphicsItem::ItemIsSelectable);
+    regularRect->setRect(0, 0, 50, 50);
+    regularRect->setSelected(true);
+    AlwaysSelectedItem *selectedRect = new AlwaysSelectedItem;
+    selectedRect->setFlag(QGraphicsItem::ItemIsSelectable);
+    selectedRect->setRect(50, 50, 50, 50);
+    selectedRect->setSelected(true);
+    scene.addItem(regularRect);
+    scene.addItem(selectedRect);
+
+    QCOMPARE(spy.count(), 2);
+
+    QCOMPARE(scene.selectedItems().count(), 2);
+    scene.clearSelection();
+    QVERIFY(!regularRect->isSelected());
+    QVERIFY(selectedRect->isSelected());
+    QCOMPARE(scene.selectedItems().count(), 1);
+    QCOMPARE(spy.count(), 3);
+
+    delete regularRect;
+    QCOMPARE(spy.count(), 3);
+
+    scene.clearSelection();
+    QCOMPARE(spy.count(), 3);
+
+    delete selectedRect;
+    QCOMPARE(spy.count(), 4);
+}
+
 void tst_QGraphicsScene::taskQTBUG_15977_renderWithDeviceCoordinateCache()
 {
     QGraphicsScene scene;