[WebGPU] Code quality concerns raised for 191291: [WebGPU] Experimental prototype...
authorjustin_fan@apple.com <justin_fan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Nov 2018 00:33:53 +0000 (00:33 +0000)
committerjustin_fan@apple.com <justin_fan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Nov 2018 00:33:53 +0000 (00:33 +0000)
commit18f05d855efbf587c6dbe233d021b9096c9a89c9
treed5d28c717685d81e7088e9752a0fa7ab525adfca
parenta02c090f0162dbc7d383591cb06a1bfbea4d54ee
[WebGPU] Code quality concerns raised for 191291: [WebGPU] Experimental prototype for WebGPURenderPipeline and WebGPUSwapChain
https://bugs.webkit.org/show_bug.cgi?id=191383

Reviewed by Dean Jackson.

Covered by existing WebGPU tests introduced in original patch.

* Modules/webgpu/GPUDevice.h:
* Modules/webgpu/GPUPipelineStageDescriptor.h:
* Modules/webgpu/GPURenderPipelineDescriptor.h: Now a base struct with a guaranteed vertex stage member.
(): Refactored into enum class.
(WebCore::GPURenderPipelineDescriptor::GPURenderPipelineDescriptor): Removed in favor of init-list construction.
(WebCore::GPURenderPipelineDescriptor::primitiveTopology): Now a proper enum class member.
* Modules/webgpu/GPUShaderModule.h:
* Modules/webgpu/WebGPUDevice.cpp:
(WebCore::WebGPUDevice::createRenderPipeline const):
* Modules/webgpu/WebGPUShaderModule.h:
(WebCore::WebGPUShaderModule::module const):
* Modules/webgpu/WebGPUShaderStage.h: Replaced enum with constants to better reflect IDL.
* Modules/webgpu/cocoa/GPURenderPipeline.h:
* Modules/webgpu/cocoa/GPURenderPipelineMetal.mm:
(WebCore::setFunctionsForPipelineDescriptor):
(WebCore::GPURenderPipeline::create):
* Modules/webgpu/cocoa/GPUSwapChain.h:
* WebCore.xcodeproj/project.pbxproj: Removed GPUPipelineDescriptorBase.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@238208 268f45cc-cd09-0410-ab3c-d52691b4dbfc
12 files changed:
Source/WebCore/ChangeLog
Source/WebCore/Modules/webgpu/GPUDevice.h
Source/WebCore/Modules/webgpu/GPUPipelineStageDescriptor.h
Source/WebCore/Modules/webgpu/GPURenderPipelineDescriptor.h
Source/WebCore/Modules/webgpu/GPUShaderModule.h
Source/WebCore/Modules/webgpu/WebGPUDevice.cpp
Source/WebCore/Modules/webgpu/WebGPUShaderModule.h
Source/WebCore/Modules/webgpu/WebGPUShaderStage.h
Source/WebCore/Modules/webgpu/cocoa/GPURenderPipeline.h
Source/WebCore/Modules/webgpu/cocoa/GPURenderPipelineMetal.mm
Source/WebCore/Modules/webgpu/cocoa/GPUSwapChain.h
Source/WebCore/WebCore.xcodeproj/project.pbxproj