Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 8194ca4f8b6fa18634b140720c8933e722193e8d
      
https://github.com/WebKit/WebKit/commit/8194ca4f8b6fa18634b140720c8933e722193e8d
  Author: Youenn Fablet <[email protected]>
  Date:   2025-01-07 (Tue, 07 Jan 2025)

  Changed paths:
    M LayoutTests/fast/mediastream/media-element-current-time.html
    M 
Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.mm
    M Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp

  Log Message:
  -----------
  HTML audio element currentTime property too high when setting srcObject to 
MediaStream (macOS and iOS)
https://bugs.webkit.org/show_bug.cgi?id=285391
rdar://142465970

Reviewed by Jean-Yves Avenard.

We were not initializing m_startTime to an invalid value since 
MediaPlayerPrivateMediaStreamAVFObjC::play is checking for whether it is valid 
or not.
We initialize m_startTime to an invalid value so that it can be correctly 
initialized in MediaPlayerPrivateMediaStreamAVFObjC::play.

Covered by updated test.

* LayoutTests/fast/mediastream/media-element-current-time.html:
* 
Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.mm:
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::MediaPlayerPrivateMediaStreamAVFObjC):
* Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
(WebCore::MediaPlayerPrivateGStreamer::MediaPlayerPrivateGStreamer):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to