Recover MediaStreamTrack audio rendering from GPUProcess crash
Created attachment 418718 [details] Patch
Comment on attachment 418718 [details] Patch WK1 failure is unrelated
<rdar://problem/73838195>
Comment on attachment 418718 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=418718&action=review > Source/WebCore/platform/mediastream/AudioMediaStreamTrackRenderer.h:108 > + Nit. An extra empty line. > Source/WebCore/platform/mediastream/AudioTrackPrivateMediaStream.cpp:172 > +void AudioTrackPrivateMediaStream::createNewRenderer() Nit. Seems a callback. Maybe we can name it "newRendererCreated"?
(In reply to Peng Liu from comment #4) > Comment on attachment 418718 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=418718&action=review > > > Source/WebCore/platform/mediastream/AudioMediaStreamTrackRenderer.h:108 > > + > > Nit. An extra empty line. Will fix. > > Source/WebCore/platform/mediastream/AudioTrackPrivateMediaStream.cpp:172 > > +void AudioTrackPrivateMediaStream::createNewRenderer() > > Nit. Seems a callback. Maybe we can name it "newRendererCreated"? Not really, it is actually creating a new renderer, even though this is triggered by a callback telling current renderer is crashing.
Created attachment 418977 [details] Patch
Committed r272213: <https://trac.webkit.org/changeset/272213> All reviewed patches have been landed. Closing bug and clearing flags on attachment 418977 [details].