[PATCH 2/2 try2] wininet: Added support for persistent HTTP connections

[Jacek Caban]

On 05/04/11 16:07, Alexandre Julliard wrote:
> Jacek Caban <jacek at codeweavers.com> writes:
>
>> try2: Using GetTickCount64() for timestamps
>>
>> ---
>>  dlls/urlmon/tests/protocol.c |    2 +-
>>  dlls/urlmon/tests/url.c      |   20 +-
>>  dlls/wininet/http.c          |  567
>> ++++++++++++++++++++++++------------------
>>  dlls/wininet/internet.c      |    2 +-
>>  dlls/wininet/internet.h      |   42 +++-
>>  dlls/wininet/netconnection.c |  138 ++++-------
>>  dlls/wininet/tests/http.c    |   32 +--
>>  7 files changed, 427 insertions(+), 376 deletions(-)
> It doesn't work here:
>
> ../../../tools/runtest -q -P wine -M wininet.dll -T ../../.. -p wininet_test.exe.so http.c && touch http.ok
> http.c:2621: Test failed: expected 1 retrievals, got 0
> http.c:2628: Test failed: count was wrong
> http.c:2629: Test failed: http data wrong
> http.c:2649: Test succeeded inside todo block: expected 1 retrievals
> make: *** [http.ok] Error 4

That's an already existing race in tests. It fails on Windows as well:
http://test.winehq.org/data/a343c1fbd0f37a5cadd98698ae5d580727ddf0c7/2003_fg-win2003es-sp2/wininet:http.html
I've seen it here and my patch seems to make is more likely to fail. I
plan fixing it, but, as it's not new, so I think it shouldn't block
committing my patch.

Jacek