[PATCH 1/2] mf/session: Use a dedicated state when source is shutdown.

[Rémi Bernon]

From: Rémi Bernon

Until the topology is set again.
---
 dlls/mf/session.c | 27 +++++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/dlls/mf/session.c b/dlls/mf/session.c
index 47760cc44a7..e6f37146221 100644
--- a/dlls/mf/session.c
+++ b/dlls/mf/session.c
@@ -103,6 +103,7 @@ enum session_state
     SESSION_STATE_STOPPING_SINKS,
     SESSION_STATE_STOPPING_SOURCES,
     SESSION_STATE_FINALIZING_SINKS,
+    SESSION_STATE_SOURCE_SHUTDOWN,
     SESSION_STATE_CLOSED,
     SESSION_STATE_SHUT_DOWN,
 };
@@ -1051,7 +1052,7 @@ static void session_reset(struct media_session *session)
 {
     /* Media sessions in native Windows are not consistently usable after a
      * forced source shutdown, but we try to clean up as well as possible. */
-    session->state = SESSION_STATE_STOPPED;
+    session->state = SESSION_STATE_SOURCE_SHUTDOWN;
     session_clear_presentation(session);
     session_purge_pending_commands(session);
     session_command_complete(session);
@@ -1061,7 +1062,7 @@ static void session_handle_start_error(struct media_session *session, HRESULT hr
 {
     if (hr == MF_E_SHUTDOWN)
     {
-        session_reset(session);
+        session_source_shutdown(session);
         hr = MF_E_INVALIDREQUEST;
     }
     session_command_complete_with_event(session, MESessionStarted, hr, NULL);
@@ -1155,6 +1156,9 @@ static void session_start(struct media_session *session, const GUID *time_format
             /* SESSION_STATE_STARTED -> SESSION_STATE_RESTARTING_SOURCES -> SESSION_STATE_STARTED */
             session->state = SESSION_STATE_RESTARTING_SOURCES;
             break;
+        case SESSION_STATE_SOURCE_SHUTDOWN:
+            session_command_complete_with_event(session, MESessionStarted, MF_E_SHUTDOWN, NULL);
+            break;
         default:
             session_command_complete_with_event(session, MESessionStarted, MF_E_INVALIDREQUEST, NULL);
             break;
@@ -1233,6 +1237,9 @@ static void session_pause(struct media_session *session)
         case SESSION_STATE_STOPPED:
             hr = MF_E_SESSION_PAUSEWHILESTOPPED;
             break;
+        case SESSION_STATE_SOURCE_SHUTDOWN:
+            hr = MF_E_SHUTDOWN;
+            break;
         default:
             hr = MF_E_INVALIDREQUEST;
     }
@@ -1294,10 +1301,13 @@ static void session_stop(struct media_session *session)

break;
         case SESSION_STATE_STOPPED:
-            hr = S_OK;
-            /* fallthrough */
+            session_command_complete_with_event(session, MESessionStopped, S_OK, NULL);
+            break;
+        case SESSION_STATE_SOURCE_SHUTDOWN:
+            session_command_complete_with_event(session, MESessionStopped, MF_E_SHUTDOWN, NULL);
+            break;
         default:
-            session_command_complete_with_event(session, MESessionStopped, hr, NULL);
+            session_command_complete_with_event(session, MESessionStopped, MF_E_INVALIDREQUEST, NULL);
             break;
     }
 }
@@ -1349,6 +1359,9 @@ static void session_close(struct media_session *session)
             if (SUCCEEDED(hr = IMFPresentationClock_Stop(session->clock)))
                 session->state = SESSION_STATE_STOPPING_SINKS;
             break;
+        case SESSION_STATE_SOURCE_SHUTDOWN:
+            hr = MF_E_SHUTDOWN;
+            break;
         default:
             hr = MF_E_INVALIDREQUEST;
             break;
@@ -2012,6 +2025,8 @@ static HRESULT session_set_current_topology(struct media_session *session, IMFTo
     }

session->source_shutdown_handled = FALSE;
+    if (session->state == SESSION_STATE_SOURCE_SHUTDOWN)
+        session->state = SESSION_STATE_STOPPED;

session_collect_nodes(session);

@@ -2960,7 +2975,7 @@ static void session_handle_source_shutdown(struct media_session *session)
         if (finalize_sinks)
             session_finalize_sinks(session);
         else
-            session_reset(session);
+            session_source_shutdown(session);
     }

LeaveCriticalSection(&session->cs);