Re: [PATCH 2/2] wininet: Add support for INTERNET_OPTION_SETTINGS_CHANGED in InternetSetOption.

[Jacek Caban]

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