Re: [PATCH 0/16] MR4046: windows.media.mediacontrol: Implement some IMusicDisplayProperties and IMusicDisplayProperties2 properties.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.media.mediacontrol/tests/mediacontrol.c:
>  hr = ISystemMediaTransportControls_get_DisplayUpdater( media_control_statics, &display_updater );
>  ok( hr == S_OK, "got hr %#lx.\n", hr );
>
> - hr = ISystemMediaTransportControlsDisplayUpdater_put_Type( display_updater, -1 );
> - todo_wine ok( hr == E_INVALIDARG, "got hr %#lx.\n", hr );
> - hr = ISystemMediaTransportControlsDisplayUpdater_put_Type( display_updater, 4 );
> - todo_wine ok( hr == E_INVALIDARG, "got hr %#lx.\n", hr );
> - hr = ISystemMediaTransportControlsDisplayUpdater_put_Type( display_updater, 1 );
> - todo_wine ok( hr == S_OK, "got hr %#lx.\n", hr );
> -
> - playback_type = -1;
> - hr = ISystemMediaTransportControlsDisplayUpdater_get_Type( display_updater, &playback_type );
> - todo_wine ok( hr == S_OK, "got hr %#lx.\n", hr );
> - todo_wine ok( playback_type == 1, "got playback_type %d.\n", playback_type );

Would be better to use MediaPlaybackType values here, except for the invalid ones.