Make backend FG node use FrameGraphDirty instead of AllDirty

So as to later on allow finer grained job launching and not force
every job to be rerun when we know only the FG has changed.

Change-Id: I09f935f8a1793722159ba645acc2ad4b00983041
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>

3 files changed, 24 insertions, 0 deletions

diff --git a/tests/auto/render/blitframebuffer/tst_blitframebuffer.cpp b/tests/auto/render/blitframebuffer/tst_blitframebuffer.cpp
index 26c43b5c2..9232aa70f 100644
--- a/tests/auto/render/blitframebuffer/tst_blitframebuffer.cpp
+++ b/tests/auto/render/blitframebuffer/tst_blitframebuffer.cpp
@@ -118,6 +118,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendBlitFramebuffer.isEnabled(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
         {
              // WHEN
@@ -130,6 +132,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendBlitFramebuffer.sourceRenderTargetId(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
         {
              // WHEN
@@ -142,6 +146,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendBlitFramebuffer.destinationRenderTargetId(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
         {
              // WHEN
@@ -153,6 +159,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendBlitFramebuffer.sourceRect(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
         {
              // WHEN
@@ -164,6 +172,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendBlitFramebuffer.destinationRect(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
         {
              // WHEN
@@ -175,6 +185,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendBlitFramebuffer.sourceAttachmentPoint(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
         {
              // WHEN
@@ -186,6 +198,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendBlitFramebuffer.destinationAttachmentPoint(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
     }
 
diff --git a/tests/auto/render/memorybarrier/tst_memorybarrier.cpp b/tests/auto/render/memorybarrier/tst_memorybarrier.cpp
index f0d4931d9..33a1247fa 100644
--- a/tests/auto/render/memorybarrier/tst_memorybarrier.cpp
+++ b/tests/auto/render/memorybarrier/tst_memorybarrier.cpp
@@ -98,6 +98,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendMemoryBarrier.isEnabled(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
         {
              // WHEN
@@ -109,6 +111,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendMemoryBarrier.waitOperations(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
     }
 
diff --git a/tests/auto/render/proximityfilter/tst_proximityfilter.cpp b/tests/auto/render/proximityfilter/tst_proximityfilter.cpp
index 774e2dd1f..ac9cc610c 100644
--- a/tests/auto/render/proximityfilter/tst_proximityfilter.cpp
+++ b/tests/auto/render/proximityfilter/tst_proximityfilter.cpp
@@ -101,6 +101,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendProximityFilter.isEnabled(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
         {
              // WHEN
@@ -112,6 +114,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendProximityFilter.distanceThreshold(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
         {
              // WHEN
@@ -123,6 +127,8 @@ private Q_SLOTS:
 
              // THEN
             QCOMPARE(backendProximityFilter.entityId(), newValue);
+            QVERIFY(renderer.dirtyBits() & Qt3DRender::Render::AbstractRenderer::FrameGraphDirty);
+            renderer.clearDirtyBits(Qt3DRender::Render::AbstractRenderer::AllDirty);
         }
     }