Bug 274382

Summary: [MSE] MediaPlayerPrivateMediaSourceAVFObjC not notifying MediaPlayer of playback state change
Product: WebKit Reporter: Philippe Normand <philn>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: eric.carlson, jean-yves.avenard, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Philippe Normand
Reported 2024-05-20 02:30:05 PDT
The test introduced in https://github.com/WebKit/WebKit/pull/28770 fails on mac. While checking the MediaPlayerPrivateMediaSourceAVFObjC code I couldn't find where MediaPlayer::playbackStateChanged() is called. It's done correctly in MediaPlayerPrivateAVFoundationObjC but the MediaSource player doesn't inherit from that one.
Attachments
Philippe Normand
Comment 1 2024-05-20 02:31:14 PDT
Actual test result: RUN(video.src = URL.createObjectURL(source)) EVENT(sourceopen) RUN(sourceBuffer = source.addSourceBuffer(loader.type())) RUN(sourceBuffer.appendBuffer(loader.initSegment())) EVENT(update) Append all media segments EXPECTED (internals.elementIsBlockingDisplaySleep(video) == 'false') OK RUN(video.play()) EVENT(playing) EVENT(timeupdate) EXPECTED (internals.elementIsBlockingDisplaySleep(video) == 'true') OK RUN(video.pause()) EVENT(pause) EXPECTED (internals.elementIsBlockingDisplaySleep(video) == 'false'), OBSERVED 'true' FAIL END OF TEST
Radar WebKit Bug Importer
Comment 2 2024-05-27 02:31:12 PDT
Jean-Yves Avenard [:jya]
Comment 3 2024-05-27 20:30:00 PDT
it is done in MediaSource::monitorSourceBuffers
Note You need to log in before you can comment on or make changes to this bug.