Enable WebRTC for Mac Catalyst
Created attachment 448375 [details] Patch
Created attachment 448376 [details] Patch
Created attachment 448573 [details] Patch
Created attachment 448579 [details] Patch
<<rdar://86040748>>
Comment on attachment 448579 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=448579&action=review > Source/WebCore/platform/mediarecorder/cocoa/AudioSampleBufferCompressor.mm:58 > + , m_currentNativePresentationTimeStamp { PAL::kCMTimeInvalid } > + , m_currentOutputPresentationTimeStamp { PAL::kCMTimeInvalid } > + , m_remainingPrimeDuration { PAL::kCMTimeInvalid } Why make this change? > Source/WebCore/platform/mediastream/mac/RealtimeIncomingVideoSourceCocoa.mm:184 > +RetainPtr<CVPixelBufferRef> createBlackPixelBuffer(size_t width, size_t height) Maybe move this to RealtimeOutgoingVideoSourceCocoa.mm?
Don't we need rdar://77080901 first?
(In reply to Alex Christensen from comment #7) > Don't we need rdar://77080901 first? Yep, we need that one before landing this patch. (In reply to Eric Carlson from comment #6) > Comment on attachment 448579 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=448579&action=review > > > Source/WebCore/platform/mediarecorder/cocoa/AudioSampleBufferCompressor.mm:58 > > + , m_currentNativePresentationTimeStamp { PAL::kCMTimeInvalid } > > + , m_currentOutputPresentationTimeStamp { PAL::kCMTimeInvalid } > > + , m_remainingPrimeDuration { PAL::kCMTimeInvalid } > > Why make this change? So that we do not add soft link headers in the header but keep them in source files. > > Source/WebCore/platform/mediastream/mac/RealtimeIncomingVideoSourceCocoa.mm:184 > > +RetainPtr<CVPixelBufferRef> createBlackPixelBuffer(size_t width, size_t height) > > Maybe move this to RealtimeOutgoingVideoSourceCocoa.mm? We should probably move it to a utility file indeed.