nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1893188: Make ComputePipeline::Cleanup gracefully handle a missing bridge. r=webgpu-reviewers,ErichDonGubler

Browse source 
This aligns the implementation with RenderPipeline::Cleanup(). It also
adds a drive-by assert to GetBindGroupLayout() in both classes to add
confidence that the bridge is in the expected state.

Differential Revision: https://phabricator.services.mozilla.com/D208916
This commit is contained in:
Brad Werth 2024-04-29 16:07:57 +00:00
parent 14624240c9
commit c0e2f475a2
2 changed files with 6 additions and 0 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

5
dom/webgpu/ComputePipeline.cpp
View file

@ -33,6 +33,10 @@ void ComputePipeline::Cleanup() {
mValid = false;
auto bridge = mParent->GetBridge();
if (!bridge) {
return;
}
if (bridge->CanSend()) {
bridge->SendComputePipelineDrop(mId);
if (mImplicitPipelineLayoutId) {
@ -54,6 +58,7 @@ void ComputePipeline::Cleanup() {
already_AddRefed<BindGroupLayout> ComputePipeline::GetBindGroupLayout(
uint32_t aIndex) const {
auto bridge = mParent->GetBridge();
MOZ_ASSERT(bridge && bridge->CanSend());
auto* client = bridge->GetClient();
ipc::ByteBuf bb;

1
dom/webgpu/RenderPipeline.cpp
View file

@ -58,6 +58,7 @@ void RenderPipeline::Cleanup() {
already_AddRefed<BindGroupLayout> RenderPipeline::GetBindGroupLayout(
uint32_t aIndex) const {
auto bridge = mParent->GetBridge();
MOZ_ASSERT(bridge && bridge->CanSend());
auto* client = bridge->GetClient();
ipc::ByteBuf bb;