Summary: | MediaRecorder stopRecorder() returns empty Blob after first use | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | majo | ||||||||||||||
Component: | Media | Assignee: | youenn fablet <youennf> | ||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||
Severity: | Normal | CC: | cdumez, eric.carlson, ews-watchlist, glenn, hta, jer.noble, Lawrence.j, majo, philipj, sergio, tommyw, webkit-bug-importer, youennf, zach | ||||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||||
Version: | Safari Technology Preview | ||||||||||||||||
Hardware: | Mac | ||||||||||||||||
OS: | macOS 10.15 | ||||||||||||||||
Attachments: |
|
Description
majo
2020-05-22 13:05:04 PDT
Created attachment 402637 [details]
WIP
Created attachment 402653 [details]
Patch
Comment on attachment 402653 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=402653&action=review > Source/WebCore/platform/mediarecorder/cocoa/AudioSampleBufferCompressor.mm:192 > + fprintf(stderr, "AudioSampleBufferCompressor::attachPrimingTrimsIfNeeded1 %p\n", this); Oops! > Source/WebCore/platform/mediarecorder/cocoa/AudioSampleBufferCompressor.mm:194 > + fprintf(stderr, "AudioSampleBufferCompressor::attachPrimingTrimsIfNeeded2 %p\n", this); Ditto. > Source/WebCore/platform/mediarecorder/cocoa/AudioSampleBufferCompressor.mm:206 > + fprintf(stderr, "AudioSampleBufferCompressor::attachPrimingTrimsIfNeeded3 %p\n", this); Ditto. > Source/WebCore/platform/mediarecorder/cocoa/AudioSampleBufferCompressor.mm:208 > + fprintf(stderr, "AudioSampleBufferCompressor::attachPrimingTrimsIfNeeded4 %p\n", this); Ditto. Created attachment 402728 [details]
Patch
Committed r263511: <https://trac.webkit.org/changeset/263511> All reviewed patches have been landed. Closing bug and clearing flags on attachment 402728 [details]. Follow-up build fix: <https://trac.webkit.org/changeset/263514> Reverted r263511, r263514, and r263565 for reason: r263511 caused MediaRecorder test crashes on internal testers. Committed r263588: <https://trac.webkit.org/changeset/263588> Fix is in MediaRecorderPrivateMock::fetchData to unhold m_bufferLock before calling the completion handler since calling the completion handler might delete 'this' Created attachment 402911 [details]
Patch
ChangeLog entry in LayoutTests/ChangeLog contains OOPS!. Created attachment 402988 [details]
Patch for relanding
Committed r263633: <https://trac.webkit.org/changeset/263633> All reviewed patches have been landed. Closing bug and clearing flags on attachment 402988 [details]. Reverted r263633, r263651, and r263671 for reason: Still seeing MediaRecorder test crashes after re-landing r263633 Committed r263854: <https://trac.webkit.org/changeset/263854> (In reply to Ryan Haddad from comment #14) > Reverted r263633, r263651, and r263671 for reason: > > Still seeing MediaRecorder test crashes after re-landing r263633 Details in radar. Created attachment 403447 [details]
Patch for relanding
Committed r263891: <https://trac.webkit.org/changeset/263891> All reviewed patches have been landed. Closing bug and clearing flags on attachment 403447 [details]. |