Re: [PATCH v6 0/8] MR6800: mf: Handle unexpected media source shutdown in a media session.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/mf/tests/mf.c:
> -
>    ok(hr == S_OK, "Failed to create grabber sink, hr %#lx.\n", hr);
> -
>    IMFMediaType_Release(output_type);
> -
> -
>    source = create_media_source(L"test.mp4", L"video/mp4");
> -
>    ok(!!source, "Failed to create source.\n");
> -
> -
>    topology = create_test_topology(source, sink_activate, &duration);
> -
>    hr = IMFMediaSession_SetTopology(session, 0, topology);
> -
>    ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> -
>    IMFTopology_Release(topology);
> -
> -
>    propvar.vt = VT_EMPTY;
> -
>    hr = IMFMediaSession_Start(session, &GUID_NULL, &propvar);
> -
>    ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> -
>    hr = wait_media_event_until_blocking(session, callback, MESessionStarted, 5000, &propvar);
> -
>    flaky_wine

How flaky is this? Is it not going to be fixed after this MR? It doesn't seem very useful to add a test that's flaky from the beginning, unless you can fix it.