Re: [1/5]dwrite: Implement Font File Loader registration

[Nikolay Sivov]

> +static HRESULT WINAPI fontfileloader_QueryInterface(IDWriteFontFileLoader *iface, REFIID riid, void **obj)
> +{
> - *obj = iface;
> - return S_OK;
>
> +}

This should follow normal QI behavior, with test failure on unexpected 
riids.
> +static HRESULT WINAPI fontfileloader_CreateStreamFromKey(IDWriteFontFileLoader *iface, const void *fontFileReferenceKey, UINT32 fontFileReferenceKeySize, IDWriteFontFileStream **fontFileStream)
> +{
> - return 0x8faecafe;

I'm sure this works, but it's cleaner to return E_FAIL/E_NOTIMPL imho.
> - hr = IDWriteFactory_RegisterFontFileLoader(factory, &floader);
> - ok(hr == S_OK, "got 0x%08x\n", hr);

Probably it makes sense to test NULL argument as well, but that's 
certainly optional (and dwrite tends to crash in such cases).