Re: [PATCH v2 0/3] MR8951: windows.media.playback.mediaplayer: Implement IMediaPlayer2::get_SystemMediaTransportControls().

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/windows.media.playback.mediaplayer/main.c:
> -    FIXME( "iface %p, value %p stub!\n", iface, value );
> -    return E_NOTIMPL;
> +    struct media_player *impl = impl_from_IMediaPlayer2( iface );
> +    HRESULT hr;
> +
> +    TRACE( "iface %p, value %p\n", iface, value );
> +
> +    EnterCriticalSection( &impl->cs );
> +
> +    if (!impl->controls && FAILED(hr = get_system_media_transport_controls( &impl->window, &impl->controls )))
> +    {
> +        if (impl->window)
> +        {
> +            DestroyWindow( impl->window );
> +            impl->window = NULL;
> +        }

Then you wouldn't need this.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8951#note_115652