CopyFromStagingToBuffer: ASSERT size is not 0

Zero-sized copies are invalid in a couple backends, and in follow up
CLs CreateBufferMapped will be change to handle zero-sized buffers
correctly.

Bug: chromium:1069076
Change-Id: Ieef62a13182bbe1e939a3847980c91339e42aa8f
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/22460
Reviewed-by: Stephen White <senorblanco@chromium.org>
Reviewed-by: Austin Eng <enga@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>

src/dawn_native/Buffer.cpp

@@ -319,6 +319,10 @@ namespace dawn_native {
 
     MaybeError BufferBase::CopyFromStagingBuffer() {
         ASSERT(mStagingBuffer);
+        if (GetSize() == 0) {
+            return {};
+        }
+
         DAWN_TRY(GetDevice()->CopyFromStagingToBuffer(mStagingBuffer.get(), 0, this, 0, GetSize()));
 
         DynamicUploader* uploader = GetDevice()->GetDynamicUploader();

src/dawn_native/Queue.cpp

@@ -113,6 +113,10 @@ namespace dawn_native {
                                           uint64_t bufferOffset,
                                           const void* data,
                                           size_t size) {
+        if (size == 0) {
+            return {};
+        }
+
         DeviceBase* device = GetDevice();
 
         UploadHandle uploadHandle;
@@ -122,10 +126,8 @@ namespace dawn_native {
 
         memcpy(uploadHandle.mappedBuffer, data, size);
 
-        DAWN_TRY(device->CopyFromStagingToBuffer(
+        return device->CopyFromStagingToBuffer(uploadHandle.stagingBuffer, uploadHandle.startOffset,
-            uploadHandle.stagingBuffer, uploadHandle.startOffset, buffer, bufferOffset, size));
+                                               buffer, bufferOffset, size);
-
-        return {};
     }
 
     MaybeError QueueBase::ValidateSubmit(uint32_t commandCount,

src/dawn_native/metal/DeviceMTL.mm

@@ -247,10 +247,9 @@ namespace dawn_native { namespace metal {
                                                BufferBase* destination,
                                                uint64_t destinationOffset,
                                                uint64_t size) {
-        // Metal validation layers forbid  0-sized copies, skip it since it is a noop.
+        // Metal validation layers forbid  0-sized copies, assert it is skipped prior to calling
-        if (size == 0) {
+        // this function.
-            return {};
+        ASSERT(size != 0);
-        }
 
         id<MTLBuffer> uploadBuffer = ToBackend(source)->GetBufferHandle();
         id<MTLBuffer> buffer = ToBackend(destination)->GetMTLBuffer();

src/dawn_native/vulkan/DeviceVk.cpp

@@ -541,12 +541,9 @@ namespace dawn_native { namespace vulkan {
                                                BufferBase* destination,
                                                uint64_t destinationOffset,
                                                uint64_t size) {
-        // It is a validation error to do a 0-sized copy in Vulkan skip it since it is a noop.
+        // It is a validation error to do a 0-sized copy in Vulkan, check it is skipped prior to
-        if (size == 0) {
+        // calling this function.
-            return {};
+        ASSERT(size != 0);
-        }
-
-        CommandRecordingContext* recordingContext = GetPendingRecordingContext();
 
         // Insert memory barrier to ensure host write operations are made visible before
         // copying from the staging buffer. However, this barrier can be removed (see note below).
@@ -557,6 +554,7 @@ namespace dawn_native { namespace vulkan {
 
         // Insert pipeline barrier to ensure correct ordering with previous memory operations on the
         // buffer.
+        CommandRecordingContext* recordingContext = GetPendingRecordingContext();
         ToBackend(destination)->TransitionUsageNow(recordingContext, wgpu::BufferUsage::CopyDst);
 
         VkBufferCopy copy;