[webkit-changes] [WebKit/WebKit] 2562b4: [MediaRecorder/WebM] First frames are missed in a ...

[Jean-Yves Avenard]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2562b4f9c893a95f830c2244efbd9d919109ef88
      https://github.com/WebKit/WebKit/commit/2562b4f9c893a95f830c2244efbd9d919109ef88
  Author: Jean-Yves Avenard <jya at apple.com>
  Date:   2024-10-15 (Tue, 15 Oct 2024)

  Changed paths:
    A LayoutTests/http/wpt/mediarecorder/MediaRecorder-first-frame-expected.txt
    A LayoutTests/http/wpt/mediarecorder/MediaRecorder-first-frame.html
    M LayoutTests/platform/glib/TestExpectations
    M Source/WebCore/platform/mediarecorder/cocoa/MediaRecorderPrivateWriterWebM.mm

  Log Message:
  -----------
  [MediaRecorder/WebM] First frames are missed in a recording
https://bugs.webkit.org/show_bug.cgi?id=281406
rdar://137854083

Reviewed by Eric Carlson.

Don't drop video frames received before the first audio frame.
Manual expirementation shows that we don't need to worry about shifting the timestamp
of the first audio frame by the delay between first video and first audio.

Added test.

* LayoutTests/http/wpt/mediarecorder/MediaRecorder-first-frame-expected.txt: Added.
* LayoutTests/http/wpt/mediarecorder/MediaRecorder-first-frame.html: Added.
* LayoutTests/platform/glib/TestExpectations:
* Source/WebCore/platform/mediarecorder/cocoa/MediaRecorderPrivateWriterWebM.mm:
(WebCore::MediaRecorderPrivateWriterWebM::appendVideoFrame):

Canonical link: https://commits.webkit.org/285236@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications