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.