wine-commits December 2017

wine-commits@winehq.org

2 participants
450 discussions

Jacek Caban : mshtml/tests: Added more event object tests.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: afb0eeca01c4317f2e5a8e8364cc29f8c68237e0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=afb0eeca01c4317f2e5a8e836… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 4 18:02:30 2017 +0100 mshtml/tests: Added more event object tests. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/tests/events.js | 256 +++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 255 insertions(+), 1 deletion(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=afb0eeca01c4317f2e5a8…

1 0

Jacek Caban : mshtml: Expose IDocumentEvent to scripts.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: 49679eb97b7b0233f7d4db65c47520a3a20a5ecb URL: http://source.winehq.org/git/wine.git/?a=commit;h=49679eb97b7b0233f7d4db65c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 4 18:02:11 2017 +0100 mshtml: Expose IDocumentEvent to scripts. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmldoc.c | 4 +++- dlls/mshtml/mshtml_private.h | 1 + dlls/mshtml/tests/documentmode.js | 1 + 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/dlls/mshtml/htmldoc.c b/dlls/mshtml/htmldoc.c index c9f21fa..5e3ed75 100644 --- a/dlls/mshtml/htmldoc.c +++ b/dlls/mshtml/htmldoc.c @@ -5107,8 +5107,10 @@ static void HTMLDocumentNode_init_dispex_info(dispex_data_t *info, compat_mode_t { HTMLDOMNode_init_dispex_info(info, mode); - if(mode >= COMPAT_MODE_IE9) + if(mode >= COMPAT_MODE_IE9) { dispex_info_add_interface(info, IHTMLDocument7_tid, NULL); + dispex_info_add_interface(info, IDocumentEvent_tid, NULL); + } /* Depending on compatibility version, we add interfaces in different order * so that the right getElementById implementation is used. */ diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index 2387372..aef82cd 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -126,6 +126,7 @@ typedef struct EventTarget EventTarget; XDIID(HTMLDocumentEvents) \ XDIID(HTMLElementEvents2) \ XIID(IDOMEvent) \ + XIID(IDocumentEvent) \ XIID(IDocumentSelector) \ XIID(IElementSelector) \ XIID(IElementTraversal) \ diff --git a/dlls/mshtml/tests/documentmode.js b/dlls/mshtml/tests/documentmode.js index beee493..498dfed 100644 --- a/dlls/mshtml/tests/documentmode.js +++ b/dlls/mshtml/tests/documentmode.js @@ -62,6 +62,7 @@ function test_doc_props() { test_exposed("addEventListener", v >= 9); test_exposed("removeEventListener", v >= 9); test_exposed("dispatchEvent", v >= 9); + test_exposed("createEvent", v >= 9); test_exposed("parentWindow", true); if(v >= 9) ok(document.defaultView === document.parentWindow, "defaultView != parentWindow");

1 0

Jacek Caban : mshtml: Added IDOMEvent::dispatchEvent implementation.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: d6d23a5339ff5aca147e98e73b46b09fce5b6856 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d6d23a5339ff5aca147e98e73… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 4 18:02:01 2017 +0100 mshtml: Added IDOMEvent::dispatchEvent implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 46 ++++++++++++++++++++++++++++++++++------------ dlls/mshtml/tests/events.c | 11 +++++++++++ 2 files changed, 45 insertions(+), 12 deletions(-) diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c index aed6111..7a01ba5 100644 --- a/dlls/mshtml/htmlevent.c +++ b/dlls/mshtml/htmlevent.c @@ -56,6 +56,7 @@ typedef struct { typedef enum { DISPATCH_BOTH, + DISPATCH_STANDARD, DISPATCH_LEGACY } dispatch_mode_t; @@ -896,6 +897,13 @@ static inline DOMEvent *impl_from_IDOMEvent(IDOMEvent *iface) return CONTAINING_RECORD(iface, DOMEvent, IDOMEvent_iface); } +static const IDOMEventVtbl DOMEventVtbl; + +static inline DOMEvent *unsafe_impl_from_IDOMEvent(IDOMEvent *iface) +{ + return iface && iface->lpVtbl == &DOMEventVtbl ? impl_from_IDOMEvent(iface) : NULL; +} + static HRESULT WINAPI DOMEvent_QueryInterface(IDOMEvent *iface, REFIID riid, void **ppv) { DOMEvent *This = impl_from_IDOMEvent(iface); @@ -1434,7 +1442,7 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event, disp continue; break; case LISTENER_TYPE_ATTACHED: - if(event->phase == DEP_CAPTURING_PHASE) + if(event->phase == DEP_CAPTURING_PHASE || dispatch_mode == DISPATCH_STANDARD) continue; break; } @@ -1570,8 +1578,8 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event, disp event->current_target = NULL; } -static void dispatch_event_object(EventTarget *event_target, DOMEvent *event, - dispatch_mode_t dispatch_mode) +static HRESULT dispatch_event_object(EventTarget *event_target, DOMEvent *event, + dispatch_mode_t dispatch_mode, VARIANT_BOOL *r) { EventTarget *target_chain_buf[8], **target_chain = target_chain_buf; unsigned chain_cnt, chain_buf_size, i; @@ -1583,14 +1591,14 @@ static void dispatch_event_object(EventTarget *event_target, DOMEvent *event, TRACE("(%p) %s\n", event_target, debugstr_w(event->type)); - if(event->event_id == EVENTID_LAST) { - FIXME("Unsupported on unknown events\n"); - return; + if(!event->type) { + FIXME("Uninitialized event.\n"); + return E_FAIL; } if(event->current_target) { FIXME("event is being dispatched.\n"); - return; + return E_FAIL; } iter = event_target; @@ -1654,6 +1662,9 @@ static void dispatch_event_object(EventTarget *event_target, DOMEvent *event, call_event_handlers(target_chain[i], event, dispatch_mode); } + if(r) + *r = variant_bool(!event->prevent_default); + if(target_vtbl && target_vtbl->set_current_event) { prev_event = target_vtbl->set_current_event(&event_target->dispex, prev_event); if(prev_event) @@ -1675,6 +1686,7 @@ static void dispatch_event_object(EventTarget *event_target, DOMEvent *event, } } + event->prevent_default = FALSE; if(event_obj_ref) { event->event_obj = NULL; IHTMLEventObj_Release(&event_obj_ref->IHTMLEventObj_iface); @@ -1684,11 +1696,13 @@ static void dispatch_event_object(EventTarget *event_target, DOMEvent *event, IDispatchEx_Release(&target_chain[i]->dispex.IDispatchEx_iface); if(target_chain != target_chain_buf) heap_free(target_chain); + + return S_OK; } void dispatch_event(EventTarget *event_target, DOMEvent *event) { - dispatch_event_object(event_target, event, DISPATCH_BOTH); + dispatch_event_object(event_target, event, DISPATCH_BOTH, NULL); } HRESULT fire_event(HTMLDOMNode *node, const WCHAR *event_name, VARIANT *event_var, VARIANT_BOOL *cancelled) @@ -1738,7 +1752,7 @@ HRESULT fire_event(HTMLDOMNode *node, const WCHAR *event_name, VARIANT *event_va if(SUCCEEDED(hres)) { event_obj->event->event_obj = &event_obj->IHTMLEventObj_iface; - dispatch_event_object(&node->event_target, event_obj->event, DISPATCH_LEGACY); + dispatch_event_object(&node->event_target, event_obj->event, DISPATCH_LEGACY, NULL); event_obj->event->event_obj = NULL; } @@ -2220,11 +2234,19 @@ static HRESULT WINAPI EventTarget_removeEventListener(IEventTarget *iface, BSTR return S_OK; } -static HRESULT WINAPI EventTarget_dispatchEvent(IEventTarget *iface, IDOMEvent *event, VARIANT_BOOL *result) +static HRESULT WINAPI EventTarget_dispatchEvent(IEventTarget *iface, IDOMEvent *event_iface, VARIANT_BOOL *result) { EventTarget *This = impl_from_IEventTarget(iface); - FIXME("(%p)->(%p %p)\n", This, event, result); - return E_NOTIMPL; + DOMEvent *event = unsafe_impl_from_IDOMEvent(event_iface); + + TRACE("(%p)->(%p %p)\n", This, event, result); + + if(!event) { + WARN("Invalid event\n"); + return E_INVALIDARG; + } + + return dispatch_event_object(This, event, DISPATCH_STANDARD, result); } HRESULT IEventTarget_addEventListener_hook(DispatchEx *dispex, LCID lcid, WORD flags, diff --git a/dlls/mshtml/tests/events.c b/dlls/mshtml/tests/events.c index a943e41..28aa93c 100644 --- a/dlls/mshtml/tests/events.c +++ b/dlls/mshtml/tests/events.c @@ -2667,6 +2667,8 @@ static void test_iframe_connections(IHTMLDocument2 *doc) static void test_create_event(IHTMLDocument2 *doc) { IDocumentEvent *doc_event; + IEventTarget *event_target; + IHTMLElement *elem; IDOMEvent *event; VARIANT_BOOL b; USHORT phase; @@ -2709,6 +2711,15 @@ static void test_create_event(IHTMLDocument2 *doc) ok(hres == S_OK, "get_cancelable failed: %08x\n", hres); ok(!b, "cancelable = %x\n", b); + elem = doc_get_body(doc); + hres = IHTMLElement_QueryInterface(elem, &IID_IEventTarget, (void**)&event_target); + IHTMLElement_Release(elem); + + hres = IEventTarget_dispatchEvent(event_target, NULL, &b); + ok(hres == E_INVALIDARG, "dispatchEvent failed: %08x\n", hres); + + IEventTarget_Release(event_target); + IDOMEvent_Release(event); IDocumentEvent_Release(doc_event);

1 0

Jacek Caban : mshtml: Don't store dispatch mode in DOMEvent object.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: 986f701cd4c09329786177cd7d493403889f1450 URL: http://source.winehq.org/git/wine.git/?a=commit;h=986f701cd4c09329786177cd7… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 4 18:01:43 2017 +0100 mshtml: Don't store dispatch mode in DOMEvent object. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 33 +++++++++++++++++++++------------ dlls/mshtml/htmlevent.h | 1 - 2 files changed, 21 insertions(+), 13 deletions(-) diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c index 3d68555..aed6111 100644 --- a/dlls/mshtml/htmlevent.c +++ b/dlls/mshtml/htmlevent.c @@ -54,6 +54,11 @@ typedef struct { WCHAR type[1]; } listener_container_t; +typedef enum { + DISPATCH_BOTH, + DISPATCH_LEGACY +} dispatch_mode_t; + static const WCHAR abortW[] = {'a','b','o','r','t',0}; static const WCHAR beforeactivateW[] = {'b','e','f','o','r','e','a','c','t','i','v','a','t','e',0}; static const WCHAR beforeunloadW[] = {'b','e','f','o','r','e','u','n','l','o','a','d',0}; @@ -1362,7 +1367,7 @@ static BOOL is_cp_event(cp_static_data_t *data, DISPID dispid) return FALSE; } -static void call_event_handlers(EventTarget *event_target, DOMEvent *event) +static void call_event_handlers(EventTarget *event_target, DOMEvent *event, dispatch_mode_t dispatch_mode) { const listener_container_t *container = get_listener_container(event_target, event->type, FALSE); const BOOL use_quirks = use_event_quirks(event_target); @@ -1421,13 +1426,13 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event) continue; break; case LISTENER_TYPE_CAPTURE: - if(event->phase == DEP_BUBBLING_PHASE || event->in_fire_event) + if(event->phase == DEP_BUBBLING_PHASE || dispatch_mode == DISPATCH_LEGACY) continue; break; case LISTENER_TYPE_BUBBLE: - if(event->in_fire_event) + if(event->phase == DEP_CAPTURING_PHASE || dispatch_mode == DISPATCH_LEGACY) continue; - /* fallthrough */ + break; case LISTENER_TYPE_ATTACHED: if(event->phase == DEP_CAPTURING_PHASE) continue; @@ -1463,7 +1468,7 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event) V_VT(args) = VT_DISPATCH; V_DISPATCH(args) = (IDispatch*)&event_target->dispex.IDispatchEx_iface; V_VT(args+1) = VT_DISPATCH; - V_DISPATCH(args+1) = event->in_fire_event + V_DISPATCH(args+1) = dispatch_mode == DISPATCH_LEGACY ? (IDispatch*)event->event_obj : (IDispatch*)&event->IDOMEvent_iface; V_VT(&v) = VT_EMPTY; @@ -1565,7 +1570,8 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event) event->current_target = NULL; } -void dispatch_event(EventTarget *event_target, DOMEvent *event) +static void dispatch_event_object(EventTarget *event_target, DOMEvent *event, + dispatch_mode_t dispatch_mode) { EventTarget *target_chain_buf[8], **target_chain = target_chain_buf; unsigned chain_cnt, chain_buf_size, i; @@ -1635,17 +1641,17 @@ void dispatch_event(EventTarget *event_target, DOMEvent *event) event->phase = DEP_CAPTURING_PHASE; i = chain_cnt-1; while(!event->stop_propagation && i) - call_event_handlers(target_chain[i--], event); + call_event_handlers(target_chain[i--], event, dispatch_mode); if(!event->stop_propagation) { event->phase = DEP_AT_TARGET; - call_event_handlers(target_chain[0], event); + call_event_handlers(target_chain[0], event, dispatch_mode); } if(event->bubbles) { event->phase = DEP_BUBBLING_PHASE; for(i = 1; !event->stop_propagation && i < chain_cnt; i++) - call_event_handlers(target_chain[i], event); + call_event_handlers(target_chain[i], event, dispatch_mode); } if(target_vtbl && target_vtbl->set_current_event) { @@ -1680,6 +1686,11 @@ void dispatch_event(EventTarget *event_target, DOMEvent *event) heap_free(target_chain); } +void dispatch_event(EventTarget *event_target, DOMEvent *event) +{ + dispatch_event_object(event_target, event, DISPATCH_BOTH); +} + HRESULT fire_event(HTMLDOMNode *node, const WCHAR *event_name, VARIANT *event_var, VARIANT_BOOL *cancelled) { HTMLEventObj *event_obj = NULL; @@ -1727,9 +1738,7 @@ HRESULT fire_event(HTMLDOMNode *node, const WCHAR *event_name, VARIANT *event_va if(SUCCEEDED(hres)) { event_obj->event->event_obj = &event_obj->IHTMLEventObj_iface; - event_obj->event->in_fire_event++; - dispatch_event(&node->event_target, event_obj->event); - event_obj->event->in_fire_event--; + dispatch_event_object(&node->event_target, event_obj->event, DISPATCH_LEGACY); event_obj->event->event_obj = NULL; } diff --git a/dlls/mshtml/htmlevent.h b/dlls/mshtml/htmlevent.h index f99c893..3775e44 100644 --- a/dlls/mshtml/htmlevent.h +++ b/dlls/mshtml/htmlevent.h @@ -75,7 +75,6 @@ typedef struct { IHTMLEventObj *event_obj; BOOL no_event_obj; - unsigned in_fire_event; } DOMEvent; void check_event_attr(HTMLDocumentNode*,nsIDOMHTMLElement*) DECLSPEC_HIDDEN;

1 0

Jacek Caban : mshtml: Don't call preventDefault in stopPropagation.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: f0b3fe8f836317dd073444570f185b4aedd2739b URL: http://source.winehq.org/git/wine.git/?a=commit;h=f0b3fe8f836317dd073444570… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 4 18:01:31 2017 +0100 mshtml: Don't call preventDefault in stopPropagation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 1 - 1 file changed, 1 deletion(-) diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c index fba04b5..3d68555 100644 --- a/dlls/mshtml/htmlevent.c +++ b/dlls/mshtml/htmlevent.c @@ -1117,7 +1117,6 @@ static HRESULT WINAPI DOMEvent_stopPropagation(IDOMEvent *iface) This->stop_propagation = TRUE; nsIDOMEvent_StopPropagation(This->nsevent); - IDOMEvent_preventDefault(&This->IDOMEvent_iface); return S_OK; }

1 0

Jacek Caban : mshtml: Added IDOMEvent::initEvent implementation.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: 30e2762b823c802f0c2e20fbbd2539b5a96d0c5b URL: http://source.winehq.org/git/wine.git/?a=commit;h=30e2762b823c802f0c2e20fbb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Dec 4 18:01:23 2017 +0100 mshtml: Added IDOMEvent::initEvent implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 33 ++++++++++++++++++++++++++++++--- dlls/mshtml/mshtml_private.h | 4 ++++ dlls/mshtml/nsembed.c | 1 - 3 files changed, 34 insertions(+), 4 deletions(-) diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c index c06d062..fba04b5 100644 --- a/dlls/mshtml/htmlevent.c +++ b/dlls/mshtml/htmlevent.c @@ -218,7 +218,6 @@ static eventid_t str_to_eid(const WCHAR *str) return i; } - ERR("unknown type %s\n", debugstr_w(str)); return EVENTID_LAST; } @@ -1062,11 +1061,39 @@ static HRESULT WINAPI DOMEvent_get_type(IDOMEvent *iface, BSTR *p) return S_OK; } +#ifdef __i386__ +#define nsIDOMEvent_InitEvent(_this,type,bubbles,cancelable) \ + ((void (WINAPI*)(void*,nsIDOMEvent*,const nsAString*,cpp_bool,cpp_bool)) \ + &call_thiscall_func)((_this)->lpVtbl->InitEvent,_this,type,bubbles,cancelable) + +#endif + static HRESULT WINAPI DOMEvent_initEvent(IDOMEvent *iface, BSTR type, VARIANT_BOOL can_bubble, VARIANT_BOOL cancelable) { DOMEvent *This = impl_from_IDOMEvent(iface); - FIXME("(%p)->()\n", This); - return E_NOTIMPL; + nsAString nsstr; + + TRACE("(%p)->(%s %x %x)\n", This, debugstr_w(type), can_bubble, cancelable); + + if(This->target) { + TRACE("called on already dispatched event\n"); + return S_OK; + } + + heap_free(This->type); + This->type = heap_strdupW(type); + if(!This->type) + return E_OUTOFMEMORY; + This->event_id = str_to_eid(type); + + This->bubbles = !!can_bubble; + This->cancelable = !!cancelable; + + nsAString_InitDepend(&nsstr, type); + nsIDOMEvent_InitEvent(This->nsevent, &nsstr, This->bubbles, This->cancelable); + nsAString_Finish(&nsstr); + + return S_OK; } static HRESULT WINAPI DOMEvent_preventDefault(IDOMEvent *iface) diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index f55f064..2387372 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -1341,6 +1341,10 @@ static inline VARIANT_BOOL variant_bool(BOOL b) return b ? VARIANT_TRUE : VARIANT_FALSE; } +#ifdef __i386__ +extern void *call_thiscall_func; +#endif + UINT cp_from_charset_string(BSTR) DECLSPEC_HIDDEN; BSTR charset_string_from_cp(UINT) DECLSPEC_HIDDEN; HDC get_display_dc(void) DECLSPEC_HIDDEN; diff --git a/dlls/mshtml/nsembed.c b/dlls/mshtml/nsembed.c index 4db61dc..8782ece 100644 --- a/dlls/mshtml/nsembed.c +++ b/dlls/mshtml/nsembed.c @@ -2200,7 +2200,6 @@ void NSContainer_Release(NSContainer *This) * This will be removed after the next Gecko update, that will change calling convention on Gecko side. */ #ifdef __i386__ -extern void *call_thiscall_func; __ASM_GLOBAL_FUNC(call_thiscall_func, "popl %eax\n\t" "popl %edx\n\t"

1 0

Nikolay Sivov : mshtml: Avoid potential moniker instance leak.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: 2b37b97d0c6356f8b89401183295222123e4b619 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2b37b97d0c6356f8b89401183… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Dec 4 14:56:15 2017 +0300 mshtml: Avoid potential moniker instance leak. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/persist.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/dlls/mshtml/persist.c b/dlls/mshtml/persist.c index 25767ef..b82750a 100644 --- a/dlls/mshtml/persist.c +++ b/dlls/mshtml/persist.c @@ -918,10 +918,9 @@ static HRESULT WINAPI PersistStreamInit_Load(IPersistStreamInit *iface, IStream prepare_for_binding(This, mon, FALSE); hres = set_moniker(This->window, mon, NULL, NULL, NULL, TRUE); - if(FAILED(hres)) - return hres; + if(SUCCEEDED(hres)) + hres = channelbsc_load_stream(This->window->pending_window, mon, pStm); - hres = channelbsc_load_stream(This->window->pending_window, mon, pStm); IMoniker_Release(mon); return hres; } @@ -976,10 +975,9 @@ static HRESULT WINAPI PersistStreamInit_InitNew(IPersistStreamInit *iface) prepare_for_binding(This, mon, FALSE); hres = set_moniker(This->window, mon, NULL, NULL, NULL, FALSE); - if(FAILED(hres)) - return hres; + if(SUCCEEDED(hres)) + hres = channelbsc_load_stream(This->window->pending_window, mon, NULL); - hres = channelbsc_load_stream(This->window->pending_window, mon, NULL); IMoniker_Release(mon); return hres; }

1 0

Nikolay Sivov : user32/tests: Added some tests for WM_GETTEXTLENGTH/ WM_GETTEXT after EM_GETHANDLE was used.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: 71eac4a70b56ce5a8b20a0e10bacc7dcf7638627 URL: http://source.winehq.org/git/wine.git/?a=commit;h=71eac4a70b56ce5a8b20a0e10… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Dec 4 21:31:35 2017 +0300 user32/tests: Added some tests for WM_GETTEXTLENGTH/WM_GETTEXT after EM_GETHANDLE was used. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/edit.c | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/dlls/user32/tests/edit.c b/dlls/user32/tests/edit.c index 0926a58..783ced6 100644 --- a/dlls/user32/tests/edit.c +++ b/dlls/user32/tests/edit.c @@ -2735,6 +2735,7 @@ static void test_EM_GETHANDLE(void) { static const char str0[] = "untouched"; static const char str1[] = "1111+1111+1111#"; + static const char str1_1[] = "2111+1111+1111#"; static const char str2[] = "2222-2222-2222-2222#"; static const char str3[] = "3333*3333*3333*3333*3333#"; CHAR current[42]; @@ -2783,6 +2784,33 @@ static void test_EM_GETHANDLE(void) "got %d and \"%s\" (expected %d and \"%s\")\n", len, buffer, lstrlenA(str1), str1); LocalUnlock(hmem); + /* See if WM_GETTEXTLENGTH/WM_GETTEXT still work. */ + len = SendMessageA(hEdit, WM_GETTEXTLENGTH, 0, 0); +todo_wine + ok(len == lstrlenA(str1), "Unexpected text length %d.\n", len); + + lstrcpyA(current, str0); + r = SendMessageA(hEdit, WM_GETTEXT, sizeof(current), (LPARAM)current); +todo_wine + ok((r == lstrlenA(str1)) && !lstrcmpA(current, str1), + "Unexpected retval %d and text \"%s\" (expected %d and \"%s\")\n", r, current, lstrlenA(str1), str1); + + /* Application altered buffer contents, see if WM_GETTEXTLENGTH/WM_GETTEXT pick that up. */ + buffer = LocalLock(hmem); + ok(buffer != NULL, "got %p (expected != NULL)\n", buffer); + buffer[0] = '2'; + LocalUnlock(hmem); + + len = SendMessageA(hEdit, WM_GETTEXTLENGTH, 0, 0); +todo_wine + ok(len == lstrlenA(str1), "Unexpected text length %d.\n", len); + + lstrcpyA(current, str0); + r = SendMessageA(hEdit, WM_GETTEXT, sizeof(current), (LPARAM)current); +todo_wine + ok((r == lstrlenA(str1_1)) && !lstrcmpA(current, str1_1), + "Unexpected retval %d and text \"%s\" (expected %d and \"%s\")\n", r, current, lstrlenA(str1_1), str1_1); + /* use LocalAlloc first to get a different handle */ halloc = LocalAlloc(LMEM_MOVEABLE, 42); ok(halloc != NULL, "got %p (expected != NULL)\n", halloc);

1 0

Henri Verbeet : wined3d: Really remove an outdated comment.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: 89d038e189515f08ec1d23a3abfb6ec5647aacf1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=89d038e189515f08ec1d23a3a… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Dec 4 17:30:25 2017 +0330 wined3d: Really remove an outdated comment. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/glsl_shader.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/dlls/wined3d/glsl_shader.c b/dlls/wined3d/glsl_shader.c index 04ab66b..5d08c00 100644 --- a/dlls/wined3d/glsl_shader.c +++ b/dlls/wined3d/glsl_shader.c @@ -2713,10 +2713,6 @@ static void shader_generate_glsl_declarations(const struct wined3d_context *cont } } -/***************************************************************************** - * Functions to generate GLSL strings from DirectX Shader bytecode begin here. - ****************************************************************************/ - /* Prototypes */ static void shader_glsl_add_src_param(const struct wined3d_shader_instruction *ins, const struct wined3d_shader_src_param *wined3d_src, DWORD mask, struct glsl_src_param *glsl_src);

1 0

Fabian Maurer : wined3d: Unbind currently bound FBOs before dispatching compute shaders.
by Alexandre Julliard 04 Dec '17

04 Dec '17

Module: wine Branch: master Commit: 0d45106054b8fedd8b05bada8e0e589c11bf6375 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0d45106054b8fedd8b05bada8… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Mon Dec 4 17:30:24 2017 +0330 wined3d: Unbind currently bound FBOs before dispatching compute shaders. Many thanks to Matias N. Goldberg and Philip Rebohle for investigating and solving the bug! Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/context.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 8af9e4b..23f63aa 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -4013,6 +4013,16 @@ void context_apply_compute_state(struct wined3d_context *context, context->update_unordered_access_view_bindings = 1; } + /* Updates to currently bound render targets aren't necessarily coherent + * between the graphics and compute pipelines. Unbind any currently bound + * FBO here to ensure preceding updates to its attachments by the graphics + * pipeline are visible to the compute pipeline. + * + * Without this, the bloom effect in Nier:Automata is too bright on the + * Mesa radeonsi driver, and presumably on other Mesa based drivers. */ + context_bind_fbo(context, GL_FRAMEBUFFER, 0); + context_invalidate_state(context, STATE_FRAMEBUFFER); + context->last_was_blit = FALSE; }

1 0

← Newer
1
...
32
33
34
35
36
37
38
...
45
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 2017