[webkit-changes] [WebKit/WebKit] ee90c1: Finish off smart pointer coverage in WebGPU

geoffreygaren noreply at github.com
Sun Oct 27 13:18:58 PDT 2024 

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ee90c1745f1a38eb48656cbf01916274168c7984
      https://github.com/WebKit/WebKit/commit/ee90c1745f1a38eb48656cbf01916274168c7984
  Author: Geoffrey Garen <ggaren at apple.com>
  Date:   2024-10-27 (Sun, 27 Oct 2024)

  Changed paths:
    M Source/WebCore/Modules/WebGPU/GPUPipelineLayoutDescriptor.h
    M Source/WebCore/Modules/WebGPU/GPUQueue.cpp
    M Source/WebCore/Modules/WebGPU/GPURenderPassEncoder.cpp
    M Source/WebCore/Modules/WebGPU/Implementation/WebGPUQueueImpl.cpp
    M Source/WebCore/Modules/WebGPU/Implementation/WebGPUQueueImpl.h
    M Source/WebCore/Modules/WebGPU/Implementation/WebGPURenderPassEncoderImpl.cpp
    M Source/WebCore/Modules/WebGPU/Implementation/WebGPURenderPassEncoderImpl.h
    M Source/WebCore/Modules/WebGPU/InternalAPI/WebGPUPipelineLayoutDescriptor.h
    M Source/WebCore/Modules/WebGPU/InternalAPI/WebGPUQueue.h
    M Source/WebCore/Modules/WebGPU/InternalAPI/WebGPURenderPassEncoder.h
    M Source/WebGPU/WebGPU/Adapter.h
    M Source/WebGPU/WebGPU/BindGroup.h
    M Source/WebGPU/WebGPU/BindGroup.mm
    M Source/WebGPU/WebGPU/Buffer.h
    M Source/WebGPU/WebGPU/Buffer.mm
    M Source/WebGPU/WebGPU/CommandEncoder.mm
    M Source/WebGPU/WebGPU/CommandsMixin.mm
    M Source/WebGPU/WebGPU/ComputePassEncoder.mm
    M Source/WebGPU/WebGPU/ComputePipeline.mm
    M Source/WebGPU/WebGPU/Device.h
    M Source/WebGPU/WebGPU/Device.mm
    M Source/WebGPU/WebGPU/ExternalTexture.mm
    M Source/WebGPU/WebGPU/Pipeline.mm
    M Source/WebGPU/WebGPU/PipelineLayout.mm
    M Source/WebGPU/WebGPU/PresentationContextIOSurface.mm
    M Source/WebGPU/WebGPU/QuerySet.mm
    M Source/WebGPU/WebGPU/Queue.h
    M Source/WebGPU/WebGPU/Queue.mm
    M Source/WebGPU/WebGPU/RenderBundle.mm
    M Source/WebGPU/WebGPU/RenderBundleEncoder.mm
    M Source/WebGPU/WebGPU/RenderPassEncoder.h
    M Source/WebGPU/WebGPU/RenderPassEncoder.mm
    M Source/WebGPU/WebGPU/RenderPipeline.h
    M Source/WebGPU/WebGPU/RenderPipeline.mm
    M Source/WebGPU/WebGPU/ShaderModule.mm
    M Source/WebGPU/WebGPU/Texture.h
    M Source/WebGPU/WebGPU/Texture.mm
    M Source/WebGPU/WebGPU/TextureView.mm
    M Source/WebGPU/WebGPU/XRBinding.h
    M Source/WebGPU/WebGPU/XRBinding.mm
    M Source/WebGPU/WebGPU/XRProjectionLayer.mm
    M Source/WebGPU/WebGPU/XRSubImage.mm
    M Source/WebKit/GPUProcess/graphics/WebGPU/RemoteQueue.cpp
    M Source/WebKit/GPUProcess/graphics/WebGPU/RemoteRenderPassEncoder.cpp
    M Source/WebKit/SaferCPPExpectations/UncountedCallArgsCheckerExpectations
    M Source/WebKit/Shared/WebGPU/WebGPUPipelineLayoutDescriptor.cpp
    M Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteQueueProxy.cpp
    M Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteQueueProxy.h
    M Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteRenderPassEncoderProxy.cpp
    M Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteRenderPassEncoderProxy.h

  Log Message:
  -----------
  Finish off smart pointer coverage in WebGPU
https://bugs.webkit.org/show_bug.cgi?id=282145
rdar://138712492

Reviewed by Mike Wyrzykowski.

Fixed the remaining static analyzer complaints.

I skipped false positives to do with the implementations of RefCounted,
ThreadSafeRefCounted, String, and Thread.

* Source/WebCore/Modules/WebGPU/Implementation/WebGPUQueueImpl.cpp:
(WebCore::WebGPU::QueueImpl::submit):
* Source/WebCore/Modules/WebGPU/Implementation/WebGPUQueueImpl.h:
* Source/WebCore/Modules/WebGPU/Implementation/WebGPURenderPassEncoderImpl.cpp:
(WebCore::WebGPU::RenderPassEncoderImpl::executeBundles):
* Source/WebCore/Modules/WebGPU/Implementation/WebGPURenderPassEncoderImpl.h:
* Source/WebCore/Modules/WebGPU/InternalAPI/WebGPUQueue.h:
* Source/WebCore/Modules/WebGPU/InternalAPI/WebGPURenderPassEncoder.h:
* Source/WebGPU/WebGPU/Adapter.h:
* Source/WebGPU/WebGPU/BindGroup.h:
(WebGPU::BindGroup::bindGroupLayout const):
* Source/WebGPU/WebGPU/BindGroup.mm:
(WebGPU::BindGroup::rebindSamplersIfNeeded const):
(WebGPU::BindGroup::bindGroupLayout const): Deleted.
* Source/WebGPU/WebGPU/Buffer.h:
(WebGPU::Buffer::indirectIndexedBuffer const):
(WebGPU::Buffer::isDestroyed const):
* Source/WebGPU/WebGPU/Buffer.mm:
(WebGPU::Buffer::incrementBufferMapCount):
(WebGPU::Buffer::decrementBufferMapCount):
(WebGPU::Buffer::destroy):
(WebGPU::Buffer::isDestroyed const): Deleted.
(WebGPU::Buffer::indirectIndexedBuffer const): Deleted.
* Source/WebGPU/WebGPU/CommandEncoder.mm:
(WebGPU::CommandEncoder::discardCommandBuffer):
(WebGPU::CommandEncoder::copyBufferToTexture):
* Source/WebGPU/WebGPU/CommandsMixin.mm:
* Source/WebGPU/WebGPU/ComputePassEncoder.mm:
(WebGPU::ComputePassEncoder::executePreDispatchCommands):
(WebGPU::ComputePassEncoder::setBindGroup):
* Source/WebGPU/WebGPU/ComputePipeline.mm:
(WebGPU::ComputePipeline::getBindGroupLayout):
* Source/WebGPU/WebGPU/Device.h:
(WebGPU::Device::instance const):
(WebGPU::Device::maxBuffersPlusVertexBuffersForVertexStage const):
(WebGPU::Device::maxBuffersForFragmentStage const):
(WebGPU::Device::vertexBufferIndexForBindGroup const):
(WebGPU::Device::placeholderBuffer const):
* Source/WebGPU/WebGPU/Device.mm:
(WebGPU::Device::getXRViewSubImage):
(WebGPU::Device::makeInvalid):
(WebGPU::Device::loseTheDevice):
(WebGPU::Device::placeholderBuffer const): Deleted.
(WebGPU::Device::maxBuffersPlusVertexBuffersForVertexStage const): Deleted.
(WebGPU::Device::maxBuffersForFragmentStage const): Deleted.
(WebGPU::Device::vertexBufferIndexForBindGroup const): Deleted.
* Source/WebGPU/WebGPU/ExternalTexture.mm:
(WebGPU::ExternalTexture::destroy):
* Source/WebGPU/WebGPU/Pipeline.mm:
(WebGPU::createLibrary):
(WebGPU::errorValidatingBindGroup):
* Source/WebGPU/WebGPU/PipelineLayout.mm:
(WebGPU::PipelineLayout::errorValidatingBindGroupCompatibility const):
* Source/WebGPU/WebGPU/PresentationContextIOSurface.mm:
(WebGPU::PresentationContextIOSurface::onSubmittedWorkScheduled):
(WebGPU::PresentationContextIOSurface::getTextureAsNativeImage):
* Source/WebGPU/WebGPU/QuerySet.mm:
(WebGPU::QuerySet::destroy):
* Source/WebGPU/WebGPU/Queue.h:
* Source/WebGPU/WebGPU/Queue.mm:
(WebGPU::Queue::errorValidatingSubmit const):
(WebGPU::invalidateCommandBuffers):
(WebGPU::Queue::submit):
(WebGPU::Queue::clearTextureViewIfNeeded):
(wgpuQueueSubmit):
* Source/WebGPU/WebGPU/RenderBundle.mm:
(WebGPU::RenderBundle::replayCommands const):
(WebGPU::RenderBundle::updateMinMaxDepths):
(WebGPU::RenderBundle::validateRenderPass const):
* Source/WebGPU/WebGPU/RenderBundleEncoder.mm:
(WebGPU::RenderBundleEncoder::returnIfEncodingIsFinished):
(WebGPU::RenderBundleEncoder::executePreDrawCommands):
(WebGPU::RenderBundleEncoder::endCurrentICB):
(WebGPU::RenderBundleEncoder::finish):
(WebGPU::RenderBundleEncoder::setBindGroup):
(WebGPU::RenderBundleEncoder::setPipeline):
* Source/WebGPU/WebGPU/RenderPassEncoder.h:
(WebGPU::RenderPassEncoder::parentEncoder):
* Source/WebGPU/WebGPU/RenderPassEncoder.mm:
(WebGPU::RenderPassEncoder::RenderPassEncoder):
(WebGPU::m_maxDrawCount):
(WebGPU::RenderPassEncoder::~RenderPassEncoder):
(WebGPU::RenderPassEncoder::executePreDrawCommands):
(WebGPU::RenderPassEncoder::splitRenderPass):
(WebGPU::RenderPassEncoder::endPass):
(WebGPU::RenderPassEncoder::executeBundles):
(WebGPU::RenderPassEncoder::makeInvalid):
(WebGPU::RenderPassEncoder::setBindGroup):
(WebGPU::RenderPassEncoder::errorValidatingPipeline const):
(WebGPU::RenderPassEncoder::setPipeline):
(wgpuRenderPassEncoderExecuteBundles):
(WebGPU::RenderPassEncoder::parentEncoder): Deleted.
* Source/WebGPU/WebGPU/RenderPipeline.h:
(WebGPU::RenderPipeline::pipelineLayout const):
(WebGPU::RenderPipeline::protectedPipelineLayout const):
(WebGPU::RenderPipeline::primitiveTopology const):
(WebGPU::RenderPipeline::stripIndexFormat const):
* Source/WebGPU/WebGPU/RenderPipeline.mm:
(WebGPU::RenderPipeline::getBindGroupLayout):
(WebGPU::RenderPipeline::colorDepthStencilTargetsMatch const):
(WebGPU::RenderPipeline::pipelineLayout const): Deleted.
(WebGPU::RenderPipeline::primitiveTopology const): Deleted.
(WebGPU::RenderPipeline::stripIndexFormat const): Deleted.
* Source/WebGPU/WebGPU/ShaderModule.mm:
(WebGPU::ShaderModule::convertPipelineLayout):
* Source/WebGPU/WebGPU/Texture.h:
(WebGPU::Texture::isDestroyed const):
(WebGPU::Texture::isCanvasBacking const):
* Source/WebGPU/WebGPU/Texture.mm:
(WebGPU::Texture::createView):
(WebGPU::Texture::waitForCommandBufferCompletion):
(WebGPU::Texture::destroy):
(WebGPU::Texture::isCanvasBacking const): Deleted.
(WebGPU::Texture::isDestroyed const): Deleted.
* Source/WebGPU/WebGPU/TextureView.mm:
(WebGPU::TextureView::previouslyCleared const):
(WebGPU::TextureView::setPreviouslyCleared):
(WebGPU::TextureView::width const):
(WebGPU::TextureView::height const):
(WebGPU::TextureView::depthOrArrayLayers const):
(WebGPU::TextureView::destroy):
* Source/WebGPU/WebGPU/XRBinding.h:
(WebGPU::XRBinding::device):
(WebGPU::XRBinding::protectedDevice):
* Source/WebGPU/WebGPU/XRBinding.mm:
(WebGPU::XRBinding::device): Deleted.
* Source/WebGPU/WebGPU/XRProjectionLayer.mm:
(WebGPU::XRBinding::createXRProjectionLayer):
* Source/WebGPU/WebGPU/XRSubImage.mm:
(WebGPU::XRSubImage::update):
(WebGPU::XRBinding::getViewSubImage):

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



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications

More information about the webkit-changes mailing list