| Summary: | http/wpt/mediarecorder/mute-tracks.html fails with WebRTCRemoteVideoFrameEnabled=true | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | youenn fablet <youennf> | ||||
| Component: | WebRTC | Assignee: | youenn fablet <youennf> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | eric.carlson, ews-watchlist, glenn, jer.noble, kkinnunen, philipj, sergio, webkit-bug-importer, youennf | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | WebKit Nightly Build | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 236969 | ||||||
| Attachments: |
|
||||||
|
Description
youenn fablet
2022-02-22 00:00:22 PST
Created attachment 452833 [details]
Patch
Comment on attachment 452833 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452833&action=review > Source/WebKit/WebProcess/GPU/webrtc/MediaRecorderPrivate.cpp:99 > // FIXME: We could optimize sending black frames by only sending width/height. I think it'd be the time to address the fixme. Sending dummy pixels of possibly invalid size.. (In reply to Kimmo Kinnunen from comment #2) > Comment on attachment 452833 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=452833&action=review > > > Source/WebKit/WebProcess/GPU/webrtc/MediaRecorderPrivate.cpp:99 > > // FIXME: We could optimize sending black frames by only sending width/height. > > I think it'd be the time to address the fixme. Sending dummy pixels of > possibly invalid size.. Yes, I have a local patch that does that through a new SharedVideoFrame that will replace RemoteVideoSample. As of the size, when the track is muted, the spec is very light on whether we should respect the size of the frame or not. Currently, we stick to using the same size over and over, which is probably the safest option. Committed r290307 (247630@main): <https://commits.webkit.org/247630@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 452833 [details]. |