wine-commits November 2010

wine-commits@winehq.org

2 participants
866 discussions

David Hedberg : shdocvw: Implement ControlSite::TranslateAccelerator.
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: 0adfeb28696aacf426977c578012bfa84e257cbc URL: http://source.winehq.org/git/wine.git/?a=commit;h=0adfeb28696aacf426977c578… Author: David Hedberg <dhedberg(a)codeweavers.com> Date: Wed Nov 17 13:12:21 2010 +0100 shdocvw: Implement ControlSite::TranslateAccelerator. --- dlls/shdocvw/client.c | 27 ++++++++- dlls/shdocvw/tests/webbrowser.c | 125 ++++++++++++++++++++++++++++++++++++++- 2 files changed, 149 insertions(+), 3 deletions(-) diff --git a/dlls/shdocvw/client.c b/dlls/shdocvw/client.c index bb569aa..5c5da44 100644 --- a/dlls/shdocvw/client.c +++ b/dlls/shdocvw/client.c @@ -415,8 +415,31 @@ static HRESULT WINAPI ControlSite_TranslateAccelerator(IOleControlSite *iface, M DWORD grfModifiers) { DocHost *This = impl_from_IOleControlSite(iface); - FIXME("(%p)->(%p, %08x)\n", This, pMsg, grfModifiers); - return E_NOTIMPL; + IOleObject *wb_obj; + IOleClientSite *clientsite; + IOleControlSite *controlsite; + HRESULT hr; + + TRACE("(%p)->(%p, %08x)\n", This, pMsg, grfModifiers); + + hr = IDispatch_QueryInterface(This->disp, &IID_IOleObject, (void**)&wb_obj); + if(SUCCEEDED(hr)) { + hr = IOleObject_GetClientSite(wb_obj, &clientsite); + if(SUCCEEDED(hr)) { + hr = IOleClientSite_QueryInterface(clientsite, &IID_IOleControlSite, (void**)&controlsite); + if(SUCCEEDED(hr)) { + hr = IOleControlSite_TranslateAccelerator(controlsite, pMsg, grfModifiers); + IOleControlSite_Release(controlsite); + } + IOleClientSite_Release(clientsite); + } + IOleObject_Release(wb_obj); + } + + if(FAILED(hr)) + return S_FALSE; + else + return hr; } static HRESULT WINAPI ControlSite_OnFocus(IOleControlSite *iface, BOOL fGotFocus) diff --git a/dlls/shdocvw/tests/webbrowser.c b/dlls/shdocvw/tests/webbrowser.c index 7f9a24a..6f84eee 100644 --- a/dlls/shdocvw/tests/webbrowser.c +++ b/dlls/shdocvw/tests/webbrowser.c @@ -130,6 +130,8 @@ DEFINE_EXPECT(TranslateUrl); DEFINE_EXPECT(ShowUI); DEFINE_EXPECT(HideUI); DEFINE_EXPECT(RequestUIActivate); +DEFINE_EXPECT(ControlSite_TranslateAccelerator); +DEFINE_EXPECT(OnFocus); static const WCHAR wszItem[] = {'i','t','e','m',0}; static const WCHAR emptyW[] = {0}; @@ -140,6 +142,7 @@ static IWebBrowser2 *wb; static HWND container_hwnd, shell_embedding_hwnd; static BOOL is_downloading, is_first_load; static HRESULT hr_dochost_TranslateAccelerator = E_NOTIMPL; +static HRESULT hr_site_TranslateAccelerator = E_NOTIMPL; static const char *current_url; #define DWL_EXPECT_BEFORE_NAVIGATE 0x01 @@ -852,6 +855,85 @@ static const IOleClientSiteVtbl ClientSiteVtbl = { static IOleClientSite ClientSite = { &ClientSiteVtbl }; +static HRESULT WINAPI IOleControlSite_fnQueryInterface(IOleControlSite *iface, REFIID riid, void **ppv) +{ + *ppv = NULL; + return E_NOINTERFACE; +} + +static ULONG WINAPI IOleControlSite_fnAddRef(IOleControlSite *iface) +{ + return 2; +} + +static ULONG WINAPI IOleControlSite_fnRelease(IOleControlSite *iface) +{ + return 1; +} + +static HRESULT WINAPI IOleControlSite_fnOnControlInfoChanged(IOleControlSite* This) +{ + ok(0, "unexpected call\n"); + return E_NOTIMPL; +} + +static HRESULT WINAPI IOleControlSite_fnLockInPlaceActive(IOleControlSite* This, + BOOL fLock) +{ + ok(0, "unexpected call\n"); + return E_NOTIMPL; +} + +static HRESULT WINAPI IOleControlSite_fnGetExtendedControl(IOleControlSite* This, + IDispatch **ppDisp) +{ + ok(0, "unexpected call\n"); + return E_NOTIMPL; +} + +static HRESULT WINAPI IOleControlSite_fnTransformCoords(IOleControlSite* This, + POINTL *pPtlHimetric, + POINTF *pPtfContainer, + DWORD dwFlags) +{ + ok(0, "unexpected call\n"); + return E_NOTIMPL; +} + +static HRESULT WINAPI IOleControlSite_fnTranslateAccelerator(IOleControlSite* This, MSG *pMsg, + DWORD grfModifiers) +{ + CHECK_EXPECT(ControlSite_TranslateAccelerator); + return hr_site_TranslateAccelerator; +} + +static HRESULT WINAPI IOleControlSite_fnOnFocus(IOleControlSite* This, BOOL fGotFocus) +{ + CHECK_EXPECT2(OnFocus); + return E_NOTIMPL; +} + +static HRESULT WINAPI IOleControlSite_fnShowPropertyFrame(IOleControlSite* This) +{ + ok(0, "unexpected call\n"); + return E_NOTIMPL; +} + +static IOleControlSiteVtbl ControlSiteVtbl = { + IOleControlSite_fnQueryInterface, + IOleControlSite_fnAddRef, + IOleControlSite_fnRelease, + IOleControlSite_fnOnControlInfoChanged, + IOleControlSite_fnLockInPlaceActive, + IOleControlSite_fnGetExtendedControl, + IOleControlSite_fnTransformCoords, + IOleControlSite_fnTranslateAccelerator, + IOleControlSite_fnOnFocus, + IOleControlSite_fnShowPropertyFrame +}; + +static IOleControlSite ControlSite = { &ControlSiteVtbl }; + static HRESULT WINAPI InPlaceUIWindow_QueryInterface(IOleInPlaceFrame *iface, REFIID riid, void **ppv) { @@ -1433,7 +1515,7 @@ static HRESULT QueryInterface(REFIID riid, void **ppv) else if(IsEqualGUID(&IID_IDocHostShowUI, riid)) trace("interface IID_IDocHostShowUI\n"); else if(IsEqualGUID(&IID_IOleControlSite, riid)) - trace("interface IID_IOleControlSite\n"); + *ppv = &ControlSite; else if(IsEqualGUID(&IID_IOleCommandTarget, riid)) trace("interface IID_IOleCommandTarget\n"); @@ -1494,6 +1576,7 @@ static void test_DoVerb(IUnknown *unk) SET_EXPECT(Frame_SetActiveObject); SET_EXPECT(UIWindow_SetActiveObject); SET_EXPECT(SetMenu); + SET_EXPECT(OnFocus); hres = IOleObject_DoVerb(oleobj, OLEIVERB_SHOW, NULL, &ClientSite, 0, (HWND)0xdeadbeef, &rect); @@ -1510,6 +1593,7 @@ static void test_DoVerb(IUnknown *unk) CHECK_CALLED(Frame_SetActiveObject); CHECK_CALLED(UIWindow_SetActiveObject); CHECK_CALLED(SetMenu); + todo_wine CHECK_CALLED(OnFocus); hres = IOleObject_DoVerb(oleobj, OLEIVERB_SHOW, NULL, &ClientSite, 0, (HWND)0xdeadbeef, &rect); @@ -2539,6 +2623,7 @@ static void test_TranslateAccelerator(IUnknown *unk) ok(hres == S_OK, "Got 0x%08x\n", hres); if(SUCCEEDED(hres)) { IDocHostUIHandler2 *dochost; + IOleControlSite *doc_controlsite; IUnknown *unk_test; hres = IOleClientSite_QueryInterface(doc_clientsite, &IID_IOleInPlaceFrame, (void**)&unk_test); @@ -2575,6 +2660,44 @@ static void test_TranslateAccelerator(IUnknown *unk) IDocHostUIHandler2_Release(dochost); } + hres = IOleClientSite_QueryInterface(doc_clientsite, &IID_IOleControlSite, (void**)&doc_controlsite); + ok(hres == S_OK, "Got 0x%08x\n", hres); + if(SUCCEEDED(hres)) { + + msg_a.message = WM_KEYDOWN; + hr_site_TranslateAccelerator = 0xdeadbeef; + for(keycode = 0; keycode < 0x100; keycode++) { + msg_a.wParam = keycode; + SET_EXPECT(ControlSite_TranslateAccelerator); + hres = IOleControlSite_TranslateAccelerator(doc_controlsite, &msg_a, 0); + if(keycode == 0x9 || keycode == 0x75) + todo_wine ok(hres == S_OK, "Got 0x%08x (keycode: %04x)\n", hres, keycode); + else + ok(hres == S_FALSE, "Got 0x%08x (keycode: %04x)\n", hres, keycode); + + CHECK_CALLED(ControlSite_TranslateAccelerator); + } + msg_a.wParam = VK_LEFT; + SET_EXPECT(ControlSite_TranslateAccelerator); + hres = IOleControlSite_TranslateAccelerator(doc_controlsite, &msg_a, 0); + ok(hres == S_FALSE, "Got 0x%08x (keycode: %04x)\n", hres, keycode); + CHECK_CALLED(ControlSite_TranslateAccelerator); + + hr_site_TranslateAccelerator = S_OK; + SET_EXPECT(ControlSite_TranslateAccelerator); + hres = IOleControlSite_TranslateAccelerator(doc_controlsite, &msg_a, 0); + ok(hres == S_OK, "Got 0x%08x (keycode: %04x)\n", hres, keycode); + CHECK_CALLED(ControlSite_TranslateAccelerator); + + hr_site_TranslateAccelerator = E_NOTIMPL; + SET_EXPECT(ControlSite_TranslateAccelerator); + hres = IOleControlSite_TranslateAccelerator(doc_controlsite, &msg_a, 0); + ok(hres == S_FALSE, "Got 0x%08x (keycode: %04x)\n", hres, keycode); + CHECK_CALLED(ControlSite_TranslateAccelerator); + + IOleControlSite_Release(doc_controlsite); + } + IOleClientSite_Release(doc_clientsite); } IOleObject_Release(obj_doc);

1 0

David Hedberg : shdocvw: Add IOleControlSite interface to dochost.
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: 42ef5a0fce16e003e9679e16771873a19c100ec3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=42ef5a0fce16e003e9679e167… Author: David Hedberg <dhedberg(a)codeweavers.com> Date: Wed Nov 17 13:12:20 2010 +0100 shdocvw: Add IOleControlSite interface to dochost. --- dlls/shdocvw/client.c | 91 ++++++++++++++++++++++++++++++++++++++++++++++++ dlls/shdocvw/shdocvw.h | 1 + 2 files changed, 92 insertions(+), 0 deletions(-) diff --git a/dlls/shdocvw/client.c b/dlls/shdocvw/client.c index dfb6d34..bb569aa 100644 --- a/dlls/shdocvw/client.c +++ b/dlls/shdocvw/client.c @@ -55,6 +55,9 @@ static HRESULT WINAPI ClientSite_QueryInterface(IOleClientSite *iface, REFIID ri }else if(IsEqualGUID(&IID_IOleDocumentSite, riid)) { TRACE("(%p)->(IID_IOleDocumentSite %p)\n", This, ppv); *ppv = &This->IOleDocumentSite_iface; + }else if(IsEqualGUID(&IID_IOleControlSite, riid)) { + TRACE("(%p)->(IID_IOleControlSite %p)\n", This, ppv); + *ppv = &This->IOleControlSite_iface; }else if(IsEqualGUID(&IID_IOleCommandTarget, riid)) { TRACE("(%p)->(IID_IOleCommandTarget %p)\n", This, ppv); *ppv = &This->IOleCommandTarget_iface; @@ -356,6 +359,93 @@ static const IOleDocumentSiteVtbl OleDocumentSiteVtbl = { OleDocumentSite_ActivateMe }; +static inline DocHost *impl_from_IOleControlSite(IOleControlSite *iface) +{ + return (DocHost*)((char*)iface - FIELD_OFFSET(DocHost, IOleControlSite_iface)); +} + +static HRESULT WINAPI ControlSite_QueryInterface(IOleControlSite *iface, REFIID riid, void **ppv) +{ + DocHost *This = impl_from_IOleControlSite(iface); + return IOleClientSite_QueryInterface(&This->IOleClientSite_iface, riid, ppv); +} + +static ULONG WINAPI ControlSite_AddRef(IOleControlSite *iface) +{ + DocHost *This = impl_from_IOleControlSite(iface); + return IOleClientSite_AddRef(&This->IOleClientSite_iface); +} + +static ULONG WINAPI ControlSite_Release(IOleControlSite *iface) +{ + DocHost *This = impl_from_IOleControlSite(iface); + return IOleClientSite_Release(&This->IOleClientSite_iface); +} + +static HRESULT WINAPI ControlSite_OnControlInfoChanged(IOleControlSite *iface) +{ + DocHost *This = impl_from_IOleControlSite(iface); + FIXME("(%p)\n", This); + return E_NOTIMPL; +} + +static HRESULT WINAPI ControlSite_LockInPlaceActive(IOleControlSite *iface, BOOL fLock) +{ + DocHost *This = impl_from_IOleControlSite(iface); + FIXME("(%p)->(%d)\n", This, fLock); + return E_NOTIMPL; +} + +static HRESULT WINAPI ControlSite_GetExtendedControl(IOleControlSite *iface, IDispatch **ppDisp) +{ + DocHost *This = impl_from_IOleControlSite(iface); + FIXME("(%p)->(%p)\n", This, ppDisp); + return E_NOTIMPL; +} + +static HRESULT WINAPI ControlSite_TransformCoords(IOleControlSite *iface, POINTL *pPtlHimetric, + POINTF *pPtfContainer, DWORD dwFlags) +{ + DocHost *This = impl_from_IOleControlSite(iface); + FIXME("(%p)->(%p, %p, %08x)\n", This, pPtlHimetric, pPtfContainer, dwFlags); + return E_NOTIMPL; +} + +static HRESULT WINAPI ControlSite_TranslateAccelerator(IOleControlSite *iface, MSG *pMsg, + DWORD grfModifiers) +{ + DocHost *This = impl_from_IOleControlSite(iface); + FIXME("(%p)->(%p, %08x)\n", This, pMsg, grfModifiers); + return E_NOTIMPL; +} + +static HRESULT WINAPI ControlSite_OnFocus(IOleControlSite *iface, BOOL fGotFocus) +{ + DocHost *This = impl_from_IOleControlSite(iface); + FIXME("(%p)->(%d)\n", This, fGotFocus); + return E_NOTIMPL; +} + +static HRESULT WINAPI ControlSite_ShowPropertyFrame(IOleControlSite *iface) +{ + DocHost *This = impl_from_IOleControlSite(iface); + FIXME("(%p)\n", This); + return E_NOTIMPL; +} + +static IOleControlSiteVtbl OleControlSiteVtbl = { + ControlSite_QueryInterface, + ControlSite_AddRef, + ControlSite_Release, + ControlSite_OnControlInfoChanged, + ControlSite_LockInPlaceActive, + ControlSite_GetExtendedControl, + ControlSite_TransformCoords, + ControlSite_TranslateAccelerator, + ControlSite_OnFocus, + ControlSite_ShowPropertyFrame +}; + static inline DocHost *impl_from_IDispatch(IDispatch *iface) { return (DocHost*)((char*)iface - FIELD_OFFSET(DocHost, IDispatch_iface)); @@ -521,6 +611,7 @@ void DocHost_ClientSite_Init(DocHost *This) This->IOleClientSite_iface.lpVtbl = &OleClientSiteVtbl; This->IOleInPlaceSite_iface.lpVtbl = &OleInPlaceSiteVtbl; This->IOleDocumentSite_iface.lpVtbl = &OleDocumentSiteVtbl; + This->IOleControlSite_iface.lpVtbl = &OleControlSiteVtbl; This->IDispatch_iface.lpVtbl = &DispatchVtbl; This->IServiceProvider_iface.lpVtbl = &ServiceProviderVtbl; } diff --git a/dlls/shdocvw/shdocvw.h b/dlls/shdocvw/shdocvw.h index bd6152d..6df8527 100644 --- a/dlls/shdocvw/shdocvw.h +++ b/dlls/shdocvw/shdocvw.h @@ -97,6 +97,7 @@ struct DocHost { IOleInPlaceSite IOleInPlaceSite_iface; IDocHostUIHandler2 IDocHostUIHandler2_iface; IOleDocumentSite IOleDocumentSite_iface; + IOleControlSite IOleControlSite_iface; IOleCommandTarget IOleCommandTarget_iface; IDispatch IDispatch_iface; IPropertyNotifySink IPropertyNotifySink_iface;

1 0

David Hedberg : shdocvw: Implement DocHostUIHandler::TranslateAccelerator.
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: 910cac49a464fea8633488511f033b7adcd6ef70 URL: http://source.winehq.org/git/wine.git/?a=commit;h=910cac49a464fea8633488511… Author: David Hedberg <dhedberg(a)codeweavers.com> Date: Wed Nov 17 13:12:19 2010 +0100 shdocvw: Implement DocHostUIHandler::TranslateAccelerator. --- dlls/shdocvw/dochost.c | 9 +++- dlls/shdocvw/tests/webbrowser.c | 89 ++++++++++++++++++++++++++++++++++++-- 2 files changed, 91 insertions(+), 7 deletions(-) diff --git a/dlls/shdocvw/dochost.c b/dlls/shdocvw/dochost.c index e7c8c0f..31ecce3 100644 --- a/dlls/shdocvw/dochost.c +++ b/dlls/shdocvw/dochost.c @@ -620,8 +620,13 @@ static HRESULT WINAPI DocHostUIHandler_TranslateAccelerator(IDocHostUIHandler2 * LPMSG lpMsg, const GUID *pguidCmdGroup, DWORD nCmdID) { DocHost *This = impl_from_IDocHostUIHandler2(iface); - FIXME("(%p)->(%p %p %d)\n", This, lpMsg, pguidCmdGroup, nCmdID); - return E_NOTIMPL; + HRESULT hr = S_FALSE; + TRACE("(%p)->(%p %p %d)\n", This, lpMsg, pguidCmdGroup, nCmdID); + + if(This->hostui) + hr = IDocHostUIHandler_TranslateAccelerator(This->hostui, lpMsg, pguidCmdGroup, nCmdID); + + return hr; } static HRESULT WINAPI DocHostUIHandler_GetOptionKeyPath(IDocHostUIHandler2 *iface, diff --git a/dlls/shdocvw/tests/webbrowser.c b/dlls/shdocvw/tests/webbrowser.c index 1b1afc1..7f9a24a 100644 --- a/dlls/shdocvw/tests/webbrowser.c +++ b/dlls/shdocvw/tests/webbrowser.c @@ -124,8 +124,12 @@ DEFINE_EXPECT(QueryStatus_SETPROGRESSTEXT); DEFINE_EXPECT(QueryStatus_STOP); DEFINE_EXPECT(DocHost_EnableModeless_TRUE); DEFINE_EXPECT(DocHost_EnableModeless_FALSE); +DEFINE_EXPECT(DocHost_TranslateAccelerator); DEFINE_EXPECT(GetDropTarget); DEFINE_EXPECT(TranslateUrl); +DEFINE_EXPECT(ShowUI); +DEFINE_EXPECT(HideUI); +DEFINE_EXPECT(RequestUIActivate); static const WCHAR wszItem[] = {'i','t','e','m',0}; static const WCHAR emptyW[] = {0}; @@ -135,6 +139,7 @@ static IWebBrowser2 *wb; static HWND container_hwnd, shell_embedding_hwnd; static BOOL is_downloading, is_first_load; +static HRESULT hr_dochost_TranslateAccelerator = E_NOTIMPL; static const char *current_url; #define DWL_EXPECT_BEFORE_NAVIGATE 0x01 @@ -1140,7 +1145,7 @@ static HRESULT WINAPI InPlaceSite_OnInPlaceDeactivateEx(IOleInPlaceSiteEx *iface static HRESULT WINAPI InPlaceSite_RequestUIActivate(IOleInPlaceSiteEx *iface) { - ok(0, "unexpected call\n"); + CHECK_EXPECT2(RequestUIActivate); return S_OK; } @@ -1207,13 +1212,13 @@ static HRESULT WINAPI DocHostUIHandler_ShowUI(IDocHostUIHandler2 *iface, DWORD d IOleInPlaceActiveObject *pActiveObject, IOleCommandTarget *pCommandTarget, IOleInPlaceFrame *pFrame, IOleInPlaceUIWindow *pDoc) { - ok(0, "unexpected call\n"); + CHECK_EXPECT(ShowUI); return E_NOTIMPL; } static HRESULT WINAPI DocHostUIHandler_HideUI(IDocHostUIHandler2 *iface) { - ok(0, "unexpected call\n"); + CHECK_EXPECT(HideUI); return E_NOTIMPL; } @@ -1255,8 +1260,12 @@ static HRESULT WINAPI DocHostUIHandler_ResizeBorder(IDocHostUIHandler2 *iface, L static HRESULT WINAPI DocHostUIHandler_TranslateAccelerator(IDocHostUIHandler2 *iface, LPMSG lpMsg, const GUID *pguidCmdGroup, DWORD nCmdID) { - ok(0, "unexpected call\n"); - return E_NOTIMPL; + CHECK_EXPECT(DocHost_TranslateAccelerator); + ok(pguidCmdGroup != NULL, "Got NULL pguidCmdGroup.\n"); + if(pguidCmdGroup) + ok(IsEqualGUID(pguidCmdGroup, &CGID_MSHTML), "Unexpected pguidCmdGroup\n"); + ok(lpMsg != NULL, "Got NULL lpMsg.\n"); + return hr_dochost_TranslateAccelerator; } static HRESULT WINAPI DocHostUIHandler_GetOptionKeyPath(IDocHostUIHandler2 *iface, @@ -2505,6 +2514,75 @@ static void test_QueryInterface(IUnknown *unk) } +static void test_TranslateAccelerator(IUnknown *unk) +{ + IOleObject *obj_doc; + IDispatch *disp_doc; + HRESULT hres; + DWORD keycode; + MSG msg_a = { + container_hwnd, + 0, 0, 0, + GetTickCount(), + {5, 5} + }; + + test_Navigate2(unk); + + disp_doc = get_document(unk); + hres = IDispatch_QueryInterface(disp_doc, &IID_IOleObject, (void**)&obj_doc); + ok(hres == S_OK, "Got 0x%08x\n", hres); + if(SUCCEEDED(hres)) { + IOleClientSite *doc_clientsite; + + hres = IOleObject_GetClientSite(obj_doc, &doc_clientsite); + ok(hres == S_OK, "Got 0x%08x\n", hres); + if(SUCCEEDED(hres)) { + IDocHostUIHandler2 *dochost; + IUnknown *unk_test; + + hres = IOleClientSite_QueryInterface(doc_clientsite, &IID_IOleInPlaceFrame, (void**)&unk_test); + ok(hres == E_NOINTERFACE, "Got 0x%08x\n", hres); + if(SUCCEEDED(hres)) IUnknown_Release(unk_test); + + hres = IOleClientSite_QueryInterface(doc_clientsite, &IID_IDocHostShowUI, (void**)&unk_test); + todo_wine ok(hres == S_OK, "Got 0x%08x\n", hres); + if(SUCCEEDED(hres)) IUnknown_Release(unk_test); + + hres = IOleClientSite_QueryInterface(doc_clientsite, &IID_IDocHostUIHandler, (void**)&unk_test); + ok(hres == S_OK, "Got 0x%08x\n", hres); + if(SUCCEEDED(hres)) IUnknown_Release(unk_test); + + hres = IOleClientSite_QueryInterface(doc_clientsite, &IID_IDocHostUIHandler2, (void**)&dochost); + ok(hres == S_OK, "Got 0x%08x\n", hres); + if(SUCCEEDED(hres)) { + + msg_a.message = WM_KEYDOWN; + hr_dochost_TranslateAccelerator = 0xdeadbeef; + for(keycode = 0; keycode <= 0x100; keycode++) { + msg_a.wParam = keycode; + SET_EXPECT(DocHost_TranslateAccelerator); + hres = IDocHostUIHandler_TranslateAccelerator(dochost, &msg_a, &CGID_MSHTML, 1234); + ok(hres == 0xdeadbeef, "Got 0x%08x\n", hres); + CHECK_CALLED(DocHost_TranslateAccelerator); + } + hr_dochost_TranslateAccelerator = E_NOTIMPL; + + SET_EXPECT(DocHost_TranslateAccelerator); + hres = IDocHostUIHandler_TranslateAccelerator(dochost, &msg_a, &CGID_MSHTML, 1234); + ok(hres == E_NOTIMPL, "Got 0x%08x\n", hres); + CHECK_CALLED(DocHost_TranslateAccelerator); + + IDocHostUIHandler2_Release(dochost); + } + IOleClientSite_Release(doc_clientsite); + } + IOleObject_Release(obj_doc); + } + + IDispatch_Release(disp_doc); +} + static void test_WebBrowser(BOOL do_download) { IUnknown *unk = NULL; @@ -2559,6 +2637,7 @@ static void test_WebBrowser(BOOL do_download) test_Navigate2(unk); test_download(DWL_EXPECT_BEFORE_NAVIGATE); + test_TranslateAccelerator(unk); doc2 = get_document(unk); ok(doc == doc2, "doc != doc2\n"); IDispatch_Release(doc2);

1 0

Maarten Lankhorst : winegstreamer: Destroy thread in GST_Connect.
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: a63c81d480749d183ea5c742f9cd5adb32fd700a URL: http://source.winehq.org/git/wine.git/?a=commit;h=a63c81d480749d183ea5c742f… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Wed Nov 17 13:27:22 2010 +0100 winegstreamer: Destroy thread in GST_Connect. --- dlls/winegstreamer/gstdemux.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/winegstreamer/gstdemux.c b/dlls/winegstreamer/gstdemux.c index a53355d..6c9aca0 100644 --- a/dlls/winegstreamer/gstdemux.c +++ b/dlls/winegstreamer/gstdemux.c @@ -977,6 +977,8 @@ static HRESULT GST_Connect(GSTInPin *pPin, IPin *pConnectPin, ALLOCATOR_PROPERTI *props = This->props; gst_element_set_state(This->gstfilter, GST_STATE_READY); gst_element_get_state(This->gstfilter, NULL, NULL, -1); + if (This->push_thread) + gst_pad_activate_push(This->my_src, 0); This->initial = 0; This->nextofs = This->nextpullofs = 0;

1 0

Maarten Lankhorst : winegstreamer: Fix reference leak.
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: f3481b0218a30fe49af245183fd89c7ebbefac40 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f3481b0218a30fe49af245183… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Wed Nov 17 13:25:51 2010 +0100 winegstreamer: Fix reference leak. --- dlls/winegstreamer/gstdemux.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/winegstreamer/gstdemux.c b/dlls/winegstreamer/gstdemux.c index 0af7014..a53355d 100644 --- a/dlls/winegstreamer/gstdemux.c +++ b/dlls/winegstreamer/gstdemux.c @@ -1556,6 +1556,7 @@ static HRESULT GST_RemoveOutputPins(GSTImpl *This) { gst_element_set_bus(This->gstfilter, NULL); gst_element_set_state(This->gstfilter, GST_STATE_NULL); gst_pad_unlink(This->my_src, This->their_sink); + gst_object_unref(This->my_src); This->my_src = This->their_sink = NULL; for (i = 0; i < This->cStreams; i++) {

1 0

Alexandre Julliard : oleaut32: Fix the overflow check in VarNumFromParseNum .
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: 4e94fd55676675a89ad46ca3d1fa3e8ef54b55b2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4e94fd55676675a89ad46ca3d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 17 13:42:50 2010 +0100 oleaut32: Fix the overflow check in VarNumFromParseNum. Spotted by Adam Martinson. --- dlls/oleaut32/tests/vartype.c | 4 +++- dlls/oleaut32/variant.c | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/oleaut32/tests/vartype.c b/dlls/oleaut32/tests/vartype.c index fe089bd..279fd7b 100644 --- a/dlls/oleaut32/tests/vartype.c +++ b/dlls/oleaut32/tests/vartype.c @@ -2592,7 +2592,9 @@ static void test_VarUI8FromStr(void) CONVERT_STR(VarUI8FromStr,"0",0); EXPECTI8(0); CONVERT_STR(VarUI8FromStr,"-1",0); EXPECT_OVERFLOW; CONVERT_STR(VarUI8FromStr,"2147483647",0); EXPECTI8(2147483647); - CONVERT_STR(VarUI8FromStr,"18446744073709551615",0); todo_wine EXPECTI864(0xFFFFFFFF,0xFFFFFFFF); + CONVERT_STR(VarUI8FromStr,"18446744073709551614",0); EXPECTI864(0xFFFFFFFF,0xFFFFFFFE); + CONVERT_STR(VarUI8FromStr,"18446744073709551615",0); EXPECTI864(0xFFFFFFFF,0xFFFFFFFF); + CONVERT_STR(VarUI8FromStr,"18446744073709551616",0); EXPECT_OVERFLOW; CONVERT_STR(VarUI8FromStr,"-1.5",LOCALE_NOUSEROVERRIDE); EXPECT_OVERFLOW; CONVERT_STR(VarUI8FromStr,"-0.6",LOCALE_NOUSEROVERRIDE); EXPECT_OVERFLOW; diff --git a/dlls/oleaut32/variant.c b/dlls/oleaut32/variant.c index 92a86fd..776c0e7 100644 --- a/dlls/oleaut32/variant.c +++ b/dlls/oleaut32/variant.c @@ -2186,7 +2186,7 @@ HRESULT WINAPI VarNumFromParseNum(NUMPARSE *pNumprs, BYTE *rgbDig, /* Convert the integer part of the number into a UI8 */ for (i = 0; i < wholeNumberDigits; i++) { - if (ul64 > (UI8_MAX / 10 - rgbDig[i])) + if (ul64 > UI8_MAX / 10 || (ul64 == UI8_MAX / 10 && rgbDig[i] > UI8_MAX % 10)) { TRACE("Overflow multiplying digits\n"); bOverflow = TRUE;

1 0

Adam Martinson : oleaut32/tests: Test VT_BSTR => VT_UI8 at upper limit.
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: 5d7aac8c38d8294fd1255706f2a134b8517c2893 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5d7aac8c38d8294fd1255706f… Author: Adam Martinson <amartinson(a)codeweavers.com> Date: Wed Nov 17 01:41:36 2010 -0600 oleaut32/tests: Test VT_BSTR => VT_UI8 at upper limit. --- dlls/oleaut32/tests/vartype.c | 9 +++++---- 1 files changed, 5 insertions(+), 4 deletions(-) diff --git a/dlls/oleaut32/tests/vartype.c b/dlls/oleaut32/tests/vartype.c index 361342e..fe089bd 100644 --- a/dlls/oleaut32/tests/vartype.c +++ b/dlls/oleaut32/tests/vartype.c @@ -2588,10 +2588,11 @@ static void test_VarUI8FromStr(void) CHECKPTR(VarUI8FromStr); - CONVERT_STR(VarUI8FromStr,NULL,0); EXPECT_MISMATCH; - CONVERT_STR(VarUI8FromStr,"0",0); EXPECTI8(0); - CONVERT_STR(VarUI8FromStr,"-1",0); EXPECT_OVERFLOW; - CONVERT_STR(VarUI8FromStr,"2147483647",0); EXPECTI8(2147483647); + CONVERT_STR(VarUI8FromStr,NULL,0); EXPECT_MISMATCH; + CONVERT_STR(VarUI8FromStr,"0",0); EXPECTI8(0); + CONVERT_STR(VarUI8FromStr,"-1",0); EXPECT_OVERFLOW; + CONVERT_STR(VarUI8FromStr,"2147483647",0); EXPECTI8(2147483647); + CONVERT_STR(VarUI8FromStr,"18446744073709551615",0); todo_wine EXPECTI864(0xFFFFFFFF,0xFFFFFFFF); CONVERT_STR(VarUI8FromStr,"-1.5",LOCALE_NOUSEROVERRIDE); EXPECT_OVERFLOW; CONVERT_STR(VarUI8FromStr,"-0.6",LOCALE_NOUSEROVERRIDE); EXPECT_OVERFLOW;

1 0

Henri Verbeet : ddraw/tests: Fix some failure messages.
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: d3e2e84feb873eaf4490c255ea9d6ef14e7bb1b7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d3e2e84feb873eaf4490c255e… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Nov 17 12:39:00 2010 +0100 ddraw/tests: Fix some failure messages. --- dlls/ddraw/tests/visual.c | 16 ++++++++-------- 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/dlls/ddraw/tests/visual.c b/dlls/ddraw/tests/visual.c index c756bac..2b4cf23 100644 --- a/dlls/ddraw/tests/visual.c +++ b/dlls/ddraw/tests/visual.c @@ -356,14 +356,14 @@ static void lighting_test(IDirect3DDevice7 *device) ok(hr == D3D_OK, "IDirect3DDevice7_EndScene failed with %08x\n", hr); } - color = getPixelColor(device, 160, 360); /* lower left quad - unlit without normals */ - ok(color == 0x00ff0000, "Unlit quad without normals has color %08x\n", color); - color = getPixelColor(device, 160, 120); /* upper left quad - lit without normals */ - ok(color == 0x00000000, "Lit quad without normals has color %08x\n", color); - color = getPixelColor(device, 480, 360); /* lower left quad - unlit width normals */ - ok(color == 0x000000ff, "Unlit quad width normals has color %08x\n", color); - color = getPixelColor(device, 480, 120); /* upper left quad - lit width normals */ - ok(color == 0x00000000, "Lit quad width normals has color %08x\n", color); + color = getPixelColor(device, 160, 360); /* Lower left quad - unlit without normals */ + ok(color == 0x00ff0000, "Unlit quad without normals has color 0x%08x, expected 0x00ff0000.\n", color); + color = getPixelColor(device, 160, 120); /* Upper left quad - lit without normals */ + ok(color == 0x00000000, "Lit quad without normals has color 0x%08x, expected 0x00000000.\n", color); + color = getPixelColor(device, 480, 360); /* Lower right quad - unlit with normals */ + ok(color == 0x000000ff, "Unlit quad with normals has color 0x%08x, expected 0x000000ff.\n", color); + color = getPixelColor(device, 480, 120); /* Upper right quad - lit with normals */ + ok(color == 0x00000000, "Lit quad with normals has color 0x%08x, expected 0x00000000.\n", color); hr = IDirect3DDevice7_SetRenderState(device, D3DRENDERSTATE_LIGHTING, FALSE); ok(hr == D3D_OK, "IDirect3DDevice7_SetRenderState returned %08x\n", hr);

1 0

Henri Verbeet : d3d8/tests: Fix some failure messages.
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: d2f23ff61ecf725566ae5cddc4ad827c4577d250 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d2f23ff61ecf725566ae5cddc… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Nov 17 12:38:59 2010 +0100 d3d8/tests: Fix some failure messages. --- dlls/d3d8/tests/visual.c | 16 ++++++++-------- 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/dlls/d3d8/tests/visual.c b/dlls/d3d8/tests/visual.c index 6819134..a6a9c9b 100644 --- a/dlls/d3d8/tests/visual.c +++ b/dlls/d3d8/tests/visual.c @@ -275,14 +275,14 @@ static void lighting_test(IDirect3DDevice8 *device) ok(hr == D3D_OK, "IDirect3DDevice8_EndScene failed with %#08x\n", hr); } - color = getPixelColor(device, 160, 360); /* lower left quad - unlit without normals */ - ok(color == 0x00ff0000, "Unlit quad without normals has color %08x\n", color); - color = getPixelColor(device, 160, 120); /* upper left quad - lit without normals */ - ok(color == 0x00000000, "Lit quad without normals has color %08x\n", color); - color = getPixelColor(device, 480, 360); /* lower left quad - unlit width normals */ - ok(color == 0x000000ff, "Unlit quad width normals has color %08x\n", color); - color = getPixelColor(device, 480, 120); /* upper left quad - lit width normals */ - ok(color == 0x00000000, "Lit quad width normals has color %08x\n", color); + color = getPixelColor(device, 160, 360); /* Lower left quad - unlit without normals */ + ok(color == 0x00ff0000, "Unlit quad without normals has color 0x%08x, expected 0x00ff0000.\n", color); + color = getPixelColor(device, 160, 120); /* Upper left quad - lit without normals */ + ok(color == 0x00000000, "Lit quad without normals has color 0x%08x, expected 0x00000000.\n", color); + color = getPixelColor(device, 480, 360); /* Lower right quad - unlit with normals */ + ok(color == 0x000000ff, "Unlit quad with normals has color 0x%08x, expected 0x000000ff.\n", color); + color = getPixelColor(device, 480, 120); /* Upper right quad - lit with normals */ + ok(color == 0x00000000, "Lit quad with normals has color 0x%08x, expected 0x00000000.\n", color); IDirect3DDevice8_Present(device, NULL, NULL, NULL, NULL);

1 0

Henri Verbeet : d3d9/tests: Fix some failure messages.
by Alexandre Julliard 17 Nov '10

17 Nov '10

Module: wine Branch: master Commit: 10ccba52e70a4f6898d030f7d3478a2b63c9ebc8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=10ccba52e70a4f6898d030f7d… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Nov 17 12:38:58 2010 +0100 d3d9/tests: Fix some failure messages. --- dlls/d3d9/tests/visual.c | 16 ++++++++-------- 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index 887f302..9e5b9a0 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -346,14 +346,14 @@ static void lighting_test(IDirect3DDevice9 *device) ok(hr == D3D_OK, "IDirect3DDevice9_EndScene failed with %08x\n", hr); } - color = getPixelColor(device, 160, 360); /* lower left quad - unlit without normals */ - ok(color == 0x00ff0000, "Unlit quad without normals has color %08x\n", color); - color = getPixelColor(device, 160, 120); /* upper left quad - lit without normals */ - ok(color == 0x00000000, "Lit quad without normals has color %08x\n", color); - color = getPixelColor(device, 480, 360); /* lower left quad - unlit with normals */ - ok(color == 0x000000ff, "Unlit quad with normals has color %08x\n", color); - color = getPixelColor(device, 480, 120); /* upper left quad - lit with normals */ - ok(color == 0x00000000, "Lit quad with normals has color %08x\n", color); + color = getPixelColor(device, 160, 360); /* Lower left quad - unlit without normals */ + ok(color == 0x00ff0000, "Unlit quad without normals has color 0x%08x, expected 0x00ff0000.\n", color); + color = getPixelColor(device, 160, 120); /* Upper left quad - lit without normals */ + ok(color == 0x00000000, "Lit quad without normals has color 0x%08x, expected 0x00000000.\n", color); + color = getPixelColor(device, 480, 360); /* Lower left quad - unlit with normals */ + ok(color == 0x000000ff, "Unlit quad with normals has color 0x%08x, expected 0x000000ff.\n", color); + color = getPixelColor(device, 480, 120); /* Upper left quad - lit with normals */ + ok(color == 0x00000000, "Lit quad with normals has color 0x%08x, expected 0x00000000.\n", color); IDirect3DDevice9_Present(device, NULL, NULL, NULL, NULL);

1 0

← Newer
1
...
23
24
25
26
27
28
29
...
87
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 November 2010