Re: [PATCH v2 0/6] MR3439: mf: Implement basic urlmon scheme handler.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfreadwrite/reader.c:
> - hr = IMFSourceResolver_CreateObjectFromURL(resolver, url, MF_RESOLUTION_MEDIASOURCE, props, &obj_type,
> -
>        &object);
> - hr = IMFSourceResolver_CreateObjectFromURL(resolver, url, MF_RESOLUTION_MEDIASOURCE
> -
>        | MF_RESOLUTION_CONTENT_DOES_NOT_HAVE_TO_MATCH_EXTENSION_OR_MIME_TYPE, props, &obj_type, &object);
> if (SUCCEEDED(hr))
>    {
>        switch (obj_type)
>        {
>            case MF_OBJECT_BYTESTREAM:
> -
>            hr = IMFSourceResolver_CreateObjectFromByteStream(resolver, (IMFByteStream *)object, NULL,
> -
>                    MF_RESOLUTION_MEDIASOURCE, props, &obj_type, (IUnknown **)&source);
> -
>            hr = IMFSourceResolver_CreateObjectFromByteStream(resolver, (IMFByteStream *)object, NULL, MF_RESOLUTION_MEDIASOURCE
> -
>                    | MF_RESOLUTION_CONTENT_DOES_NOT_HAVE_TO_MATCH_EXTENSION_OR_MIME_TYPE, props, &obj_type, (IUnknown **)&source);
>            break;
>        case MF_OBJECT_MEDIASOURCE:
>            source = (IMFMediaSource *)object;

Could you add a test for this? I know we are using this flag for stream case, and I think it's correct. When creating from url, using test.wav with e.g. mp3 content type should be enough I guess.