diff options
author | Paul Lemire <paul.lemire@kdab.com> | 2018-07-11 11:18:24 +0200 |
---|---|---|
committer | Paul Lemire <paul.lemire@kdab.com> | 2018-07-18 13:44:26 +0000 |
commit | cdf92a8ba06c3df17a11c8931d540a4bf9229f61 (patch) | |
tree | 4ef115907f603ac321f4bfc97fb6dcecc1637603 /tests/auto/render/texture | |
parent | ee53b5366eb925b2fbe3cd28209e2f8c7bc9e143 (diff) |
Fix: Do not enforce TextureImage to be parented only by Texture
This behavior prevented using TextureImage not directly parented by the
Texture that uses them (assert would be triggered). In turn, this also
prevents sharing a TextureImage among several Texture instances which is
counter productive since this is where the data is actually stored.
This patch fixes this issue. It removes all direct coupling between Texture
and TextureImages. Now Texture only contains the list of TextureImage ids it
references. This allows to not make look-ups into the TextureImageManager to
retrieve handles, which could be an issue if TextureImages have not yet had
their backend created. TextureImage doesn't keep track of the referencing texture
that uses it anymore. Instead, we let the renderer do the job of checking if any of
the TextureImage referenced by a Texture has changed to trigger actual Texture
update.
Change-Id: I3c63379d0f4b314e9b53f225870eeaded0bb4aec
Task-number: QTBUG-69407
Reviewed-by: Mike Krus <mike.krus@kdab.com>
Diffstat (limited to 'tests/auto/render/texture')
-rw-r--r-- | tests/auto/render/texture/tst_texture.cpp | 66 |
1 files changed, 66 insertions, 0 deletions
diff --git a/tests/auto/render/texture/tst_texture.cpp b/tests/auto/render/texture/tst_texture.cpp index 784186690..2e94cda24 100644 --- a/tests/auto/render/texture/tst_texture.cpp +++ b/tests/auto/render/texture/tst_texture.cpp @@ -29,6 +29,8 @@ #include <QtTest/QTest> #include <qbackendnodetester.h> #include <Qt3DCore/qdynamicpropertyupdatedchange.h> +#include <Qt3DCore/qpropertynodeaddedchange.h> +#include <Qt3DCore/qpropertynoderemovedchange.h> #include <Qt3DRender/private/texture_p.h> #include "testpostmanarbiter.h" @@ -57,6 +59,7 @@ private slots: void checkFrontendPropertyNotifications(); void checkPropertyMirroring(); void checkPropertyChanges(); + void checkTextureImageBookeeping(); }; void tst_RenderTexture::checkFrontendPropertyNotifications() @@ -165,6 +168,31 @@ void tst_RenderTexture::checkFrontendPropertyNotifications() // THEN QCOMPARE(arbiter.events.size(), 0); + + // WHEN + Qt3DRender::QTextureImage img; + texture.addTextureImage(&img); + + // THEN + QCOMPARE(arbiter.events.size(), 1); + const auto addedChange = arbiter.events.first().staticCast<Qt3DCore::QPropertyNodeAddedChange>(); + QCOMPARE(addedChange->propertyName(), "textureImage"); + QCOMPARE(addedChange->addedNodeId(), img.id()); + QCOMPARE(addedChange->type(), Qt3DCore::PropertyValueAdded); + + arbiter.events.clear(); + + // WHEN + texture.removeTextureImage(&img); + + // THEN + QCOMPARE(arbiter.events.size(), 1); + const auto removedChange = arbiter.events.first().staticCast<Qt3DCore::QPropertyNodeRemovedChange>(); + QCOMPARE(removedChange->propertyName(), "textureImage"); + QCOMPARE(removedChange->removedNodeId(), img.id()); + QCOMPARE(removedChange->type(), Qt3DCore::PropertyValueRemoved); + + arbiter.events.clear(); } template <typename FrontendTextureType, Qt3DRender::QAbstractTexture::Target Target> @@ -269,6 +297,44 @@ void tst_RenderTexture::checkPropertyChanges() QCOMPARE(backend.properties().samples, 64); QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::TexturesDirty); renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty); + + // WHEN + Qt3DRender::QTextureImage img; + const auto imageAddChange = Qt3DCore::QPropertyNodeAddedChangePtr::create(Qt3DCore::QNodeId(), &img); + imageAddChange->setPropertyName("textureImage"); + backend.sceneChangeEvent(imageAddChange); + + // THEN + QCOMPARE(backend.textureImageIds().size(), 1); + QCOMPARE(backend.textureImageIds().first(), img.id()); + QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::TexturesDirty); + renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty); +} + +void tst_RenderTexture::checkTextureImageBookeeping() +{ + // GIVEN + TestArbiter arbiter; + DummyTexture texture; + arbiter.setArbiterOnNode(&texture); + + QCoreApplication::processEvents(); + + { + // WHEN + Qt3DRender::QTextureImage img; + texture.addTextureImage(&img); + + // THEN + QCOMPARE(img.parent(), &texture); + QCOMPARE(texture.textureImages().size(), 1); + QCOMPARE(texture.textureImages().first()->id(), img.id()); + } + // WHEN -> img is destroyed + + // THEN + QCOMPARE(texture.textureImages().size(), 0); + } QTEST_APPLESS_MAIN(tst_RenderTexture) |