Re: [PATCH v7 0/2] MR2778: mfmediaengine: Add support for video effects.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/tests/mfmediaengine.c:
> +    hr = IMFMediaEngineEx_InsertVideoEffect(media_engine_ex, (IUnknown *)&video_effect->IMFTransform_iface, FALSE);
> +    todo_wine ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> +    EXPECT_REF(&video_effect->IMFTransform_iface, 2);
> +
> +    hr = IMFMediaEngineEx_InsertVideoEffect(media_engine_ex, (IUnknown *)&video_effect2->IMFTransform_iface, FALSE);
> +    todo_wine ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> +    EXPECT_REF(&video_effect2->IMFTransform_iface, 2);
> +
> +    hr = IMFMediaEngineEx_InsertVideoEffect(media_engine_ex, (IUnknown *)&video_effect3->IMFTransform_iface, FALSE);
> +    todo_wine ok(hr == S_OK || broken(hr == MF_E_INVALIDREQUEST), "Unexpected hr %#lx.\n", hr);
> +
> +    if (FAILED(hr)) /* Win8 to Win10v1507 and Wine */
> +    {
> +        goto done;
> +    }
> +    EXPECT_REF(&video_effect3->IMFTransform_iface, 2);
> This part that I mentioned before might as well go away, if two effects are always supported.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2778#note_36456