wine-commits December 2007

wine-commits@winehq.org

3 participants
824 discussions

Jacek Caban : mshtml: Use IBindCtx passed to Load in BindToStorage call.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: 82a199bcee410a32f221595bb07113bfd58ca900 URL: http://source.winehq.org/git/wine.git/?a=commit;h=82a199bcee410a32f221595bb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 12:48:43 2007 +0100 mshtml: Use IBindCtx passed to Load in BindToStorage call. --- dlls/mshtml/mshtml_private.h | 2 +- dlls/mshtml/navigate.c | 18 +++++++++++------- dlls/mshtml/persist.c | 2 +- dlls/mshtml/task.c | 2 +- 4 files changed, 14 insertions(+), 10 deletions(-) diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index 3e0c8eb..2180184 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -425,7 +425,7 @@ void init_nsevents(NSContainer*); nsresult get_nsinterface(nsISupports*,REFIID,void**); BSCallback *create_bscallback(IMoniker*); -HRESULT start_binding(HTMLDocument*,BSCallback*); +HRESULT start_binding(HTMLDocument*,BSCallback*,IBindCtx*); HRESULT load_stream(BSCallback*,IStream*); void set_document_bscallback(HTMLDocument*,BSCallback*); void set_current_mon(HTMLDocument*,IMoniker*); diff --git a/dlls/mshtml/navigate.c b/dlls/mshtml/navigate.c index 29a3798..b1b5609 100644 --- a/dlls/mshtml/navigate.c +++ b/dlls/mshtml/navigate.c @@ -829,20 +829,24 @@ void hlink_frame_navigate(HTMLDocument *doc, IHlinkFrame *hlink_frame, IBindStatusCallback_Release(STATUSCLB(callback)); } -HRESULT start_binding(HTMLDocument *doc, BSCallback *bscallback) +HRESULT start_binding(HTMLDocument *doc, BSCallback *bscallback, IBindCtx *bctx) { IStream *str = NULL; - IBindCtx *bctx; HRESULT hres; bscallback->doc = doc; call_docview_84(bscallback->doc); - hres = CreateAsyncBindCtx(0, STATUSCLB(bscallback), NULL, &bctx); - if(FAILED(hres)) { - WARN("CreateAsyncBindCtx failed: %08x\n", hres); - on_stop_nsrequest(bscallback); - return hres; + if(bctx) { + RegisterBindStatusCallback(bctx, STATUSCLB(bscallback), NULL, 0); + IBindCtx_AddRef(bctx); + }else { + hres = CreateAsyncBindCtx(0, STATUSCLB(bscallback), NULL, &bctx); + if(FAILED(hres)) { + WARN("CreateAsyncBindCtx failed: %08x\n", hres); + on_stop_nsrequest(bscallback); + return hres; + } } hres = IMoniker_BindToStorage(bscallback->mon, bctx, NULL, &IID_IStream, (void**)&str); diff --git a/dlls/mshtml/persist.c b/dlls/mshtml/persist.c index c48fc64..2f3d5cc 100644 --- a/dlls/mshtml/persist.c +++ b/dlls/mshtml/persist.c @@ -429,7 +429,7 @@ static HRESULT WINAPI PersistMoniker_Load(IPersistMoniker *iface, BOOL fFullyAva return hres; if(!bind_complete) - return start_binding(This, This->bscallback); + return start_binding(This, This->bscallback, pibc); return S_OK; } diff --git a/dlls/mshtml/task.c b/dlls/mshtml/task.c index e612d81..b154022 100644 --- a/dlls/mshtml/task.c +++ b/dlls/mshtml/task.c @@ -214,7 +214,7 @@ static void set_progress(HTMLDocument *doc) static void task_start_binding(HTMLDocument *doc, BSCallback *bscallback) { if(doc) - start_binding(doc, bscallback); + start_binding(doc, bscallback, NULL); IBindStatusCallback_Release(STATUSCLB(bscallback)); }

1 0

Jacek Caban : urlmon: Fixed BindToObject tests.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: d067fabd383b2cc84931a47012b4f32dc8b9a09d URL: http://source.winehq.org/git/wine.git/?a=commit;h=d067fabd383b2cc84931a4701… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 13:21:27 2007 +0100 urlmon: Fixed BindToObject tests. --- dlls/urlmon/tests/url.c | 30 ++++++++++++------------------ 1 files changed, 12 insertions(+), 18 deletions(-) diff --git a/dlls/urlmon/tests/url.c b/dlls/urlmon/tests/url.c index 91c705d..9be2de7 100644 --- a/dlls/urlmon/tests/url.c +++ b/dlls/urlmon/tests/url.c @@ -39,17 +39,16 @@ #define SET_EXPECT(func) \ expect_ ## func = TRUE -#define CHECK_EXPECT(func) \ +#define CHECK_EXPECT2(func) \ do { \ ok(expect_ ##func, "unexpected call " #func "\n"); \ - expect_ ## func = FALSE; \ called_ ## func = TRUE; \ }while(0) -#define CHECK_EXPECT2(func) \ +#define CHECK_EXPECT(func) \ do { \ - ok(expect_ ##func, "unexpected call " #func "\n"); \ - called_ ## func = TRUE; \ + CHECK_EXPECT2(func); \ + expect_ ## func = FALSE; \ }while(0) #define CHECK_CALLED(func) \ @@ -1607,7 +1606,6 @@ static void test_BindToObject(int protocol, BOOL emul) LPOLESTR display_name; IBindCtx *bctx; MSG msg; - IBindStatusCallback *previousclb; IUnknown *unk = (IUnknown*)0x00ff00ff; IBinding *bind; @@ -1625,14 +1623,6 @@ static void test_BindToObject(int protocol, BOOL emul) return; CHECK_CALLED(QueryInterface_IServiceProvider); - SET_EXPECT(QueryInterface_IServiceProvider); - hres = RegisterBindStatusCallback(bctx, &bsc, &previousclb, 0); - ok(SUCCEEDED(hres), "RegisterBindStatusCallback failed: %08x\n", hres); - ok(previousclb == &bsc, "previousclb(%p) != sclb(%p)\n", previousclb, &bsc); - CHECK_CALLED(QueryInterface_IServiceProvider); - if(previousclb) - IBindStatusCallback_Release(previousclb); - hres = CreateURLMoniker(NULL, urls[test_protocol], &mon); ok(SUCCEEDED(hres), "failed to create moniker: %08x\n", hres); if(FAILED(hres)) { @@ -1652,8 +1642,10 @@ static void test_BindToObject(int protocol, BOOL emul) ok(hres == S_OK, "GetDisplayName failed %08x\n", hres); ok(!lstrcmpW(display_name, urls[test_protocol]), "GetDisplayName got wrong name\n"); - SET_EXPECT(QueryInterface_IServiceProvider); SET_EXPECT(GetBindInfo); + SET_EXPECT(QueryInterface_IInternetProtocol); + if(!emulate_protocol) + SET_EXPECT(QueryService_IInternetProtocol); SET_EXPECT(OnStartBinding); if(emulate_protocol) { SET_EXPECT(Start); @@ -1693,7 +1685,7 @@ static void test_BindToObject(int protocol, BOOL emul) /* no point testing the calls if binding didn't even work */ if (!SUCCEEDED(hres)) return; - if((bindf & BINDF_ASYNCHRONOUS) && !data_available) { + if((bindf & BINDF_ASYNCHRONOUS)) { ok(hres == MK_S_ASYNCHRONOUS, "IMoniker_BindToStorage failed: %08x\n", hres); ok(unk == NULL, "istr should be NULL\n"); }else { @@ -1709,8 +1701,10 @@ static void test_BindToObject(int protocol, BOOL emul) DispatchMessage(&msg); } - todo_wine CHECK_NOT_CALLED(QueryInterface_IServiceProvider); CHECK_CALLED(GetBindInfo); + CHECK_CALLED(QueryInterface_IInternetProtocol); + if(!emulate_protocol) + CHECK_CALLED(QueryService_IInternetProtocol); CHECK_CALLED(OnStartBinding); if(emulate_protocol) { CHECK_CALLED(Start); @@ -1777,7 +1771,7 @@ static void create_file(void) file = CreateFileW(wszIndexHtml, GENERIC_WRITE, 0, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL); - ok(file != INVALID_HANDLE_VALUE, "CreateFile failed\n"); + ok(file != INVALID_HANDLE_VALUE, "CreateFile failed: %u\n", GetLastError()); if(file == INVALID_HANDLE_VALUE) return;

1 0

Jacek Caban : urlmon: Added more binding tests.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: 1148d4f13f0fced1f924f6452c2e15a0dd551d9c URL: http://source.winehq.org/git/wine.git/?a=commit;h=1148d4f13f0fced1f924f6452… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 12:43:35 2007 +0100 urlmon: Added more binding tests. --- dlls/urlmon/tests/protocol.c | 410 ++++++++++++++++++++++++++++++++++++++---- 1 files changed, 371 insertions(+), 39 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=1148d4f13f0fced1f924f…

1 0

Jacek Caban : urlmon: Added more BindProtocol:: ReportProgress implementation.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: 19bca907101aa3e378fdae1131fde02df97bf05c URL: http://source.winehq.org/git/wine.git/?a=commit;h=19bca907101aa3e378fdae113… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 12:43:12 2007 +0100 urlmon: Added more BindProtocol::ReportProgress implementation. --- dlls/urlmon/bindprot.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/urlmon/bindprot.c b/dlls/urlmon/bindprot.c index 0eb7274..a41575c 100644 --- a/dlls/urlmon/bindprot.c +++ b/dlls/urlmon/bindprot.c @@ -436,13 +436,14 @@ static HRESULT WINAPI InternetProtocolSink_ReportProgress(IInternetProtocolSink TRACE("(%p)->(%u %s)\n", This, ulStatusCode, debugstr_w(szStatusText)); switch(ulStatusCode) { + case BINDSTATUS_FINDINGRESOURCE: + case BINDSTATUS_CONNECTING: case BINDSTATUS_SENDINGREQUEST: - return IInternetProtocolSink_ReportProgress(This->protocol_sink, - ulStatusCode, NULL); case BINDSTATUS_CACHEFILENAMEAVAILABLE: return IInternetProtocolSink_ReportProgress(This->protocol_sink, ulStatusCode, szStatusText); case BINDSTATUS_VERIFIEDMIMETYPEAVAILABLE: + case BINDSTATUS_MIMETYPEAVAILABLE: return IInternetProtocolSink_ReportProgress(This->protocol_sink, BINDSTATUS_MIMETYPEAVAILABLE, szStatusText); default:

1 0

Jacek Caban : urlmon: Added BindProtocol::[Un]LockRequest.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: 4883c8aba620cbed1540ad80e610fabf4233593e URL: http://source.winehq.org/git/wine.git/?a=commit;h=4883c8aba620cbed1540ad80e… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 12:43:01 2007 +0100 urlmon: Added BindProtocol::[Un]LockRequest. --- dlls/urlmon/bindprot.c | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/dlls/urlmon/bindprot.c b/dlls/urlmon/bindprot.c index 9185718..0eb7274 100644 --- a/dlls/urlmon/bindprot.c +++ b/dlls/urlmon/bindprot.c @@ -253,15 +253,19 @@ static HRESULT WINAPI BindProtocol_Seek(IInternetProtocol *iface, LARGE_INTEGER static HRESULT WINAPI BindProtocol_LockRequest(IInternetProtocol *iface, DWORD dwOptions) { BindProtocol *This = PROTOCOL_THIS(iface); - FIXME("(%p)->(%08x)\n", This, dwOptions); - return E_NOTIMPL; + + TRACE("(%p)->(%08x)\n", This, dwOptions); + + return IInternetProtocol_LockRequest(This->protocol, dwOptions); } static HRESULT WINAPI BindProtocol_UnlockRequest(IInternetProtocol *iface) { BindProtocol *This = PROTOCOL_THIS(iface); - FIXME("(%p)\n", This); - return E_NOTIMPL; + + TRACE("(%p)\n", This); + + return IInternetProtocol_UnlockRequest(This->protocol); } #undef PROTOCOL_THIS

1 0

Jacek Caban : urlmon: Added BindProtocol::Continue implementation.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: f189e7a060a7a8d64d28ce5251e3a48b414aa8ca URL: http://source.winehq.org/git/wine.git/?a=commit;h=f189e7a060a7a8d64d28ce525… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 12:42:45 2007 +0100 urlmon: Added BindProtocol::Continue implementation. --- dlls/urlmon/bindprot.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/urlmon/bindprot.c b/dlls/urlmon/bindprot.c index 8e63673..9185718 100644 --- a/dlls/urlmon/bindprot.c +++ b/dlls/urlmon/bindprot.c @@ -189,8 +189,10 @@ static HRESULT WINAPI BindProtocol_Start(IInternetProtocol *iface, LPCWSTR szUrl static HRESULT WINAPI BindProtocol_Continue(IInternetProtocol *iface, PROTOCOLDATA *pProtocolData) { BindProtocol *This = PROTOCOL_THIS(iface); - FIXME("(%p)->(%p)\n", This, pProtocolData); - return E_NOTIMPL; + + TRACE("(%p)->(%p)\n", This, pProtocolData); + + return IInternetProtocol_Continue(This->protocol, pProtocolData); } static HRESULT WINAPI BindProtocol_Abort(IInternetProtocol *iface, HRESULT hrReason,

1 0

Jacek Caban : urlmon: Added BindProtocol::Switch implementation.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: a6eb0780be69dfb373ba05cbd3e83e26cd5429a1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a6eb0780be69dfb373ba05cbd… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 12:42:32 2007 +0100 urlmon: Added BindProtocol::Switch implementation. --- dlls/urlmon/bindprot.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/urlmon/bindprot.c b/dlls/urlmon/bindprot.c index b8f3a8b..8e63673 100644 --- a/dlls/urlmon/bindprot.c +++ b/dlls/urlmon/bindprot.c @@ -416,8 +416,10 @@ static HRESULT WINAPI InternetProtocolSink_Switch(IInternetProtocolSink *iface, PROTOCOLDATA *pProtocolData) { BindProtocol *This = PROTSINK_THIS(iface); - FIXME("(%p)->(%p)\n", This, pProtocolData); - return E_NOTIMPL; + + TRACE("(%p)->(%p)\n", This, pProtocolData); + + return IInternetProtocolSink_Switch(This->protocol_sink, pProtocolData); } static HRESULT WINAPI InternetProtocolSink_ReportProgress(IInternetProtocolSink *iface,

1 0

Jacek Caban : urlmon: Added IServiceProvider implementation to BindProtocol object.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: a9765337f8b8c66a898d9cfe75f8fdac8df1a541 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a9765337f8b8c66a898d9cfe7… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 12:42:22 2007 +0100 urlmon: Added IServiceProvider implementation to BindProtocol object. --- dlls/urlmon/bindprot.c | 53 +++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 52 insertions(+), 1 deletions(-) diff --git a/dlls/urlmon/bindprot.c b/dlls/urlmon/bindprot.c index 1312a84..b8f3a8b 100644 --- a/dlls/urlmon/bindprot.c +++ b/dlls/urlmon/bindprot.c @@ -25,6 +25,7 @@ typedef struct { const IInternetProtocolVtbl *lpInternetProtocolVtbl; const IInternetBindInfoVtbl *lpInternetBindInfoVtbl; const IInternetPriorityVtbl *lpInternetPriorityVtbl; + const IServiceProviderVtbl *lpServiceProviderVtbl; const IInternetProtocolSinkVtbl *lpInternetProtocolSinkVtbl; LONG ref; @@ -32,6 +33,7 @@ typedef struct { IInternetProtocol *protocol; IInternetBindInfo *bind_info; IInternetProtocolSink *protocol_sink; + IServiceProvider *service_provider; LONG priority; } BindProtocol; @@ -39,6 +41,7 @@ typedef struct { #define PROTOCOL(x) ((IInternetProtocol*) &(x)->lpInternetProtocolVtbl) #define BINDINFO(x) ((IInternetBindInfo*) &(x)->lpInternetBindInfoVtbl) #define PRIORITY(x) ((IInternetPriority*) &(x)->lpInternetPriorityVtbl) +#define SERVPROV(x) ((IServiceProvider*) &(x)->lpServiceProviderVtbl) #define PROTSINK(x) ((IInternetProtocolSink*) &(x)->lpInternetProtocolSinkVtbl) #define PROTOCOL_THIS(iface) DEFINE_THIS(BindProtocol, InternetProtocol, iface) @@ -66,7 +69,8 @@ static HRESULT WINAPI BindProtocol_QueryInterface(IInternetProtocol *iface, REFI }else if(IsEqualGUID(&IID_IAuthenticate, riid)) { FIXME("(%p)->(IID_IAuthenticate %p)\n", This, ppv); }else if(IsEqualGUID(&IID_IServiceProvider, riid)) { - FIXME("(%p)->(IID_IServiceProvider %p)\n", This, ppv); + TRACE("(%p)->(IID_IServiceProvider %p)\n", This, ppv); + *ppv = SERVPROV(This); }else if(IsEqualGUID(&IID_IInternetProtocolSink, riid)) { TRACE("(%p)->(IID_IInternetProtocolSink %p)\n", This, ppv); *ppv = PROTSINK(This); @@ -171,6 +175,8 @@ static HRESULT WINAPI BindProtocol_Start(IInternetProtocol *iface, LPCWSTR szUrl IInternetProtocolSink_AddRef(pOIProtSink); This->protocol_sink = pOIProtSink; + IInternetProtocolSink_QueryInterface(pOIProtSink, &IID_IServiceProvider, (void**)&This->service_provider); + hres = IInternetProtocol_QueryInterface(protocol, &IID_IInternetPriority, (void**)&priority); if(SUCCEEDED(hres)) { IInternetPriority_SetPriority(priority, This->priority); @@ -470,6 +476,49 @@ static const IInternetProtocolSinkVtbl InternetProtocolSinkVtbl = { InternetProtocolSink_ReportResult }; +#define SERVPROV_THIS(iface) DEFINE_THIS(BindProtocol, ServiceProvider, iface) + +static HRESULT WINAPI BPServiceProvider_QueryInterface(IServiceProvider *iface, + REFIID riid, void **ppv) +{ + BindProtocol *This = SERVPROV_THIS(iface); + return IInternetProtocol_QueryInterface(PROTOCOL(This), riid, ppv); +} + +static ULONG WINAPI BPServiceProvider_AddRef(IServiceProvider *iface) +{ + BindProtocol *This = SERVPROV_THIS(iface); + return IInternetProtocol_AddRef(PROTOCOL(This)); +} + +static ULONG WINAPI BPServiceProvider_Release(IServiceProvider *iface) +{ + BindProtocol *This = SERVPROV_THIS(iface); + return IInternetProtocol_Release(PROTOCOL(This)); +} + +static HRESULT WINAPI BPServiceProvider_QueryService(IServiceProvider *iface, + REFGUID guidService, REFIID riid, void **ppv) +{ + BindProtocol *This = SERVPROV_THIS(iface); + + TRACE("(%p)->(%s %s %p)\n", This, debugstr_guid(guidService), debugstr_guid(riid), ppv); + + if(!This->service_provider) + return E_NOINTERFACE; + + return IServiceProvider_QueryService(This->service_provider, guidService, riid, ppv); +} + +#undef SERVPROV_THIS + +static const IServiceProviderVtbl ServiceProviderVtbl = { + BPServiceProvider_QueryInterface, + BPServiceProvider_AddRef, + BPServiceProvider_Release, + BPServiceProvider_QueryService +}; + HRESULT create_binding_protocol(LPCWSTR url, IInternetProtocol **protocol) { BindProtocol *ret = heap_alloc(sizeof(BindProtocol)); @@ -477,12 +526,14 @@ HRESULT create_binding_protocol(LPCWSTR url, IInternetProtocol **protocol) ret->lpInternetProtocolVtbl = &BindProtocolVtbl; ret->lpInternetBindInfoVtbl = &InternetBindInfoVtbl; ret->lpInternetPriorityVtbl = &InternetPriorityVtbl; + ret->lpServiceProviderVtbl = &ServiceProviderVtbl; ret->lpInternetProtocolSinkVtbl = &InternetProtocolSinkVtbl; ret->ref = 1; ret->protocol = NULL; ret->bind_info = NULL; ret->protocol_sink = NULL; + ret->service_provider = NULL; ret->priority = 0; URLMON_LockModule();

1 0

Jacek Caban : urlmon: Added BindProtocol::GetBindString implementation.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: 7cd13e17adcda9ea6131399e172a4e003d807855 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7cd13e17adcda9ea6131399e1… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 12:42:09 2007 +0100 urlmon: Added BindProtocol::GetBindString implementation. --- dlls/urlmon/bindprot.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/urlmon/bindprot.c b/dlls/urlmon/bindprot.c index c2033fc..1312a84 100644 --- a/dlls/urlmon/bindprot.c +++ b/dlls/urlmon/bindprot.c @@ -317,8 +317,10 @@ static HRESULT WINAPI BindInfo_GetBindString(IInternetBindInfo *iface, ULONG ulStringType, LPOLESTR *ppwzStr, ULONG cEl, ULONG *pcElFetched) { BindProtocol *This = BINDINFO_THIS(iface); - FIXME("(%p)->(%d %p %d %p)\n", This, ulStringType, ppwzStr, cEl, pcElFetched); - return E_NOTIMPL; + + TRACE("(%p)->(%d %p %d %p)\n", This, ulStringType, ppwzStr, cEl, pcElFetched); + + return IInternetBindInfo_GetBindString(This->bind_info, ulStringType, ppwzStr, cEl, pcElFetched); } #undef BINDFO_THIS

1 0

Jacek Caban : urlmon: Fixed protocol tests.
by Alexandre Julliard 24 Dec '07

24 Dec '07

Module: wine Branch: master Commit: 0d869d539fe67166f27e42ba688f59742bcd1529 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0d869d539fe67166f27e42ba6… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 24 12:41:57 2007 +0100 urlmon: Fixed protocol tests. --- dlls/urlmon/bindprot.c | 2 +- dlls/urlmon/tests/protocol.c | 7 +++++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/dlls/urlmon/bindprot.c b/dlls/urlmon/bindprot.c index 13a3b0a..c2033fc 100644 --- a/dlls/urlmon/bindprot.c +++ b/dlls/urlmon/bindprot.c @@ -443,7 +443,7 @@ static HRESULT WINAPI InternetProtocolSink_ReportData(IInternetProtocolSink *ifa TRACE("(%p)->(%d %u %u)\n", This, grfBSCF, ulProgress, ulProgressMax); - return S_OK; + return IInternetProtocolSink_ReportData(This->protocol_sink, grfBSCF, ulProgress, ulProgressMax); } static HRESULT WINAPI InternetProtocolSink_ReportResult(IInternetProtocolSink *iface, diff --git a/dlls/urlmon/tests/protocol.c b/dlls/urlmon/tests/protocol.c index 00335b3..8c85ad5 100644 --- a/dlls/urlmon/tests/protocol.c +++ b/dlls/urlmon/tests/protocol.c @@ -393,8 +393,6 @@ static HRESULT WINAPI ProtocolSink_ReportProgress(IInternetProtocolSink *iface, ok(szStatusText != NULL, "szStatusText == NULL\n"); if(szStatusText) ok(!*szStatusText, "wrong szStatusText\n"); - }else { - ok(szStatusText == NULL, "szStatusText != NULL\n"); } break; case BINDSTATUS_VERIFIEDMIMETYPEAVAILABLE: @@ -460,7 +458,10 @@ static HRESULT WINAPI ProtocolSink_ReportData(IInternetProtocolSink *iface, DWOR } SetEvent(event_complete); } + }else { + CHECK_EXPECT(ReportData); } + return S_OK; } @@ -736,9 +737,11 @@ static HRESULT WINAPI Protocol_Start(IInternetProtocol *iface, LPCWSTR szUrl, "ReportProgress(BINDSTATUS_VERIFIEDMIMETYPEAVAILABLE) failed: %08x\n", hres); CHECK_CALLED(ReportProgress_MIMETYPEAVAILABLE); + SET_EXPECT(ReportData); hres = IInternetProtocolSink_ReportData(pOIProtSink, BSCF_FIRSTDATANOTIFICATION | BSCF_LASTDATANOTIFICATION, 13, 13); ok(hres == S_OK, "ReportData failed: %08x\n", hres); + CHECK_CALLED(ReportData); SET_EXPECT(ReportResult); hres = IInternetProtocolSink_ReportResult(pOIProtSink, S_OK, 0, NULL);

1 0

← Newer
1
...
11
12
13
14
15
16
17
...
83
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits December 2007