RESOLVED FIXED 217169
Add AVAssetWriter SPI header
https://bugs.webkit.org/show_bug.cgi?id=217169
Summary Add AVAssetWriter SPI header
youenn fablet
Reported 2020-10-01 02:17:37 PDT
Add AVAssetWriter SPI header
Attachments
Patch (12.41 KB, patch)
2020-10-01 02:27 PDT, youenn fablet
no flags
Patch (12.46 KB, patch)
2020-10-01 03:22 PDT, youenn fablet
no flags
Patch (13.43 KB, patch)
2020-10-01 05:57 PDT, youenn fablet
no flags
Patch (14.40 KB, patch)
2020-10-02 01:03 PDT, youenn fablet
no flags
youenn fablet
Comment 1 2020-10-01 02:18:03 PDT
youenn fablet
Comment 2 2020-10-01 02:27:58 PDT
youenn fablet
Comment 3 2020-10-01 03:22:08 PDT
youenn fablet
Comment 4 2020-10-01 05:57:16 PDT
EWS
Comment 5 2020-10-01 10:28:57 PDT
Committed r267841: <https://trac.webkit.org/changeset/267841> All reviewed patches have been landed. Closing bug and clearing flags on attachment 410224 [details].
Truitt Savell
Comment 6 2020-10-01 13:04:21 PDT
It looks like this patch broke OpenSource Catalina Builds https://build.webkit.org/builders/Apple-Catalina-Release-Build/builds/10761 ./platform/mediarecorder/cocoa/MediaRecorderPrivateWriterCocoa.mm:50:49: error: unknown class name 'AVAssetWriterDelegate'; did you mean 'WebAVAssetWriterDelegate'? ./platform/mediarecorder/cocoa/MediaRecorderPrivateWriterCocoa.mm:50:49: error: type argument 'WebAVAssetWriterDelegate' must be a pointer (requires a '*') ./platform/mediarecorder/cocoa/MediaRecorderPrivateWriterCocoa.mm:50:39: error: type arguments cannot be applied to non-parameterized class 'NSObject' ./platform/mediarecorder/cocoa/MediaRecorderPrivateWriterCocoa.mm:166:21: error: 'AVAssetWriter' may not respond to 'setDelegate:' [-Werror] ./platform/mediarecorder/cocoa/MediaRecorderPrivateWriterCocoa.mm:166:33: error: incompatible pointer types sending 'WTF::RetainPtr<WebAVAssetWriterDelegate>::PtrType' (aka 'WebAVAssetWriterDelegate *') to parameter of type 'id<NSFileManagerDelegate> _Nullable' [-Werror,-Wincompatible-pointer-types]
Truitt Savell
Comment 7 2020-10-01 13:05:13 PDT
Reverted r267841 for reason: Broke Catalina Builds Committed r267849: <https://trac.webkit.org/changeset/267849>
youenn fablet
Comment 8 2020-10-02 01:03:43 PDT
EWS
Comment 9 2020-10-02 02:38:57 PDT
Committed r267870: <https://trac.webkit.org/changeset/267870> All reviewed patches have been landed. Closing bug and clearing flags on attachment 410305 [details].
Note You need to log in before you can comment on or make changes to this bug.