[PATCH 2/2] wininet: Add support for INTERNET_OPTION_SETTINGS_CHANGED in InternetSetOption.
Jacek Caban
jacek at codeweavers.com
Mon Apr 30 10:37:31 CDT 2018
Hi Alistair,
On 04/30/2018 03:00 AM, Alistair Leslie-Hughes wrote:
> + ret = InternetSetOptionA(ses, INTERNET_OPTION_REFRESH, NULL, 0);
> + ok(ret == TRUE, "InternetSetOption should've succeeded\n");
> +
> + SetLastError(0xdeadbeef);
> + ret = InternetSetOptionA(req, INTERNET_OPTION_REFRESH, NULL, 0);
> + todo_wine ok(ret == FALSE, "InternetSetOption should've failed\n");
> + todo_wine ok(GetLastError() == ERROR_INTERNET_INCORRECT_HANDLE_TYPE, "GetLastError() = %x\n", GetLastError());
Given those tests, I think the implementation should go to
HTTPSESSION_SetOption instead.
Thanks,
Jacek
More information about the wine-devel
mailing list