[WebGPU] It is legal to submit zero work to a render bundle but this …

…results in a metal error

https://bugs.webkit.org/show_bug.cgi?id=263591
<radar://117417815>

Reviewed by Tadeu Zagallo.

Fix a few other places where zero work was being submitted from the
CTS tests resulting in Metal validation errors.

* Source/WebGPU/WebGPU/RenderBundleEncoder.mm:
(WebGPU::RenderBundleEncoder::drawIndexedIndirect):
(WebGPU::RenderBundleEncoder::drawIndirect):
* Source/WebGPU/WebGPU/RenderPassEncoder.mm:
(WebGPU::RenderPassEncoder::drawIndexedIndirect):

Canonical link: https://commits.webkit.org/269724@main

Source/WebGPU/WebGPU/RenderBundleEncoder.mm

@@ -186,7 +186,7 @@ static void addResource(RenderBundle::ResourcesContainer* resources, id<MTLResou
     UNUSED_PARAM(indirectOffset);
 
     auto contents = (MTLDrawIndexedPrimitivesIndirectArguments*)indirectBuffer.buffer().contents;
-    if (!contents)
+    if (!contents || !contents->indexCount)
         return;
 
     executePreDrawCommands();
@@ -209,7 +209,7 @@ static void addResource(RenderBundle::ResourcesContainer* resources, id<MTLResou
     UNUSED_PARAM(indirectOffset);
 
     auto contents = (MTLDrawPrimitivesIndirectArguments*)indirectBuffer.buffer().contents;
-    if (!contents)
+    if (!contents || !contents->instanceCount || !m_indexBuffer.length)
         return;
 
     executePreDrawCommands();

Source/WebGPU/WebGPU/RenderPassEncoder.mm

@@ -148,6 +148,9 @@
 
 void RenderPassEncoder::drawIndexedIndirect(const Buffer& indirectBuffer, uint64_t indirectOffset)
 {
+    if (!m_indexBuffer.length || !indirectBuffer.buffer().length)
+        return;
+
     executePreDrawCommands();
     [m_renderCommandEncoder drawIndexedPrimitives:m_primitiveType indexType:m_indexType indexBuffer:m_indexBuffer indexBufferOffset:m_indexBufferOffset indirectBuffer:indirectBuffer.buffer() indirectBufferOffset:indirectOffset];
 }