Title: [271491] branches/safari-611-branch/Source/WebKit
- Revision
- 271491
- Author
- [email protected]
- Date
- 2021-01-14 10:15:50 -0800 (Thu, 14 Jan 2021)
Log Message
Cherry-pick r271477. rdar://problem/73204264
REGRESSION(r270961): Flipgrid can no longer capture video
https://bugs.webkit.org/show_bug.cgi?id=220609
<rdar://73168009>
Reviewed by Darin Adler.
Add back line in RemoteAudio::setStorage() that was initializing m_buffer and that was
dropped by mistake in the refactoring in r270961. This was causing
RemoteCaptureSampleManager::RemoteAudio::audioSamplesAvailable() to return early because
m_buffer was not initialized.
No new tests, unsure how to test this.
* WebProcess/cocoa/RemoteCaptureSampleManager.cpp:
(WebKit::RemoteCaptureSampleManager::RemoteAudio::setStorage):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@271477 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-611-branch/Source/WebKit/ChangeLog (271490 => 271491)
--- branches/safari-611-branch/Source/WebKit/ChangeLog 2021-01-14 18:12:42 UTC (rev 271490)
+++ branches/safari-611-branch/Source/WebKit/ChangeLog 2021-01-14 18:15:50 UTC (rev 271491)
@@ -1,3 +1,44 @@
+2021-01-14 Alan Coon <[email protected]>
+
+ Cherry-pick r271477. rdar://problem/73204264
+
+ REGRESSION(r270961): Flipgrid can no longer capture video
+ https://bugs.webkit.org/show_bug.cgi?id=220609
+ <rdar://73168009>
+
+ Reviewed by Darin Adler.
+
+ Add back line in RemoteAudio::setStorage() that was initializing m_buffer and that was
+ dropped by mistake in the refactoring in r270961. This was causing
+ RemoteCaptureSampleManager::RemoteAudio::audioSamplesAvailable() to return early because
+ m_buffer was not initialized.
+
+ No new tests, unsure how to test this.
+
+ * WebProcess/cocoa/RemoteCaptureSampleManager.cpp:
+ (WebKit::RemoteCaptureSampleManager::RemoteAudio::setStorage):
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@271477 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2021-01-13 Chris Dumez <[email protected]>
+
+ REGRESSION(r270961): Flipgrid can no longer capture video
+ https://bugs.webkit.org/show_bug.cgi?id=220609
+ <rdar://73168009>
+
+ Reviewed by Darin Adler.
+
+ Add back line in RemoteAudio::setStorage() that was initializing m_buffer and that was
+ dropped by mistake in the refactoring in r270961. This was causing
+ RemoteCaptureSampleManager::RemoteAudio::audioSamplesAvailable() to return early because
+ m_buffer was not initialized.
+
+ No new tests, unsure how to test this.
+
+ * WebProcess/cocoa/RemoteCaptureSampleManager.cpp:
+ (WebKit::RemoteCaptureSampleManager::RemoteAudio::setStorage):
+
2021-01-13 Russell Epstein <[email protected]>
Cherry-pick r271448. rdar://problem/73168438
Modified: branches/safari-611-branch/Source/WebKit/WebProcess/cocoa/RemoteCaptureSampleManager.cpp (271490 => 271491)
--- branches/safari-611-branch/Source/WebKit/WebProcess/cocoa/RemoteCaptureSampleManager.cpp 2021-01-14 18:12:42 UTC (rev 271490)
+++ branches/safari-611-branch/Source/WebKit/WebProcess/cocoa/RemoteCaptureSampleManager.cpp 2021-01-14 18:15:50 UTC (rev 271491)
@@ -121,6 +121,7 @@
{
m_description = description;
m_ringBuffer = makeUnique<CARingBuffer>(makeUniqueRef<ReadOnlySharedRingBufferStorage>(handle), description, numberOfFrames);
+ m_buffer = makeUnique<WebAudioBufferList>(description, numberOfFrames);
}
void RemoteCaptureSampleManager::RemoteAudio::audioSamplesAvailable(MediaTime time, uint64_t numberOfFrames)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
