RESOLVED FIXED 236910
[WebGPU] Tracer bullet part 10: Implement setLabel() and Metal accessors 
https://bugs.webkit.org/show_bug.cgi?id=236910
Summary [WebGPU] Tracer bullet part 10: Implement setLabel() and Metal accessors
Myles C. Maxfield
Reported 2022-02-19 21:39:28 PST
By adding a few various trivial functions in classes that need them, this patch completely finishes the implementation of CommandBuffer, QuerySet, RenderBundle, Sampler, and TextureView classes (other than their creation functions).
Attachments
Patch (16.73 KB, patch)
2022-02-19 21:41 PST, Myles C. Maxfield 		dino: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2022-02-19 21:41:10 PST
Created attachment 452678 [details] Patch
Dean Jackson
Comment 2 2022-02-21 09:35:48 PST
Comment on attachment 452678 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452678&action=review > Source/WebGPU/WebGPU/RenderPassEncoder.mm:178 > + m_renderCommandEncoder.label = [NSString stringWithCString:label encoding:NSUTF8StringEncoding]; Do you ever want to ASSERT that m_renderCommandEncoder etc exist?
Myles C. Maxfield
Comment 3 2022-02-21 13:31:08 PST
Comment on attachment 452678 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452678&action=review >> Source/WebGPU/WebGPU/RenderPassEncoder.mm:178 >> + m_renderCommandEncoder.label = [NSString stringWithCString:label encoding:NSUTF8StringEncoding]; > > Do you ever want to ASSERT that m_renderCommandEncoder etc exist? Yep! It's done in the creation routines. And rather than ASSERT()ing, it instead just doesn't create the WebGPU wrapper object.
Myles C. Maxfield
Comment 4 2022-02-21 13:32:51 PST
Committed r290266 (247591@trunk): <https://commits.webkit.org/247591@trunk>
Radar WebKit Bug Importer
Comment 5 2022-02-21 13:33:31 PST
<rdar://problem/89253939>
Note You need to log in before you can comment on or make changes to this bug.