October 2017 - Wine-Commits

Jacek Caban : mshtml/tests: Added addEventListener tests.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: b81eea83da2c18af742d3f68a70f5ac506eec5aa URL: http://source.winehq.org/git/wine.git/?a=commit;h=b81eea83da2c18af742d3f68a… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Oct 25 18:14:05 2017 +0200 mshtml/tests: Added addEventListener tests. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/tests/events.c | 98 +++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 97 insertions(+), 1 deletion(-) diff --git a/dlls/mshtml/tests/events.c b/dlls/mshtml/tests/events.c index 91ec4b6..4f57745 100644 --- a/dlls/mshtml/tests/events.c +++ b/dlls/mshtml/tests/events.c @@ -62,6 +62,8 @@ DEFINE_EXPECT(body_onclick); DEFINE_EXPECT(doc_onclick_attached); DEFINE_EXPECT(div_onclick); DEFINE_EXPECT(div_onclick_attached); +DEFINE_EXPECT(div_onclick_capture); +DEFINE_EXPECT(div_onclick_bubble); DEFINE_EXPECT(timeout); DEFINE_EXPECT(doccp_onclick); DEFINE_EXPECT(doccp_onclick_cancel); @@ -756,6 +758,42 @@ static void _elem_attach_event(unsigned line, IUnknown *unk, const char *namea, ok_(__FILE__,line)(res == VARIANT_TRUE, "attachEvent returned %x\n", res); } +#define add_event_listener(a,b,c,d) _add_event_listener(__LINE__,a,b,c,d) +static void _add_event_listener(unsigned line, IUnknown *unk, const char *type, IDispatch *listener, VARIANT_BOOL use_capture) +{ + IEventTarget *event_target; + BSTR str; + HRESULT hres; + + hres = IUnknown_QueryInterface(unk, &IID_IEventTarget, (void**)&event_target); + ok_(__FILE__,line)(hres == S_OK, "Could not get IEventTarget iface: %08x\n", hres); + + str = a2bstr(type); + hres = IEventTarget_addEventListener(event_target, str, listener, use_capture); + SysFreeString(str); + ok_(__FILE__,line)(hres == S_OK, "addEventListener failed: %08x\n", hres); + + IEventTarget_Release(event_target); +} + +#define remove_event_listener(a,b,c,d) _remove_event_listener(__LINE__,a,b,c,d) +static void _remove_event_listener(unsigned line, IUnknown *unk, const char *type, IDispatch *listener, VARIANT_BOOL use_capture) +{ + IEventTarget *event_target; + BSTR str; + HRESULT hres; + + hres = IUnknown_QueryInterface(unk, &IID_IEventTarget, (void**)&event_target); + ok_(__FILE__,line)(hres == S_OK, "Could not get IEventTarget iface: %08x\n", hres); + + str = a2bstr(type); + hres = IEventTarget_removeEventListener(event_target, str, listener, use_capture); + SysFreeString(str); + ok_(__FILE__,line)(hres == S_OK, "removeEventListener failed: %08x\n", hres); + + IEventTarget_Release(event_target); +} + #define elem_detach_event(a,b,c) _elem_detach_event(__LINE__,a,b,c) static void _elem_detach_event(unsigned line, IUnknown *unk, const char *namea, IDispatch *disp) { @@ -998,6 +1036,30 @@ static HRESULT WINAPI body_onclick(IDispatchEx *iface, DISPID id, LCID lcid, WOR EVENT_HANDLER_FUNC_OBJ(body_onclick); +static HRESULT WINAPI div_onclick_capture(IDispatchEx *iface, DISPID id, LCID lcid, WORD wFlags, DISPPARAMS *pdp, + VARIANT *pvarRes, EXCEPINFO *pei, IServiceProvider *pspCaller) +{ + CHECK_EXPECT(div_onclick_capture); + + test_event_args(NULL, id, wFlags, pdp, pvarRes, pei, pspCaller); + test_event_src("DIV"); + return S_OK; +} + +EVENT_HANDLER_FUNC_OBJ(div_onclick_capture); + +static HRESULT WINAPI div_onclick_bubble(IDispatchEx *iface, DISPID id, LCID lcid, WORD wFlags, DISPPARAMS *pdp, + VARIANT *pvarRes, EXCEPINFO *pei, IServiceProvider *pspCaller) +{ + CHECK_EXPECT(div_onclick_bubble); + + test_event_args(NULL, id, wFlags, pdp, pvarRes, pei, pspCaller); + test_event_src("DIV"); + return S_OK; +} + +EVENT_HANDLER_FUNC_OBJ(div_onclick_bubble); + static HRESULT WINAPI img_onload(IDispatchEx *iface, DISPID id, LCID lcid, WORD wFlags, DISPPARAMS *pdp, VARIANT *pvarRes, EXCEPINFO *pei, IServiceProvider *pspCaller) { @@ -1810,6 +1872,11 @@ static void test_onclick(IHTMLDocument2 *doc) ok(V_DISPATCH(&v) == (IDispatch*)&document_onclick_obj, "V_DISPATCH(onclick) != onclickFunc\n"); VariantClear(&v); + if(document_mode >= 9) { + add_event_listener((IUnknown*)div, "click", (IDispatch*)&div_onclick_capture_obj, VARIANT_TRUE); + add_event_listener((IUnknown*)div, "click", (IDispatch*)&div_onclick_bubble_obj, VARIANT_FALSE); + } + body = doc_get_body(doc); V_VT(&v) = VT_DISPATCH; @@ -1817,7 +1884,7 @@ static void test_onclick(IHTMLDocument2 *doc) hres = IHTMLElement_put_onclick(body, v); ok(hres == S_OK, "put_onclick failed: %08x\n", hres); - if(winetest_interactive) { + if(winetest_interactive && document_mode < 9) { SET_EXPECT(div_onclick); SET_EXPECT(div_onclick_attached); SET_EXPECT(body_onclick); @@ -1833,6 +1900,10 @@ static void test_onclick(IHTMLDocument2 *doc) SET_EXPECT(div_onclick); SET_EXPECT(div_onclick_attached); + if(document_mode >= 9) { + SET_EXPECT(div_onclick_capture); + SET_EXPECT(div_onclick_bubble); + } SET_EXPECT(body_onclick); SET_EXPECT(document_onclick); SET_EXPECT(invoke_onclick); @@ -1842,6 +1913,10 @@ static void test_onclick(IHTMLDocument2 *doc) CHECK_CALLED(div_onclick); CHECK_CALLED(div_onclick_attached); + if(document_mode >= 9) { + CHECK_CALLED(div_onclick_capture); + CHECK_CALLED(div_onclick_bubble); + } CHECK_CALLED(body_onclick); CHECK_CALLED(document_onclick); CHECK_CALLED(invoke_onclick); @@ -1868,6 +1943,10 @@ static void test_onclick(IHTMLDocument2 *doc) SET_EXPECT(div_onclick); SET_EXPECT(div_onclick_disp); SET_EXPECT(div_onclick_attached); + if(document_mode >= 9) { + SET_EXPECT(div_onclick_capture); + SET_EXPECT(div_onclick_bubble); + } SET_EXPECT(body_onclick); SET_EXPECT(document_onclick); SET_EXPECT(doc_onclick_attached); @@ -1880,6 +1959,10 @@ static void test_onclick(IHTMLDocument2 *doc) CHECK_CALLED(div_onclick); CHECK_CALLED(div_onclick_disp); CHECK_CALLED(div_onclick_attached); + if(document_mode >= 9) { + CHECK_CALLED(div_onclick_capture); + CHECK_CALLED(div_onclick_bubble); + } CHECK_CALLED(body_onclick); CHECK_CALLED(document_onclick); CHECK_CALLED(doc_onclick_attached); @@ -1891,6 +1974,10 @@ static void test_onclick(IHTMLDocument2 *doc) SET_EXPECT(div_onclick); SET_EXPECT(div_onclick_disp); SET_EXPECT(div_onclick_attached); + if(document_mode >= 9) { + SET_EXPECT(div_onclick_capture); + SET_EXPECT(div_onclick_bubble); + } SET_EXPECT(elem2_cp_onclick); SET_EXPECT(body_onclick); SET_EXPECT(document_onclick); @@ -1906,6 +1993,10 @@ static void test_onclick(IHTMLDocument2 *doc) CHECK_CALLED(div_onclick); CHECK_CALLED(div_onclick_disp); CHECK_CALLED(div_onclick_attached); + if(document_mode >= 9) { + CHECK_CALLED(div_onclick_capture); + CHECK_CALLED(div_onclick_bubble); + } CHECK_CALLED(elem2_cp_onclick); CHECK_CALLED(body_onclick); CHECK_CALLED(document_onclick); @@ -1929,6 +2020,11 @@ static void test_onclick(IHTMLDocument2 *doc) elem_detach_event((IUnknown*)div, "test", (IDispatch*)&div_onclick_disp); doc_detach_event(doc, "onclick", (IDispatch*)&doc_onclick_attached_obj); + if(document_mode >= 9) { + remove_event_listener((IUnknown*)div, "click", (IDispatch*)&div_onclick_capture_obj, VARIANT_TRUE); + remove_event_listener((IUnknown*)div, "click", (IDispatch*)&div_onclick_bubble_obj, VARIANT_FALSE); + } + SET_EXPECT(div_onclick_attached); SET_EXPECT(body_onclick); SET_EXPECT(document_onclick);

1 0

Jacek Caban : mshtml: Added support for capturing event phase.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: 2bffe9af8d5e6360dc1e977535d41acf2e64c649 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2bffe9af8d5e6360dc1e97753… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Oct 25 18:13:54 2017 +0200 mshtml: Added support for capturing event phase. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c index 778b692..a3359e0 100644 --- a/dlls/mshtml/htmlevent.c +++ b/dlls/mshtml/htmlevent.c @@ -1551,8 +1551,15 @@ void dispatch_event(EventTarget *event_target, DOMEvent *event) event->target = event_target; IDispatchEx_AddRef(&event_target->dispex.IDispatchEx_iface); - event->phase = DEP_AT_TARGET; - call_event_handlers(target_chain[0], event); + event->phase = DEP_CAPTURING_PHASE; + i = chain_cnt-1; + while(!event->stop_propagation && i) + call_event_handlers(target_chain[i--], event); + + if(!event->stop_propagation) { + event->phase = DEP_AT_TARGET; + call_event_handlers(target_chain[0], event); + } if(event_flags & EVENT_BUBBLES) { event->phase = DEP_BUBBLING_PHASE;

1 0

Jacek Caban : mshtml: Properly invoke event listeners in IE9+ mode.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: d1f1e93c4659032cd6e24d44a746bf35046d1bbd URL: http://source.winehq.org/git/wine.git/?a=commit;h=d1f1e93c4659032cd6e24d44a… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Oct 25 18:13:26 2017 +0200 mshtml: Properly invoke event listeners in IE9+ mode. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 55 ++++++++++++++++++++++++++++++++++++++++++---- dlls/mshtml/htmlevent.h | 3 ++- dlls/mshtml/tests/events.c | 42 +++++++++++++++++++++++++++++++++-- 3 files changed, 93 insertions(+), 7 deletions(-) diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c index 8ca94eb..88282c0 100644 --- a/dlls/mshtml/htmlevent.c +++ b/dlls/mshtml/htmlevent.c @@ -1292,6 +1292,7 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event) const eventid_t eid = event->event_id; const listener_container_t *container = get_listener_container(event_target, eid, FALSE); const BOOL cancelable = event_info[eid].flags & EVENT_CANCELABLE; + const BOOL use_quirks = use_event_quirks(event_target); event_listener_t *listener, listeners_buf[8], *listeners = listeners_buf; unsigned listeners_cnt, listeners_size; ConnectionPointContainer *cp_container = NULL; @@ -1299,9 +1300,10 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event) VARIANT v; HRESULT hres; - if(container && !list_empty(&container->listeners)) { + if(use_quirks && container && !list_empty(&container->listeners) + && event->phase != DEP_CAPTURING_PHASE) { listener = LIST_ENTRY(list_tail(&container->listeners), event_listener_t, entry); - if(listener->function && listener->type == LISTENER_TYPE_ONEVENT) { + if(listener && listener->function && listener->type == LISTENER_TYPE_ONEVENT) { DISPID named_arg = DISPID_THIS; VARIANTARG arg; DISPPARAMS dp = {&arg, &named_arg, 1, 1}; @@ -1337,6 +1339,16 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event) LIST_FOR_EACH_ENTRY(listener, &container->listeners, event_listener_t, entry) { if(!listener->function) continue; + switch(listener->type) { + case LISTENER_TYPE_ONEVENT: + if(use_quirks || event->phase == DEP_CAPTURING_PHASE) + continue; + break; + case LISTENER_TYPE_ATTACHED: + if(event->phase == DEP_CAPTURING_PHASE) + continue; + break; + } if(listeners_cnt == listeners_size) { event_listener_t *new_listeners; @@ -1359,7 +1371,37 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event) } for(listener = listeners; listener < listeners + listeners_cnt; listener++) { - if(listener->type == LISTENER_TYPE_ATTACHED) { + if(listener->type != LISTENER_TYPE_ATTACHED) { + DISPID named_arg = DISPID_THIS; + VARIANTARG args[2]; + DISPPARAMS dp = {args, &named_arg, 2, 1}; + + 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 + ? (IDispatch*)event->event_obj : (IDispatch*)&event->IDOMEvent_iface; + V_VT(&v) = VT_EMPTY; + + TRACE("%s >>>\n", debugstr_w(event_info[event->event_id].name)); + hres = call_disp_func(listener->function, &dp, &v); + if(hres == S_OK) { + TRACE("%s <<< %s\n", debugstr_w(event_info[event->event_id].name), + debugstr_variant(&v)); + + if(cancelable) { + if(V_VT(&v) == VT_BOOL) { + if(!V_BOOL(&v)) + IDOMEvent_preventDefault(&event->IDOMEvent_iface); + }else if(V_VT(&v) != VT_EMPTY) { + FIXME("unhandled result %s\n", debugstr_variant(&v)); + } + } + VariantClear(&v); + }else { + WARN("%s <<< %08x\n", debugstr_w(event_info[event->event_id].name), hres); + } + }else { VARIANTARG arg; DISPPARAMS dp = {&arg, NULL, 1, 0}; @@ -1588,7 +1630,9 @@ 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--; event_obj->event->event_obj = NULL; } @@ -1826,7 +1870,10 @@ HRESULT attach_event(EventTarget *event_target, BSTR name, IDispatch *disp, VARI listener->type = LISTENER_TYPE_ATTACHED; IDispatch_AddRef(listener->function = disp); - list_add_head(&container->listeners, &listener->entry); + if(use_event_quirks(event_target)) + list_add_head(&container->listeners, &listener->entry); + else + list_add_tail(&container->listeners, &listener->entry); *res = VARIANT_TRUE; return S_OK; diff --git a/dlls/mshtml/htmlevent.h b/dlls/mshtml/htmlevent.h index 97d34af..eaf42eb 100644 --- a/dlls/mshtml/htmlevent.h +++ b/dlls/mshtml/htmlevent.h @@ -66,10 +66,11 @@ typedef struct { EventTarget *target; BOOL prevent_default; BOOL stop_propagation; - USHORT phase; + DOM_EVENT_PHASE phase; IHTMLEventObj *event_obj; BOOL no_event_obj; + unsigned in_fire_event; } DOMEvent; void check_event_attr(HTMLDocumentNode*,nsIDOMHTMLElement*) DECLSPEC_HIDDEN; diff --git a/dlls/mshtml/tests/events.c b/dlls/mshtml/tests/events.c index ac7ae43..91ec4b6 100644 --- a/dlls/mshtml/tests/events.c +++ b/dlls/mshtml/tests/events.c @@ -93,6 +93,7 @@ static IHTMLWindow2 *window; static IOleDocumentView *view; static BOOL is_ie9plus; static int document_mode; +static unsigned in_fire_event; typedef struct { LONG x; @@ -358,7 +359,9 @@ static void _elem_fire_event(unsigned line, IUnknown *unk, const char *event, VA b = 100; str = a2bstr(event); + in_fire_event++; hres = IHTMLElement3_fireEvent(elem3, str, evobj, &b); + in_fire_event--; SysFreeString(str); ok_(__FILE__,line)(hres == S_OK, "fireEvent failed: %08x\n", hres); ok_(__FILE__,line)(b == VARIANT_TRUE, "fireEvent returned %x\n", b); @@ -371,20 +374,51 @@ static void _test_event_args(unsigned line, const IID *dispiid, DISPID id, WORD ok_(__FILE__,line) (id == DISPID_VALUE, "id = %d\n", id); ok_(__FILE__,line) (wFlags == DISPATCH_METHOD, "wFlags = %x\n", wFlags); ok_(__FILE__,line) (pdp != NULL, "pdp == NULL\n"); - todo_wine_if(document_mode >= 9) ok_(__FILE__,line) (pdp->cArgs == (document_mode < 9 ? 1 : 2), "pdp->cArgs = %d\n", pdp->cArgs); ok_(__FILE__,line) (pdp->cNamedArgs == 1, "pdp->cNamedArgs = %d\n", pdp->cNamedArgs); ok_(__FILE__,line) (pdp->rgdispidNamedArgs[0] == DISPID_THIS, "pdp->rgdispidNamedArgs[0] = %d\n", pdp->rgdispidNamedArgs[0]); ok_(__FILE__,line) (V_VT(pdp->rgvarg) == VT_DISPATCH, "V_VT(rgvarg) = %d\n", V_VT(pdp->rgvarg)); if(pdp->cArgs > 1) - ok_(__FILE__,line) (V_VT(pdp->rgvarg+1) == VT_DISPATCH, "V_VT(rgvarg) = %d\n", V_VT(pdp->rgvarg)); + ok_(__FILE__,line) (V_VT(pdp->rgvarg+1) == VT_DISPATCH, "V_VT(rgvarg) = %d\n", V_VT(pdp->rgvarg+1)); ok_(__FILE__,line) (pvarRes != NULL, "pvarRes == NULL\n"); ok_(__FILE__,line) (pei != NULL, "pei == NULL"); ok_(__FILE__,line) (!pspCaller, "pspCaller != NULL\n"); if(dispiid) _test_disp(line, (IUnknown*)V_DISPATCH(pdp->rgvarg), dispiid); + + if(pdp->cArgs > 1) { + IHTMLEventObj *window_event, *event_obj; + IDOMEvent *event; + HRESULT hres; + + hres = IDispatch_QueryInterface(V_DISPATCH(pdp->rgvarg+1), &IID_IDOMEvent, (void**)&event); + if(in_fire_event) + ok(hres == E_NOINTERFACE, "QI(IID_IDOMEvent) returned %08x\n", hres); + else + ok(hres == S_OK, "Could not get IDOMEvent iface: %08x\n", hres); + + hres = IDispatch_QueryInterface(V_DISPATCH(pdp->rgvarg+1), &IID_IHTMLEventObj, (void**)&event_obj); + if(in_fire_event) + ok(hres == S_OK, "Could not get IDOMEventObj iface: %08x\n", hres); + else + ok(hres == E_NOINTERFACE, "QI(IID_IHTMLEventObj) returned %08x\n", hres); + + if(event) + IDOMEvent_Release(event); + if(event_obj) + IHTMLEventObj_Release(event_obj); + + hres = IHTMLWindow2_get_event(window, &window_event); + ok(hres == S_OK, "get_event failed: %08x\n", hres); + if(window_event) { + todo_wine_if(in_fire_event) + ok(!iface_cmp((IUnknown*)V_DISPATCH(pdp->rgvarg+1), (IUnknown*)window_event), + "window_event != event arg\n"); + IHTMLEventObj_Release(window_event); + } + } } #define test_attached_event_args(a,b,c,d,e) _test_attached_event_args(__LINE__,a,b,c,d,e) @@ -666,6 +700,7 @@ static void _test_event_srcfilter(unsigned line, IHTMLEventObj *event) static void _test_event_obj(unsigned line, const char *type, const xy_test_t *xy) { IHTMLEventObj *event = _get_event_obj(line); + IDOMEvent *dom_event; VARIANT v; HRESULT hres; @@ -699,6 +734,9 @@ static void _test_event_obj(unsigned line, const char *type, const xy_test_t *xy if(V_VT(&v) == VT_BOOL) ok_(__FILE__,line)(V_BOOL(&v) == VARIANT_TRUE, "V_BOOL(returnValue) = %x\n", V_BOOL(&v)); + hres = IHTMLEventObj_QueryInterface(event, &IID_IDOMEvent, (void**)&dom_event); + ok(hres == E_NOINTERFACE, "Could not get IDOMEvent iface: %08x\n", hres); + IHTMLEventObj_Release(event); }

1 0

Jacek Caban : mshtml: Store all event listeners in a list.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: 87d4a5a29a070375fcf9b4fba9a46e8263061498 URL: http://source.winehq.org/git/wine.git/?a=commit;h=87d4a5a29a070375fcf9b4fba… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Oct 25 18:12:56 2017 +0200 mshtml: Store all event listeners in a list. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 270 ++++++++++++++++++++++++++---------------------- 1 file changed, 147 insertions(+), 123 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=87d4a5a29a070375fcf9b…

1 0

Jacek Caban : mshtml: Store all attached listeners before calling them in call_event_handlers.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: 231d1d3c338b81f97a9165737783bf281c49fb4e URL: http://source.winehq.org/git/wine.git/?a=commit;h=231d1d3c338b81f97a9165737… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Oct 25 18:12:41 2017 +0200 mshtml: Store all attached listeners before calling them in call_event_handlers. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 99 ++++++++++++++++++++++++++++++++++--------------- 1 file changed, 70 insertions(+), 29 deletions(-) diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c index d64238f..19a76e0 100644 --- a/dlls/mshtml/htmlevent.c +++ b/dlls/mshtml/htmlevent.c @@ -35,6 +35,16 @@ WINE_DEFAULT_DEBUG_CHANNEL(mshtml); +typedef enum { + LISTENER_TYPE_ONEVENT, + LISTENER_TYPE_ATTACHED +} listener_type_t; + +typedef struct { + listener_type_t type; + IDispatch *function; +} event_listener_t; + typedef struct { struct wine_rb_entry entry; eventid_t event_id; @@ -1258,14 +1268,16 @@ static BOOL is_cp_event(cp_static_data_t *data, DISPID dispid) static void call_event_handlers(EventTarget *event_target, DOMEvent *event) { const eventid_t eid = event->event_id; - handler_vector_t *handler_vector = get_handler_vector(event_target, eid, FALSE); + handler_vector_t *container = get_handler_vector(event_target, eid, FALSE); const BOOL cancelable = event_info[eid].flags & EVENT_CANCELABLE; + event_listener_t *listener, listeners_buf[8], *listeners = listeners_buf; + unsigned listeners_cnt, listeners_size; ConnectionPointContainer *cp_container = NULL; const event_target_vtbl_t *vtbl; VARIANT v; HRESULT hres; - if(handler_vector && handler_vector->handler_prop) { + if(container && container->handler_prop) { DISPID named_arg = DISPID_THIS; VARIANTARG arg; DISPPARAMS dp = {&arg, &named_arg, 1, 1}; @@ -1278,7 +1290,7 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event) V_VT(&v) = VT_EMPTY; TRACE("%s >>>\n", debugstr_w(event_info[eid].name)); - hres = call_disp_func(handler_vector->handler_prop, &dp, &v); + hres = call_disp_func(container->handler_prop, &dp, &v); if(hres == S_OK) { TRACE("%s <<< %s\n", debugstr_w(event_info[eid].name), debugstr_variant(&v)); @@ -1296,40 +1308,69 @@ static void call_event_handlers(EventTarget *event_target, DOMEvent *event) } } - if(handler_vector && handler_vector->handler_cnt) { - VARIANTARG arg; - DISPPARAMS dp = {&arg, NULL, 1, 0}; - int i; - - V_VT(&arg) = VT_DISPATCH; - V_DISPATCH(&arg) = (IDispatch*)event->event_obj; + listeners_cnt = 0; + listeners_size = sizeof(listeners_buf)/sizeof(*listeners_buf); - i = handler_vector->handler_cnt; + if(container) { + unsigned i = container->handler_cnt; while(i--) { - if(handler_vector->handlers[i]) { - V_VT(&v) = VT_EMPTY; - - TRACE("%s [%d] >>>\n", debugstr_w(event_info[eid].name), i); - hres = call_disp_func(handler_vector->handlers[i], &dp, &v); - if(hres == S_OK) { - TRACE("%s [%d] <<<\n", debugstr_w(event_info[eid].name), i); - - if(cancelable) { - if(V_VT(&v) == VT_BOOL) { - if(!V_BOOL(&v)) - IDOMEvent_preventDefault(&event->IDOMEvent_iface); - }else if(V_VT(&v) != VT_EMPTY) { - FIXME("unhandled result %s\n", debugstr_variant(&v)); - } - } - VariantClear(&v); + if(!container->handlers[i]) + continue; + + if(listeners_cnt == listeners_size) { + event_listener_t *new_listeners; + if(listeners == listeners_buf) { + new_listeners = heap_alloc(listeners_size * 2 * sizeof(*new_listeners)); + if(!new_listeners) + break; + memcpy(new_listeners, listeners, listeners_cnt * sizeof(*listeners)); }else { - WARN("%s [%d] <<< %08x\n", debugstr_w(event_info[eid].name), i, hres); + new_listeners = heap_realloc(listeners, listeners_size * 2 * sizeof(*new_listeners)); } + listeners = new_listeners; + listeners_size *= 2; } + + listeners[listeners_cnt].type = LISTENER_TYPE_ATTACHED; + IDispatch_AddRef(listeners[listeners_cnt].function = container->handlers[i]); + listeners_cnt++; } } + for(listener = listeners; listener < listeners + listeners_cnt; listener++) { + if(listener->type == LISTENER_TYPE_ATTACHED) { + VARIANTARG arg; + DISPPARAMS dp = {&arg, NULL, 1, 0}; + + V_VT(&arg) = VT_DISPATCH; + V_DISPATCH(&arg) = (IDispatch*)event->event_obj; + V_VT(&v) = VT_EMPTY; + + TRACE("%s attached >>>\n", debugstr_w(event_info[eid].name)); + hres = call_disp_func(listener->function, &dp, &v); + if(hres == S_OK) { + TRACE("%s attached <<<\n", debugstr_w(event_info[eid].name)); + + if(cancelable) { + if(V_VT(&v) == VT_BOOL) { + if(!V_BOOL(&v)) + IDOMEvent_preventDefault(&event->IDOMEvent_iface); + }else if(V_VT(&v) != VT_EMPTY) { + FIXME("unhandled result %s\n", debugstr_variant(&v)); + } + } + VariantClear(&v); + }else { + WARN("%s attached <<< %08x\n", debugstr_w(event_info[eid].name), hres); + } + } + } + + if(listeners != listeners_buf) + heap_free(listeners); + if(event->phase == DEP_CAPTURING_PHASE) + return; + if((vtbl = dispex_get_vtbl(&event_target->dispex)) && vtbl->get_cp_container) cp_container = vtbl->get_cp_container(&event_target->dispex); if(cp_container) {

1 0

Alexandre Julliard : rbtree: Clarify the loop conditions to avoid confusing older gcc.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: 5d01ebab89cee8a3499ee00729c048068d5b719d URL: http://source.winehq.org/git/wine.git/?a=commit;h=5d01ebab89cee8a3499ee0072… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 25 18:41:10 2017 +0200 rbtree: Clarify the loop conditions to avoid confusing older gcc. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/wine/rbtree.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/wine/rbtree.h b/include/wine/rbtree.h index bb4edf5..dc50b5e 100644 --- a/include/wine/rbtree.h +++ b/include/wine/rbtree.h @@ -147,7 +147,7 @@ static inline struct wine_rb_entry *wine_rb_postorder_next(struct wine_rb_entry /* iterate through the tree using a tree entry */ #define WINE_RB_FOR_EACH_ENTRY(elem, tree, type, field) \ for ((elem) = WINE_RB_ENTRY_VALUE(wine_rb_head((tree)->root), type, field); \ - &(elem)->field; \ + (elem) != WINE_RB_ENTRY_VALUE(0, type, field); \ (elem) = WINE_RB_ENTRY_VALUE(wine_rb_next(&elem->field), type, field)) /* iterate through the tree using using postorder, making it safe to free the entry */ @@ -159,7 +159,7 @@ static inline struct wine_rb_entry *wine_rb_postorder_next(struct wine_rb_entry /* iterate through the tree using a tree entry and postorder, making it safe to free the entry */ #define WINE_RB_FOR_EACH_ENTRY_DESTRUCTOR(elem, elem2, tree, type, field) \ for ((elem) = WINE_RB_ENTRY_VALUE(wine_rb_postorder_head((tree)->root), type, field); \ - &(elem)->field \ + (elem) != WINE_RB_ENTRY_VALUE(0, type, field) \ && (((elem2) = WINE_RB_ENTRY_VALUE(wine_rb_postorder_next(&(elem)->field), type, field)) || 1); \ (elem) = (elem2))

1 0

André Hentschel : loader: Build the preloader for ARM64.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: 27e92e550c2b86d5ebd4966926c1fcde03838dcc URL: http://source.winehq.org/git/wine.git/?a=commit;h=27e92e550c2b86d5ebd496692… Author: André Hentschel <nerv(a)dawncrow.de> Date: Fri Oct 20 21:14:43 2017 +0200 loader: Build the preloader for ARM64. Signed-off-by: André Hentschel <nerv(a)dawncrow.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- configure | 2 +- configure.ac | 2 +- loader/preloader.c | 107 +++++++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 109 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 7a7d26a..c8a508b 100755 --- a/configure +++ b/configure @@ -15627,7 +15627,7 @@ esac case $host_os in linux*) case $host_cpu in - *i[3456789]86*|x86_64*) + *i[3456789]86*|x86_64*|*aarch64*) test "$wine_binary" = wine || wine_fn_append_file CONFIGURE_TARGETS "loader/wine-preloader" WINELOADER_PROGRAMS="$WINELOADER_PROGRAMS $wine_binary-preloader" ;; diff --git a/configure.ac b/configure.ac index cff4b8f..3039c94 100644 --- a/configure.ac +++ b/configure.ac @@ -2022,7 +2022,7 @@ esac case $host_os in linux*) case $host_cpu in - *i[[3456789]]86*|x86_64*) + *i[[3456789]]86*|x86_64*|*aarch64*) test "$wine_binary" = wine || WINE_IGNORE_FILE("loader/wine-preloader") WINELOADER_PROGRAMS="$WINELOADER_PROGRAMS $wine_binary-preloader" ;; diff --git a/loader/preloader.c b/loader/preloader.c index 5e6add7..5068a60 100644 --- a/loader/preloader.c +++ b/loader/preloader.c @@ -455,6 +455,110 @@ SYSCALL_NOERR( wld_geteuid, 107 /* SYS_geteuid */ ); gid_t wld_getegid(void); SYSCALL_NOERR( wld_getegid, 108 /* SYS_getegid */ ); +#elif defined(__aarch64__) + +/* + * The _start function is the entry and exit point of this program + * + * It calls wld_start, passing a pointer to the args it receives + * then jumps to the address wld_start returns. + */ +void _start(void); +extern char _end[]; +__ASM_GLOBAL_FUNC(_start, + "mov x0, SP\n\t" + "sub SP, SP, #144\n\t" /* allocate some space for extra aux values */ + "str x0, [SP]\n\t" /* orig stack pointer */ + "str x30, [SP, #8]\n\t" + "mov x0, SP\n\t" /* ptr to orig stack pointer */ + "bl wld_start\n\t" + "ldr x30, [SP, #8]\n\t" + "ldr x1, [SP]\n\t" /* new stack pointer */ + "mov SP, x1\n\t" + "mov x30, x0\n\t" + "mov x0, #0\n\t" + "mov x1, #0\n\t" + "mov x2, #0\n\t" + "mov x3, #0\n\t" + "mov x4, #0\n\t" + "mov x5, #0\n\t" + "mov x6, #0\n\t" + "mov x7, #0\n\t" + "mov x8, #0\n\t" + "mov x9, #0\n\t" + "mov x10, #0\n\t" + "mov x11, #0\n\t" + "mov x12, #0\n\t" + "mov x13, #0\n\t" + "mov x14, #0\n\t" + "mov x15, #0\n\t" + "mov x16, #0\n\t" + "mov x17, #0\n\t" + "mov x18, #0\n\t" + "ret") + +#define SYSCALL_FUNC( name, nr ) \ + __ASM_GLOBAL_FUNC( name, \ + "stp x8, x9, [SP, #-16]!\n\t" \ + "mov x8, #" #nr "\n\t" \ + "svc #0\n\t" \ + "ldp x8, x9, [SP], #16\n\t" \ + "cmn x0, #1, lsl#12\n\t" \ + "cinv x0, x0, hi\n\t" \ + "b.hi 1f\n\t" \ + "ret\n\t" \ + "1: mov x0, #-1\n\t" \ + "ret" ) + +#define SYSCALL_NOERR( name, nr ) \ + __ASM_GLOBAL_FUNC( name, \ + "stp x8, x9, [SP, #-16]!\n\t" \ + "mov x8, #" #nr "\n\t" \ + "svc #0\n\t" \ + "ldp x8, x9, [SP], #16\n\t" \ + "ret" ) + +void wld_exit( int code ) __attribute__((noreturn)); +SYSCALL_NOERR( wld_exit, 93 /* SYS_exit */ ); + +ssize_t wld_read( int fd, void *buffer, size_t len ); +SYSCALL_FUNC( wld_read, 63 /* SYS_read */ ); + +ssize_t wld_write( int fd, const void *buffer, size_t len ); +SYSCALL_FUNC( wld_write, 64 /* SYS_write */ ); + +int wld_openat( int dirfd, const char *name, int flags ); +SYSCALL_FUNC( wld_openat, 56 /* SYS_openat */ ); + +int wld_open( const char *name, int flags ) +{ + return wld_openat(-100 /* AT_FDCWD */, name, flags); +} + +int wld_close( int fd ); +SYSCALL_FUNC( wld_close, 57 /* SYS_close */ ); + +void *wld_mmap( void *start, size_t len, int prot, int flags, int fd, off_t offset ); +SYSCALL_FUNC( wld_mmap, 222 /* SYS_mmap */ ); + +int wld_mprotect( const void *addr, size_t len, int prot ); +SYSCALL_FUNC( wld_mprotect, 226 /* SYS_mprotect */ ); + +int wld_prctl( int code, long arg ); +SYSCALL_FUNC( wld_prctl, 167 /* SYS_prctl */ ); + +uid_t wld_getuid(void); +SYSCALL_NOERR( wld_getuid, 174 /* SYS_getuid */ ); + +gid_t wld_getgid(void); +SYSCALL_NOERR( wld_getgid, 176 /* SYS_getgid */ ); + +uid_t wld_geteuid(void); +SYSCALL_NOERR( wld_geteuid, 175 /* SYS_geteuid */ ); + +gid_t wld_getegid(void); +SYSCALL_NOERR( wld_getegid, 177 /* SYS_getegid */ ); + #else #error preloader not implemented for this CPU #endif @@ -725,6 +829,9 @@ static void map_so_lib( const char *name, struct wld_link_map *l) #elif defined(__x86_64__) if( header->e_machine != EM_X86_64 ) fatal_error("%s: not an x86-64 ELF binary... don't know how to load it\n", name ); +#elif defined(__aarch64__) + if( header->e_machine != EM_AARCH64 ) + fatal_error("%s: not an aarchs64 ELF binary... don't know how to load it\n", name ); #endif if (header->e_phnum > sizeof(loadcmds)/sizeof(loadcmds[0]))

1 0

Piotr Caban : msi: Store condition intermediate values in union.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: dcd92a168b349dcbb4c3f5a9e38bfe716f978911 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dcd92a168b349dcbb4c3f5a9e… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Tue Oct 24 15:02:56 2017 +0200 msi: Store condition intermediate values in union. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msi/cond.y | 232 ++++++++++++++++++++++++++------------------------------ 1 file changed, 109 insertions(+), 123 deletions(-) diff --git a/dlls/msi/cond.y b/dlls/msi/cond.y index 8bdaf84..712050c 100644 --- a/dlls/msi/cond.y +++ b/dlls/msi/cond.y @@ -58,6 +58,18 @@ struct cond_str { INT len; }; +struct value { + enum value_type { + VALUE_INTEGER, + VALUE_LITERAL, + VALUE_SYMBOL + } type; + union { + INT integer; + WCHAR *string; + } u; +}; + static LPWSTR COND_GetString( COND_input *info, const struct cond_str *str ); static LPWSTR COND_GetLiteral( COND_input *info, const struct cond_str *str ); static int cond_lex( void *COND_lval, COND_input *info); @@ -70,16 +82,6 @@ static void cond_free( void *ptr ); static INT compare_int( INT a, INT operator, INT b ); static INT compare_string( LPCWSTR a, INT operator, LPCWSTR b, BOOL convert ); -static INT compare_and_free_strings( LPWSTR a, INT op, LPWSTR b, BOOL convert ) -{ - INT r; - - r = compare_string( a, op, b, convert ); - cond_free( a ); - cond_free( b ); - return r; -} - static BOOL num_from_prop( LPCWSTR p, INT *val ) { INT ret = 0, sign = 1; @@ -104,6 +106,12 @@ static BOOL num_from_prop( LPCWSTR p, INT *val ) return TRUE; } +static void value_free( struct value val ) +{ + if (val.type != VALUE_INTEGER) + cond_free( val.u.string ); +} + %} %lex-param { COND_input *info } @@ -113,8 +121,7 @@ static BOOL num_from_prop( LPCWSTR p, INT *val ) %union { struct cond_str str; - LPWSTR string; - INT value; + struct value value; LPWSTR identifier; INT operator; BOOL bool; @@ -132,8 +139,7 @@ static BOOL num_from_prop( LPCWSTR p, INT *val ) %nonassoc COND_ERROR COND_EOF %type <bool> expression boolean_term boolean_factor -%type <value> value_i -%type <string> symbol_s value_s literal +%type <value> value %type <identifier> identifier %type <operator> operator @@ -189,64 +195,50 @@ boolean_term: boolean_factor: COND_NOT boolean_factor { - $$ = $2 ? 0 : 1; - } - | value_i - { - $$ = $1 ? 1 : 0; + $$ = !$2; } - | value_s + | value { - $$ = ($1 && $1[0]) ? 1 : 0; - cond_free( $1 ); - } - | value_i operator value_i - { - $$ = compare_int( $1, $2, $3 ); - } - | symbol_s operator value_i - { - int num; - if (num_from_prop( $1, &num )) - $$ = compare_int( num, $2, $3 ); - else - $$ = ($2 == COND_NE || $2 == COND_INE ); - cond_free( $1 ); - } - | value_i operator symbol_s - { - int num; - if (num_from_prop( $3, &num )) - $$ = compare_int( $1, $2, num ); - else - $$ = ($2 == COND_NE || $2 == COND_INE ); - cond_free( $3 ); - } - | symbol_s operator symbol_s - { - $$ = compare_and_free_strings( $1, $2, $3, TRUE ); - } - | symbol_s operator literal - { - $$ = compare_and_free_strings( $1, $2, $3, TRUE ); - } - | literal operator symbol_s - { - $$ = compare_and_free_strings( $1, $2, $3, TRUE ); - } - | literal operator literal - { - $$ = compare_and_free_strings( $1, $2, $3, FALSE ); - } - | literal operator value_i - { - $$ = 0; - cond_free( $1 ); + if ($1.type == VALUE_INTEGER) + $$ = $1.u.integer ? 1 : 0; + else + $$ = $1.u.string && $1.u.string[0]; + value_free( $1 ); } - | value_i operator literal + | value operator value { - $$ = 0; - cond_free( $3 ); + if ($1.type == VALUE_INTEGER && $3.type == VALUE_INTEGER) + { + $$ = compare_int($1.u.integer, $2, $3.u.integer); + } + else if ($1.type != VALUE_INTEGER && $3.type != VALUE_INTEGER) + { + $$ = compare_string($1.u.string, $2, $3.u.string, + $1.type == VALUE_SYMBOL || $3.type == VALUE_SYMBOL); + } + else if ($1.type == VALUE_LITERAL || $3.type == VALUE_LITERAL) + { + $$ = FALSE; + } + else if ($1.type == VALUE_SYMBOL) /* symbol operator integer */ + { + int num; + if (num_from_prop( $1.u.string, &num )) + $$ = compare_int( num, $2, $3.u.integer ); + else + $$ = ($2 == COND_NE || $2 == COND_INE ); + } + else /* integer operator symbol */ + { + int num; + if (num_from_prop( $3.u.string, &num )) + $$ = compare_int( $1.u.integer, $2, num ); + else + $$ = ($2 == COND_NE || $2 == COND_INE ); + } + + value_free( $1 ); + value_free( $3 ); } | COND_LPAR expression COND_RPAR { @@ -276,35 +268,52 @@ operator: | COND_IRHS { $$ = COND_IRHS; } ; -value_s: - symbol_s - { - $$ = $1; - } - | literal - { - $$ = $1; - } - ; +value: + identifier + { + COND_input* cond = (COND_input*) info; + UINT len; -literal: - COND_LITER + $$.type = VALUE_SYMBOL; + $$.u.string = msi_dup_property( cond->package->db, $1 ); + if ($$.u.string) + { + len = (lstrlenW($$.u.string) + 1) * sizeof (WCHAR); + $$.u.string = cond_track_mem( cond, $$.u.string, len ); + } + cond_free( $1 ); + } + | COND_PERCENT identifier { COND_input* cond = (COND_input*) info; - $$ = COND_GetLiteral( cond, &$1 ); - if( !$$ ) + UINT len = GetEnvironmentVariableW( $2, NULL, 0 ); + $$.type = VALUE_SYMBOL; + $$.u.string = NULL; + if (len++) + { + $$.u.string = cond_alloc( cond, len*sizeof (WCHAR) ); + if( !$$.u.string ) + YYABORT; + GetEnvironmentVariableW( $2, $$.u.string, len ); + } + cond_free( $2 ); + } + | COND_LITER + { + COND_input* cond = (COND_input*) info; + $$.type = VALUE_LITERAL; + $$.u.string = COND_GetLiteral( cond, &$1 ); + if( !$$.u.string ) YYABORT; } - ; - -value_i: - COND_NUMBER + | COND_NUMBER { COND_input* cond = (COND_input*) info; LPWSTR szNum = COND_GetString( cond, &$1 ); if( !szNum ) YYABORT; - $$ = atoiW( szNum ); + $$.type = VALUE_INTEGER; + $$.u.integer = atoiW( szNum ); cond_free( szNum ); } | COND_DOLLARS identifier @@ -313,7 +322,8 @@ value_i: INSTALLSTATE install = INSTALLSTATE_UNKNOWN, action = INSTALLSTATE_UNKNOWN; MSI_GetComponentStateW(cond->package, $2, &install, &action ); - $$ = action; + $$.type = VALUE_INTEGER; + $$.u.integer = action; cond_free( $2 ); } | COND_QUESTION identifier @@ -321,8 +331,9 @@ value_i: COND_input* cond = (COND_input*) info; INSTALLSTATE install = INSTALLSTATE_UNKNOWN, action = INSTALLSTATE_UNKNOWN; - MSI_GetComponentStateW(cond->package, $2, &install, &action ); - $$ = install; + MSI_GetComponentStateW(cond->package, $2, &install, &action );\ + $$.type = VALUE_INTEGER; + $$.u.integer = install; cond_free( $2 ); } | COND_AMPER identifier @@ -334,10 +345,14 @@ value_i: { FIXME("condition may be evaluated incorrectly\n"); /* we should return empty string in this case */ - $$ = MSICONDITION_FALSE; + $$.type = VALUE_INTEGER; + $$.u.integer = MSICONDITION_FALSE; } else - $$ = action; + { + $$.type = VALUE_INTEGER; + $$.u.integer = action; + } cond_free( $2 ); } @@ -347,37 +362,8 @@ value_i: INSTALLSTATE install = INSTALLSTATE_UNKNOWN, action = INSTALLSTATE_UNKNOWN; MSI_GetFeatureStateW(cond->package, $2, &install, &action ); - $$ = install; - cond_free( $2 ); - } - ; - -symbol_s: - identifier - { - COND_input* cond = (COND_input*) info; - UINT len; - - $$ = msi_dup_property( cond->package->db, $1 ); - if ($$) - { - len = (lstrlenW($$) + 1) * sizeof (WCHAR); - $$ = cond_track_mem( cond, $$, len ); - } - cond_free( $1 ); - } - | COND_PERCENT identifier - { - COND_input* cond = (COND_input*) info; - UINT len = GetEnvironmentVariableW( $2, NULL, 0 ); - $$ = NULL; - if (len++) - { - $$ = cond_alloc( cond, len*sizeof (WCHAR) ); - if( !$$ ) - YYABORT; - GetEnvironmentVariableW( $2, $$, len ); - } + $$.type = VALUE_INTEGER; + $$.u.integer = install; cond_free( $2 ); } ;

1 0

Jacek Caban : mshtml/tests: Added a few more event tests.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: b96934bf4a5b35a9917e34bee4cf887101031e56 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b96934bf4a5b35a9917e34bee… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Oct 25 18:13:07 2017 +0200 mshtml/tests: Added a few more event tests. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/tests/events.html | 89 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 89 insertions(+) diff --git a/dlls/mshtml/tests/events.html b/dlls/mshtml/tests/events.html index 4172e30..f20fd3b 100644 --- a/dlls/mshtml/tests/events.html +++ b/dlls/mshtml/tests/events.html @@ -190,6 +190,92 @@ function test_event_target() { div.click(); with(todo_wine) ok(last_event_arg.srcElement === null, "srcElement != null"); + + document.body.removeChild(div); +} + +function test_attach_event() { + var calls; + + var div = document.createElement("div"); + document.body.appendChild(div); + + function listener() { + calls += "listener,"; + } + + function listener2() { + calls += "listener2,"; + } + + /* if the same listener is added twice, it will be called twice */ + div.attachEvent("onclick", listener); + div.attachEvent("onclick", listener2); + div.attachEvent("onclick", listener); + + calls = ""; + div.click(); + ok(calls === "listener,listener2,listener,", "calls = " + calls); + + /* remove listener once, it will be called once */ + div.detachEvent("onclick", listener); + + calls = ""; + div.click(); + ok(calls === "listener2,listener,", "calls = " + calls); + + div.detachEvent("onclick", listener); + div.detachEvent("onclick", listener2); + + calls = ""; + div.click(); + ok(calls === "", "calls = " + calls); + + document.body.removeChild(div); +} + +function test_listener_order() { + var div = document.createElement("div"); + document.body.appendChild(div); + + var calls; + function record_call(msg) { + return function() { calls += msg + "," }; + } + + div.attachEvent("onclick", record_call("click1")); + div.onclick = record_call("onclick"); + div.attachEvent("onclick", record_call("click2")); + div.attachEvent("onclick", record_call("click3")); + + calls = ""; + div.click(); + ok(calls === "onclick,click3,click2,click1,", "calls = " + calls); + + document.body.removeChild(div); +} + +function test_attach_in_attach() { + var calls; + + var div = document.createElement("div"); + document.body.appendChild(div); + + /* listener attached inside onevent handler will be invoked in this propagation */ + div.onclick = function () { + calls += "div.onclick,"; + div.attachEvent("onclick", function() { + calls += "div.click,"; + /* listener attached inside an other attached listener will not ve invoked */ + div.attachEvent("onclick", function () { ok(false, "unexpected call"); }); + }); + } + + calls = ""; + div.click(); + ok(calls === "div.onclick,div.click,", "calls = " + calls); + + document.body.removeChild(div); } window.onload = function() { @@ -215,6 +301,9 @@ window.onload = function() { test_string_event_handler(); test_body_events(); test_event_target(); + test_attach_event(); + test_listener_order(); + test_attach_in_attach(); }catch(e) { ok(false, "Got an exception: " + e.message); }

1 0

Zebediah Figura : mfplat/tests: Skip MFTRegister() tests if permissions are too low.
by Alexandre Julliard Oct. 25, 2017

Oct. 25, 2017

Module: wine Branch: master Commit: 87b41b7369ba2fcd8bd6ad91ee06acbfdc86fc12 URL: http://source.winehq.org/git/wine.git/?a=commit;h=87b41b7369ba2fcd8bd6ad91e… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Tue Oct 24 16:13:55 2017 -0500 mfplat/tests: Skip MFTRegister() tests if permissions are too low. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mfplat/tests/mfplat.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/mfplat/tests/mfplat.c b/dlls/mfplat/tests/mfplat.c index 13853c5..f85b00c 100644 --- a/dlls/mfplat/tests/mfplat.c +++ b/dlls/mfplat/tests/mfplat.c @@ -73,9 +73,12 @@ static void test_register(void) HRESULT ret; ret = MFTRegister(DUMMY_CLSID, MFT_CATEGORY_OTHER, name, 0, 1, input, 1, output, NULL); - ok(ret == S_OK, "Failed to register dummy filter: %x\n", ret); - if (FAILED(ret)) + if (ret == E_ACCESSDENIED) + { + win_skip("Not enough permissions to register a filter\n"); return; + } + ok(ret == S_OK, "Failed to register dummy filter: %x\n", ret); if(0) {

1 0