mshtml: Fixed builtin getter call with arguments.

This commit is contained in:
Jacek Caban 2013-04-01 11:50:29 +02:00 committed by Alexandre Julliard
parent 08254df316
commit 8c0c09e4c7
2 changed files with 4 additions and 2 deletions

View File

@ -1216,7 +1216,7 @@ static HRESULT invoke_builtin_prop(DispatchEx *This, DISPID id, LCID lcid, WORD
if(FAILED(hres)) if(FAILED(hres))
return hres; return hres;
if(flags != (DISPATCH_PROPERTYGET|DISPATCH_METHOD)) { if(flags != (DISPATCH_PROPERTYGET|DISPATCH_METHOD) || dp->cArgs) {
if(V_VT(&v) != VT_DISPATCH) { if(V_VT(&v) != VT_DISPATCH) {
FIXME("Not a function %s\n", debugstr_variant(&v)); FIXME("Not a function %s\n", debugstr_variant(&v));
VariantClear(&v); VariantClear(&v);
@ -1228,7 +1228,7 @@ static HRESULT invoke_builtin_prop(DispatchEx *This, DISPID id, LCID lcid, WORD
}else if(res) { }else if(res) {
*res = v; *res = v;
}else { }else {
IDispatch_Release(V_DISPATCH(&v)); VariantClear(&v);
} }
} }
} }

View File

@ -137,6 +137,8 @@ function test_getter_call() {
e.onmousedown = function(x) { this.onmousedown_called = x; }; e.onmousedown = function(x) { this.onmousedown_called = x; };
e.onmousedown("test"); e.onmousedown("test");
ok(e.onmousedown_called === "test", "e.onmousedown_called = " + e.onmousedown_called); ok(e.onmousedown_called === "test", "e.onmousedown_called = " + e.onmousedown_called);
ok(document.all("divid").tagName === "DIV", "document.all('divid').tagName = " + document.all("divid").tagName);
} }
function test_arg_conv() { function test_arg_conv() {