REGRESSION (r279992): Crashes under RemoteLayerBackingStore::applyBackingStoreToLayer() in macCatalyst
Created attachment 434002 [details] Patch
<rdar://problem/80923581>
Comment on attachment 434002 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=434002&action=review Patch looks good. > Source/WebKit/Shared/RemoteLayerTree/RemoteLayerBackingStore.mm:429 > + RetainPtr<id> contents = nil; OK as-is. No change needed. Thoughts about removing the = nil and letting the default constructor initialize this?
(In reply to Daniel Bates from comment #3) > Comment on attachment 434002 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=434002&action=review > > Patch looks good. > > > Source/WebKit/Shared/RemoteLayerTree/RemoteLayerBackingStore.mm:429 > > + RetainPtr<id> contents = nil; > > OK as-is. No change needed. Thoughts about removing the = nil and letting > the default constructor initialize this? Heh, sure, certainly.
Created attachment 434018 [details] Patch
Committed r280183 (239876@main): <https://commits.webkit.org/239876@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 434018 [details].