Re: [PATCH v7 0/2] MR8459: urlmon: Add PersistentZoneIdentifier semi-stub

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/urlmon/tests/zone_id.c:
> - const save_file_test save = { NULL, FALSE, E_INVALIDARG };
> - IUnknown *unk;
> - IZoneIdentifier *zone_id;
> - IPersistFile *persist_file;
> - HRESULT hres;
> - DWORD zone;
> - CLSID clsid;
> - LPWSTR file_name;
> -
> - hres = CoCreateInstance(&CLSID_PersistentZoneIdentifier, NULL,
> -
>    CLSCTX_INPROC_SERVER, &IID_IUnknown, (void**)&unk);
> - if (hres != S_OK)
> - {
> -
>    skip("%s failed to obtain IUnknown iface: 0x%08lx\n", uninit_tests, hres);
> -
>    return;
> - }

Are there any Windows versions that fail to create the zone identifier? If not, this should use `ok()` instead. If there are, it should be `win_skip()` to avoid unintentionally accepting such skips on Wine. The same applies to all other `skip()` calls in the patch.