Re: [PATCH v5 0/3] MR1343: windows.media: Add WinRT Closed Captioning interface.

[Bernhard Kölbl (＠besentv)]

Bernhard Kölbl (@besentv) commented about dlls/windows.media/tests/captions.c:
> -
>    IUnknown_Release( unk );
>
> +}
> -
>
> +static void test_CaptionStatics(void)
> +{
> - static const WCHAR *caption_properties_name = L"Windows.Media.ClosedCaptioning.ClosedCaptionProperties";
> - IClosedCaptionPropertiesStatics *caption_statics;
> - IActivationFactory *factory;
> - HSTRING str;
> - HRESULT hr;
> - LONG ref;
> -
> - hr = WindowsCreateString( caption_properties_name, wcslen( caption_properties_name ), &str );
> - ok( hr == S_OK, "got hr %#lx.\n", hr );
> -
> - hr = RoGetActivationFactory( str, &IID_IActivationFactory, (void **)&factory );

Maybe add the test for the Activation factory in the first commit so your code is covered by tests.