Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 81376ca14bbf0b8e69cfa36bef4e81a76d9aa7a6
      
https://github.com/WebKit/WebKit/commit/81376ca14bbf0b8e69cfa36bef4e81a76d9aa7a6
  Author: Xabier Rodriguez-Calvar <[email protected]>
  Date:   2024-06-19 (Wed, 19 Jun 2024)

  Changed paths:
    M 
Source/WebCore/platform/graphics/gstreamer/mse/MediaPlayerPrivateGStreamerMSE.cpp

  Log Message:
  -----------
  [GStreamer][MSE] Take playbin's states lock when sending seek event
https://bugs.webkit.org/show_bug.cgi?id=275566

Reviewed by Alicia Boya Garcia.

This fixes possible race between application (triggering another seek from 
'seeked' event) and 'state chagne'
continuation (triggered by playbin).

Top level bin element does change the state as result of 'async-done' handling 
from previous seek request (see
gst_bin_continue_func in gstbin.c). Which may race with handling of 
'async-start' posted by sinks on flushing seek. And
may leave the pipeline in inconsistent state and 'hanging' seek that never 
finishes.

By taking the states lock of top level bin element player will wait for 
possible state change continuation to complete
before sending next seek event.

Based on a patch by Eugene Mutavchi <[email protected]>.

* 
Source/WebCore/platform/graphics/gstreamer/mse/MediaPlayerPrivateGStreamerMSE.cpp:
(WebCore::MediaPlayerPrivateGStreamerMSE::doSeek):

Canonical link: https://commits.webkit.org/280168@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