Remove a now-unneeded resolveReferences call

There is now a resolveReferences call in the scenemanager whenever
building a scene.

Also, add a note in the autotests since there it is still important to
have an explicit call.

Change-Id: I26d720411c6980c65eb60bbe680cbb10ce8c9cbe
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

1 files changed, 4 insertions, 0 deletions

diff --git a/tests/auto/uipparser/tst_q3dsuipparser.cpp b/tests/auto/uipparser/tst_q3dsuipparser.cpp
index 22e8bec..8f67b97 100644
--- a/tests/auto/uipparser/tst_q3dsuipparser.cpp
+++ b/tests/auto/uipparser/tst_q3dsuipparser.cpp
@@ -809,6 +809,10 @@ void tst_Q3DSUipParser::primitiveMeshes()
 
     // test changing using the "dynamic" setter
     cylinder->applyPropertyChanges({ Q3DSPropertyChange::fromVariant("sourcepath", QLatin1String("#Sphere")) });
+    // For objects that are attached (or will be attached) to a real scene the
+    // resolveReferences call is not necessary since scenemanager will do this
+    // internally. Here however we need to invoke it explicitly in order to see
+    // the change right away.
     cylinder->resolveReferences(*pres.data());
     QCOMPARE(cylinder->mesh(), sphere->mesh());
 }