[PATCH 0/2] MR10306: mfmediaengine: Implement looping behaviour.

[Brendan McGrath (＠redmcg)]

When `IMFMediaEngine::SetLoop(TRUE)` is called, media engine will repeatedly loop the video.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10306

[PATCH 1/2] mfmediaengine: Reset the sample requested flag on Stop.

[Brendan McGrath]

From: Brendan McGrath <[e-mail]>

When a sink is stopped, all pending requests are discarded by the client
(for example IMFMediaSession).
---
 dlls/mfmediaengine/video_frame_sink.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/dlls/mfmediaengine/video_frame_sink.c b/dlls/mfmediaengine/video_frame_sink.c
index f710466fed4..301a2bcd03a 100644
--- a/dlls/mfmediaengine/video_frame_sink.c
+++ b/dlls/mfmediaengine/video_frame_sink.c
@@ -1010,6 +1010,7 @@ static HRESULT video_frame_sink_set_state(struct video_frame_sink *sink, enum si
             {
                 video_frame_sink_sample_queue_flush(sink);
                 video_frame_sink_set_flag(sink, FLAGS_FIRST_FRAME, FALSE);
+                sink->sample_request_pending = FALSE;
             }
 
             if (state == SINK_STATE_RUNNING && (sink->state == SINK_STATE_STOPPED || sink->state == SINK_STATE_PAUSED ||
-- 
GitLab


https://gitlab.winehq.org/wine/wine/-/merge_requests/10306

[PATCH 2/2] mfmediaengine: Implement looping behaviour.

[Brendan McGrath]

From: Brendan McGrath <[e-mail]>

---
 dlls/mfmediaengine/main.c | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/dlls/mfmediaengine/main.c b/dlls/mfmediaengine/main.c
index 2ababd674d7..74dfefccf4b 100644
--- a/dlls/mfmediaengine/main.c
+++ b/dlls/mfmediaengine/main.c
@@ -1008,14 +1008,20 @@ static HRESULT WINAPI media_engine_session_events_Invoke(IMFAsyncCallback *iface
             IMFMediaEngineNotify_EventNotify(engine->callback, MF_MEDIA_ENGINE_EVENT_PLAYING, 0, 0);
             break;
         case MESessionEnded:
-
             EnterCriticalSection(&engine->cs);
-            media_engine_set_flag(engine, FLAGS_ENGINE_FIRST_FRAME, FALSE);
-            media_engine_set_flag(engine, FLAGS_ENGINE_IS_ENDED, TRUE);
-            engine->video_frame.pts = MINLONGLONG;
+            if (engine->flags & FLAGS_ENGINE_LOOP)
+            {
+                 media_engine_set_current_time(engine, 0.0);
+            }
+            else
+            {
+                engine->video_frame.pts = MINLONGLONG;
+                media_engine_set_flag(engine, FLAGS_ENGINE_FIRST_FRAME, FALSE);
+                media_engine_set_flag(engine, FLAGS_ENGINE_IS_ENDED, TRUE);
+                LeaveCriticalSection(&engine->cs);
+                IMFMediaEngineNotify_EventNotify(engine->callback, MF_MEDIA_ENGINE_EVENT_ENDED, 0, 0);
+            }
             LeaveCriticalSection(&engine->cs);
-
-            IMFMediaEngineNotify_EventNotify(engine->callback, MF_MEDIA_ENGINE_EVENT_ENDED, 0, 0);
             break;
 
         case MEEndOfPresentationSegment:
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/10306

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/main.c:
> -            media_engine_set_flag(engine, FLAGS_ENGINE_IS_ENDED, TRUE);
> -            engine->video_frame.pts = MINLONGLONG;
> +            if (engine->flags & FLAGS_ENGINE_LOOP)
> +            {
> +                 media_engine_set_current_time(engine, 0.0);
> +            }
> +            else
> +            {
> +                engine->video_frame.pts = MINLONGLONG;
> +                media_engine_set_flag(engine, FLAGS_ENGINE_FIRST_FRAME, FALSE);
> +                media_engine_set_flag(engine, FLAGS_ENGINE_IS_ENDED, TRUE);
> +                LeaveCriticalSection(&engine->cs);
> +                IMFMediaEngineNotify_EventNotify(engine->callback, MF_MEDIA_ENGINE_EVENT_ENDED, 0, 0);
> +            }
>              LeaveCriticalSection(&engine->cs);
> -
> There are now two leave's for one enter.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10306#note_132040