Let WTF::convertSafely deduce types from arguments.
authorjustin_fan@apple.com <justin_fan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 15 Apr 2019 21:58:14 +0000 (21:58 +0000)
committerjustin_fan@apple.com <justin_fan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 15 Apr 2019 21:58:14 +0000 (21:58 +0000)
Reviewer's (Darin Adler) follow-up to https://bugs.webkit.org/show_bug.cgi?id=196793.

* platform/graphics/gpu/cocoa/GPUBufferMetal.mm:
(WebCore::GPUBuffer::tryCreate):
* platform/graphics/gpu/cocoa/GPUCommandBufferMetal.mm:
(WebCore::GPUCommandBuffer::copyBufferToTexture):
(WebCore::GPUCommandBuffer::copyTextureToBuffer):
* platform/graphics/gpu/cocoa/GPURenderPipelineMetal.mm:
(WebCore::trySetInputStateForPipelineDescriptor):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@244297 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/gpu/cocoa/GPUBufferMetal.mm
Source/WebCore/platform/graphics/gpu/cocoa/GPUCommandBufferMetal.mm
Source/WebCore/platform/graphics/gpu/cocoa/GPURenderPipelineMetal.mm

index 9ce2213..e03e30d 100644 (file)
@@ -1,3 +1,17 @@
+2019-04-15  Justin Fan  <justin_fan@apple.com>
+
+        Let WTF::convertSafely deduce types from arguments.
+
+        Reviewer's (Darin Adler) follow-up to https://bugs.webkit.org/show_bug.cgi?id=196793.
+
+        * platform/graphics/gpu/cocoa/GPUBufferMetal.mm:
+        (WebCore::GPUBuffer::tryCreate):
+        * platform/graphics/gpu/cocoa/GPUCommandBufferMetal.mm:
+        (WebCore::GPUCommandBuffer::copyBufferToTexture):
+        (WebCore::GPUCommandBuffer::copyTextureToBuffer):
+        * platform/graphics/gpu/cocoa/GPURenderPipelineMetal.mm:
+        (WebCore::trySetInputStateForPipelineDescriptor):
+
 2019-04-12  Ryosuke Niwa  <rniwa@webkit.org>
 
         Throw TypeError when custom element constructor returns a wrong element or tries to create itself
index 2a594b9..8a3ed32 100644 (file)
@@ -66,7 +66,7 @@ RefPtr<GPUBuffer> GPUBuffer::tryCreate(Ref<GPUDevice>&& device, const GPUBufferD
 {
     // MTLBuffer size (NSUInteger) is 32 bits on some platforms.
     NSUInteger size = 0;
-    if (!WTF::convertSafely<NSUInteger, uint64_t>(descriptor.size, size)) {
+    if (!WTF::convertSafely(descriptor.size, size)) {
         LOG(WebGPU, "GPUBuffer::tryCreate(): Buffer size is too large!");
         return nullptr;
     }
index 37cd45d..43a2650 100644 (file)
@@ -141,7 +141,7 @@ void GPUCommandBuffer::copyBufferToTexture(GPUBufferCopyView&& srcBuffer, GPUTex
 
     // MTLBuffer size (NSUInteger) is 32 bits on some platforms.
     NSUInteger sourceOffset = 0;
-    if (!WTF::convertSafely<NSUInteger, uint64_t>(srcBuffer.offset, sourceOffset)) {
+    if (!WTF::convertSafely(srcBuffer.offset, sourceOffset)) {
         LOG(WebGPU, "GPUCommandBuffer::copyBufferToTexture(): Source offset is too large!");
         return;
     }
@@ -185,7 +185,7 @@ void GPUCommandBuffer::copyTextureToBuffer(GPUTextureCopyView&& srcTexture, GPUB
     }
     // MTLBuffer size (NSUInteger) is 32 bits on some platforms.
     NSUInteger destinationOffset = 0;
-    if (!WTF::convertSafely<NSUInteger, uint64_t>(dstBuffer.offset, destinationOffset)) {
+    if (!WTF::convertSafely(dstBuffer.offset, destinationOffset)) {
         LOG(WebGPU, "GPUCommandBuffer::copyTextureToBuffer(): Destination offset is too large!");
         return;
     }
index f3ea862..7a725c7 100644 (file)
@@ -343,7 +343,7 @@ static bool trySetInputStateForPipelineDescriptor(const char* const functionName
         // MTLBuffer size (NSUInteger) is 32 bits on some platforms.
         // FIXME: Ensure offset < buffer's stride + format's data size.
         NSUInteger attributeOffset = 0;
-        if (!WTF::convertSafely<NSUInteger, uint64_t>(attributes[i].offset, attributeOffset)) {
+        if (!WTF::convertSafely(attributes[i].offset, attributeOffset)) {
             LOG(WebGPU, "%s: Buffer offset for vertex attribute %u is too large!", functionName, location);
             return false;
         }
@@ -365,7 +365,7 @@ static bool trySetInputStateForPipelineDescriptor(const char* const functionName
             return false;
         }
         NSUInteger inputStride = 0;
-        if (!WTF::convertSafely<NSUInteger, uint64_t>(inputs[j].stride, inputStride)) {
+        if (!WTF::convertSafely(inputs[j].stride, inputStride)) {
             LOG(WebGPU, "%s: Stride for vertex buffer slot %d is too large!", functionName, slot);
             return false;
         }