[PATCH] urlmon: Don't allow direct sink callbacks when reporting data.

[Jacek Caban]

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=46213
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52286
Signed-off-by: Jacek Caban <jacek(a)codeweavers.com>
---
Supersedes 222914, 222932, 222998, 222999.

  dlls/urlmon/bindprot.c       | 49 +++++++++++++++++++++-------------
  dlls/urlmon/tests/protocol.c | 51 +++++++++++++++++++++++++++---------
  2 files changed, 70 insertions(+), 30 deletions(-)

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=104962

Your paranoid android.


=== w8adm (32 bit report) ===

urlmon:
protocol.c:1196: Test failed: hrResult = 800c0005, expected: 00000000
protocol.c:3460: Test failed: wait timed out
protocol.c:3464: Test failed: expected ReportData
protocol: Timeout

=== w864 (32 bit report) ===

urlmon:
protocol.c:732: Test failed: unexpected call Switch

=== w1064_tsign (64 bit report) ===

urlmon:
protocol.c:1188: Test failed: unexpected call ReportResult
protocol.c:1196: Test failed: hrResult = 00000000, expected: 80004004
protocol.c:3334: Test failed: Read failed: 00000001

=== w10pro64_ja (64 bit report) ===

urlmon:
protocol.c:1188: Test failed: unexpected call ReportResult
protocol.c:1196: Test failed: hrResult = 00000000, expected: 80004004
protocol.c:3334: Test failed: Read failed: 00000001