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

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.media/tests/captions.c:
>  hr = IActivationFactory_QueryInterface( factory, &IID_IClosedCaptionPropertiesStatics, (void **)&caption_statics );
>  ok( hr == S_OK, "got hr %#lx.\n", hr );
>
> - color = 0xdeadbeef;
> - hr = IClosedCaptionPropertiesStatics_get_FontColor( caption_statics, &color );
> - todo_wine ok( hr == S_OK, "get_FontColor returned %#lx\n", hr );
> - todo_wine ok( color == ClosedCaptionColor_Default, "expected default font color, got %d\n", color );
> -
> - hr = IClosedCaptionPropertiesStatics_get_ComputedFontColor( caption_statics, &computed_color );
> - if (color == ClosedCaptionColor_Default)

I still don't really understand, you're checking right above that the color is the default, so it should never be different here. Simply add the todos for wine failure here too.