Re: [PATCH 0/10] MR1537: Draft: windows.media: Partially implement caption properties.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.media/tests/captions.c:
> -
>    todo_wine ok( hr == E_INVALIDARG, "get_ComputedFontColor returned %#lx\n", hr );
> - }
> - else
> - {
> -
>    todo_wine
> -
>    {
> -
>        ok( hr == S_OK, "get_ComputedFontColor returned %#lx\n", hr );
> -
>        ok( computed_color.A == 255, "expected default alpha font computed color, got %d\n", computed_color.A );
> -
>        ok( computed_color.R == 255, "expected default red font computed color, got %d\n", computed_color.R );
> -
>        ok( computed_color.G == 255, "expected default green font computed color, got %d\n", computed_color.G );
> -
>        ok( computed_color.B == 255, "expected default blue font computed color, got %d\n", computed_color.B );
> -
>    }
> -
> -
>    hr = IClosedCaptionPropertiesStatics_get_ComputedFontColor( caption_statics, NULL );
> -
>    todo_wine ok( hr == S_OK, "get_ComputedFontColor returned %#lx\n", hr );
> - }

It's hard to tell which of these branches is the expected behavior.