diff options
author | Laszlo Agocs <laszlo.agocs@qt.io> | 2020-11-23 13:26:33 +0100 |
---|---|---|
committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2020-11-24 18:46:11 +0000 |
commit | 54b4e08a2a9e0d82cb2107e3df0613a0da13a48a (patch) | |
tree | c3b10457137011c4024fd7db7ca19682585441df | |
parent | 85c4facaccbe558b0462ea5d644225513d21432b (diff) |
Purge renderpass refs from renderer when redirecting via QRhiRt
Fixes: QTBUG-88761
Change-Id: Ia5df65a4a09a7554a7d0cca4533f766cb5abe97b
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>
Reviewed-by: Andy Nichols <andy.nichols@qt.io>
(cherry picked from commit bbb6298204b95f049c8a177365125247ba49dba3)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
-rw-r--r-- | src/quick/items/qquickrendertarget.cpp | 1 | ||||
-rw-r--r-- | src/quick/items/qquickwindow.cpp | 15 | ||||
-rw-r--r-- | tests/auto/quick/qquickrendercontrol/tst_qquickrendercontrol.cpp | 33 |
3 files changed, 41 insertions, 8 deletions
diff --git a/src/quick/items/qquickrendertarget.cpp b/src/quick/items/qquickrendertarget.cpp index 69063ecdbe..3ccfe85b88 100644 --- a/src/quick/items/qquickrendertarget.cpp +++ b/src/quick/items/qquickrendertarget.cpp @@ -404,6 +404,7 @@ bool QQuickRenderTargetPrivate::resolve(QRhi *rhi, QQuickWindowRenderTarget *dst case Type::RhiRenderTarget: dst->renderTarget = u.rhiRt; + dst->rpDesc = u.rhiRt->renderPassDescriptor(); // just for QQuickWindowRenderTarget::reset() dst->owns = false; return true; diff --git a/src/quick/items/qquickwindow.cpp b/src/quick/items/qquickwindow.cpp index 3a2bcb107a..470d43efd5 100644 --- a/src/quick/items/qquickwindow.cpp +++ b/src/quick/items/qquickwindow.cpp @@ -515,14 +515,15 @@ void forceUpdate(QQuickItem *item) void QQuickWindowRenderTarget::reset(QRhi *rhi, QSGRenderer *renderer) { - if (rhi && owns) { - if (renderer != nullptr && rpDesc != nullptr) + if (rhi) { + if (renderer) renderer->invalidatePipelineCacheDependency(rpDesc); - - delete renderTarget; - delete rpDesc; - delete texture; - delete depthStencil; + if (owns) { + delete renderTarget; + delete rpDesc; + delete texture; + delete depthStencil; + } } renderTarget = nullptr; diff --git a/tests/auto/quick/qquickrendercontrol/tst_qquickrendercontrol.cpp b/tests/auto/quick/qquickrendercontrol/tst_qquickrendercontrol.cpp index d81c40f5b6..4bf48f5a58 100644 --- a/tests/auto/quick/qquickrendercontrol/tst_qquickrendercontrol.cpp +++ b/tests/auto/quick/qquickrendercontrol/tst_qquickrendercontrol.cpp @@ -232,7 +232,38 @@ void tst_RenderControl::renderAndReadBackWithRhi() // redirect Qt Quick rendering into our texture quickWindow->setRenderTarget(QQuickRenderTarget::fromRhiRenderTarget(texRt.data())); + QSize currentSize = size; + for (int frame = 0; frame < 100; ++frame) { + // QTBUG-88761 - change the render target size at some point and verify the renderer continues to function + if (frame == 80) { + currentSize -= QSize(2, 2); // small enough change to not bother the pixel verification checks + tex->setPixelSize(currentSize); + QVERIFY(tex->create()); // internally we now have a whole new native texture object + ds->setPixelSize(currentSize); + QVERIFY(ds->create()); + QVERIFY(texRt->create()); + // setRenderTarget is mandatory upon changing something, even if the texRt pointer itself is the same + quickWindow->setRenderTarget(QQuickRenderTarget::fromRhiRenderTarget(texRt.data())); + } else if (frame == 90) { + // Go berserk, destroy and recreate the texture and related stuff + // (the QRhi objects themselves, not just the native stuff + // internally), it should still work. + currentSize -= QSize(2, 2); // chip off another 2 pixels + tex.reset(rhi->newTexture(QRhiTexture::RGBA8, currentSize, 1, + QRhiTexture::RenderTarget | QRhiTexture::UsedAsTransferSource)); + QVERIFY(tex->create()); + ds.reset(rhi->newRenderBuffer(QRhiRenderBuffer::DepthStencil, currentSize, 1)); + QVERIFY(ds->create()); + rtDesc = QRhiTextureRenderTargetDescription(QRhiColorAttachment(tex.data())); + rtDesc.setDepthStencilBuffer(ds.data()); + texRt.reset(rhi->newTextureRenderTarget(rtDesc)); + rp.reset(texRt->newCompatibleRenderPassDescriptor()); + texRt->setRenderPassDescriptor(rp.data()); + QVERIFY(texRt->create()); + quickWindow->setRenderTarget(QQuickRenderTarget::fromRhiRenderTarget(texRt.data())); + } + // have to process events, e.g. to get queued metacalls delivered QCoreApplication::processEvents(); @@ -276,7 +307,7 @@ void tst_RenderControl::renderAndReadBackWithRhi() QImage img = result; QVERIFY(!img.isNull()); - QCOMPARE(img.size(), rootItem->size()); + QCOMPARE(img.size(), currentSize); const int maxFuzz = 2; |