CoreMedia WrapperClass does not need the alignment fixup required by CMBaseClass
rdar://78864290
Created attachment 431348 [details] Patch
Comment on attachment 431348 [details] Patch r=me
Comment on attachment 431348 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=431348&action=review > Source/WebKit/Shared/mac/MediaFormatReader/CoreMediaWrapped.h:158 > + static constexpr struct { WrapperClass derivedClass; } derivedClass { Wrapped::wrapperClass() }; Don’t really need this to be a struct anymore.
Comment on attachment 431348 [details] Patch Didn’t mean to reset Geoff’s r+ though.
> > Source/WebKit/Shared/mac/MediaFormatReader/CoreMediaWrapped.h:158 > > + static constexpr struct { WrapperClass derivedClass; } derivedClass { Wrapped::wrapperClass() }; > > Don’t really need this to be a struct anymore. 👍🏻
(In reply to mitz from comment #4) > Comment on attachment 431348 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=431348&action=review > > > Source/WebKit/Shared/mac/MediaFormatReader/CoreMediaWrapped.h:158 > > + static constexpr struct { WrapperClass derivedClass; } derivedClass { Wrapped::wrapperClass() }; > > Don’t really need this to be a struct anymore. Good point, thanks!
Created attachment 431357 [details] Patch
Found 1 new test failure: imported/w3c/web-platform-tests/webrtc-extensions/transfer-datachannel-service-worker.https.html
Comment on attachment 431357 [details] Patch Failing test is unrelated
Committed r278870 (238814@main): <https://commits.webkit.org/238814@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 431357 [details].