April 2021 - Wine-commits - list.winehq.org

Nikolay Sivov : mfplay: Add IMFPMediaPlayer stub.
by Alexandre Julliard 02 Apr '21

02 Apr '21

Module: wine Branch: master Commit: 615cabf69ae73b232de93a85cf14e8f6f27e6291 URL: https://source.winehq.org/git/wine.git/?a=commit;h=615cabf69ae73b232de93a85… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Apr 1 23:22:57 2021 +0300 mfplay: Add IMFPMediaPlayer stub. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mfplay/Makefile.in | 1 + dlls/mfplay/player.c | 372 +++++++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 371 insertions(+), 2 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=615cabf69ae73b232de9…

1 0

Francois Gouget : testbot/web: Fix Apache intermittently not finding CreatePatches().
by Alexandre Julliard 02 Apr '21

02 Apr '21

Module: tools Branch: master Commit: 50ecc7020fc26c2117c3499c5e9256688ec7a933 URL: https://source.winehq.org/git/tools.git/?a=commit;h=50ecc7020fc26c2117c3499… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Thu Apr 1 15:52:34 2021 +0200 testbot/web: Fix Apache intermittently not finding CreatePatches(). If CreatePatches() is not refered to with its full name Apache (mod_perl really) intermittently fails to find it and issues an error of the form: Undefined subroutine &WineTestBot::Jobs::CreatePatches called at .../testbot/lib/ObjectModel/Item.pm line 275. Strangely enough it never has trouble finding any of the other Create*() functions. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/lib/WineTestBot/Jobs.pm | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm index 5dbc1a7..a5ae135 100644 --- a/testbot/lib/WineTestBot/Jobs.pm +++ b/testbot/lib/WineTestBot/Jobs.pm @@ -454,7 +454,10 @@ my @PropertyDescriptors = ( CreateBasicPropertyDescriptor("Remarks", "Remarks", !1, !1, "A", 128), CreateBasicPropertyDescriptor("Submitted", "Submitted", !1, !1, "DT", 19), CreateBasicPropertyDescriptor("Ended", "Ended", !1, !1, "DT", 19), - CreateItemrefPropertyDescriptor("Patch", "Submitted from patch", !1, !1, \&CreatePatches, ["PatchId"]), + # Somehow mod_perl sometimes fails to find CreatePatches() if not given the + # fully qualified name, but never has any trouble with the other Create*() + # functions. + CreateItemrefPropertyDescriptor("Patch", "Submitted from patch", !1, !1, \&WineTestBot::Patches::CreatePatches, ["PatchId"]), CreateDetailrefPropertyDescriptor("Steps", "Steps", !1, !1, \&CreateSteps), ); SetDetailrefKeyPrefix("Job", @PropertyDescriptors);

1 0

Francois Gouget : testbot/PatchUtils: Rework GetPatchImpacts() to fix patchset handling.
by Alexandre Julliard 02 Apr '21

02 Apr '21

Module: tools Branch: master Commit: bcf62d46c35d3583617cc51797f070aaf915cb2c URL: https://source.winehq.org/git/tools.git/?a=commit;h=bcf62d46c35d3583617cc51… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Fri Apr 2 03:27:11 2021 +0200 testbot/PatchUtils: Rework GetPatchImpacts() to fix patchset handling. Rename the Tests set to Modules since it contains information about each module impacted by the patch, including a full list of the still existing and patched test units. Better separate the build impacts from the test impacts: the former impact the build of all following parts, while only the last part impacts the tests that need to be run. This stops the TestBot from running test units which were not impacted by the _last part_ of the patchset. Document the returned $Impacts structure. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/bin/WineRunBuild.pl | 6 +- testbot/bin/build/WineTest.pl | 30 +-- testbot/lib/WineTestBot/PatchUtils.pm | 370 ++++++++++++++++++++-------------- testbot/lib/WineTestBot/Patches.pm | 15 +- testbot/web/Submit.pl | 36 ++-- 5 files changed, 266 insertions(+), 191 deletions(-) Diff: https://source.winehq.org/git/tools.git/?a=commitdiff;h=bcf62d46c35d3583617…

1 0

Hugh McMaster : reg: Better indicate when a registry value is not found.
by Alexandre Julliard 01 Apr '21

01 Apr '21

1 0

Jacek Caban : jscript: Get dispid and call invoke in a single step when setting a member property.
by Alexandre Julliard 01 Apr '21

01 Apr '21

Module: wine Branch: master Commit: f568b48e66dea7198afab67b81aa45bbeb575fc6 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f568b48e66dea7198afab67b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Apr 1 18:18:31 2021 +0200 jscript: Get dispid and call invoke in a single step when setting a member property. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/compile.c | 89 ++++++++++++++++++++++------------------------ dlls/jscript/dispex.c | 32 +++++++++++++++++ dlls/jscript/engine.c | 54 ++++++++++++++++++++++++++++ dlls/jscript/engine.h | 2 ++ dlls/jscript/jscript.h | 1 + dlls/jscript/tests/lang.js | 6 ++++ dlls/jscript/tests/run.c | 19 ++++++++++ 7 files changed, 156 insertions(+), 47 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=f568b48e66dea7198afa…

1 0

Jacek Caban : jscript: Support non-extensible objects.
by Alexandre Julliard 01 Apr '21

01 Apr '21

Module: wine Branch: master Commit: 7750753758693cbd09b978dfb53557c3b0715335 URL: https://source.winehq.org/git/wine.git/?a=commit;h=7750753758693cbd09b978df… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Apr 1 18:18:40 2021 +0200 jscript: Support non-extensible objects. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/dispex.c | 12 +++++++++--- dlls/jscript/jscript.h | 1 + dlls/jscript/object.c | 3 ++- dlls/mshtml/tests/es5.js | 8 +++++++- 4 files changed, 19 insertions(+), 5 deletions(-) diff --git a/dlls/jscript/dispex.c b/dlls/jscript/dispex.c index 0f0907063f8..303e5bb8c99 100644 --- a/dlls/jscript/dispex.c +++ b/dlls/jscript/dispex.c @@ -474,6 +474,8 @@ static HRESULT prop_put(jsdisp_t *This, dispex_prop_t *prop, jsval_t val) return prop->u.p->setter(This->ctx, This, val); case PROP_PROTREF: case PROP_DELETED: + if(!This->extensible) + return S_OK; prop->type = PROP_JSVAL; prop->flags = PROPF_ENUMERABLE | PROPF_CONFIGURABLE | PROPF_WRITABLE; prop->u.val = jsval_undefined(); @@ -1747,6 +1749,7 @@ HRESULT init_dispex(jsdisp_t *dispex, script_ctx_t *ctx, const builtin_info_t *b dispex->IDispatchEx_iface.lpVtbl = &DispatchExVtbl; dispex->ref = 1; dispex->builtin_info = builtin_info; + dispex->extensible = TRUE; dispex->props = heap_alloc_zero(sizeof(dispex_prop_t)*(dispex->buf_size=4)); if(!dispex->props) @@ -1894,7 +1897,7 @@ HRESULT jsdisp_get_id(jsdisp_t *jsdisp, const WCHAR *name, DWORD flags, DISPID * dispex_prop_t *prop; HRESULT hres; - if(flags & fdexNameEnsure) + if(jsdisp->extensible && (flags & fdexNameEnsure)) hres = ensure_prop_name(jsdisp, name, PROPF_ENUMERABLE | PROPF_CONFIGURABLE | PROPF_WRITABLE, &prop); else @@ -2147,8 +2150,11 @@ HRESULT jsdisp_propput(jsdisp_t *obj, const WCHAR *name, DWORD flags, jsval_t va dispex_prop_t *prop; HRESULT hres; - hres = ensure_prop_name(obj, name, flags, &prop); - if(FAILED(hres)) + if(obj->extensible) + hres = ensure_prop_name(obj, name, flags, &prop); + else + hres = find_prop_name_prot(obj, string_hash(name), name, &prop); + if(FAILED(hres) || !prop) return hres; return prop_put(obj, prop, val); diff --git a/dlls/jscript/jscript.h b/dlls/jscript/jscript.h index c8ec7622caf..f27f660d802 100644 --- a/dlls/jscript/jscript.h +++ b/dlls/jscript/jscript.h @@ -247,6 +247,7 @@ struct jsdisp_t { DWORD prop_cnt; dispex_prop_t *props; script_ctx_t *ctx; + BOOL extensible; jsdisp_t *prototype; diff --git a/dlls/jscript/object.c b/dlls/jscript/object.c index 1e94f163446..da91b744128 100644 --- a/dlls/jscript/object.c +++ b/dlls/jscript/object.c @@ -680,7 +680,7 @@ static HRESULT Object_preventExtensions(script_ctx_t *ctx, vdisp_t *jsthis, WORD return E_NOTIMPL; } - FIXME("(%s) semi-stub\n", debugstr_jsval(argv[0])); + TRACE("(%s)\n", debugstr_jsval(argv[0])); obj = to_jsdisp(get_object(argv[0])); if(!obj) { @@ -688,6 +688,7 @@ static HRESULT Object_preventExtensions(script_ctx_t *ctx, vdisp_t *jsthis, WORD return E_NOTIMPL; } + obj->extensible = FALSE; if(r) *r = jsval_obj(jsdisp_addref(obj)); return S_OK; } diff --git a/dlls/mshtml/tests/es5.js b/dlls/mshtml/tests/es5.js index e8c523b0bf7..3eccc38fada 100644 --- a/dlls/mshtml/tests/es5.js +++ b/dlls/mshtml/tests/es5.js @@ -947,7 +947,6 @@ sync_test("preventExtensions", function() { var r = Object.preventExtensions(o); ok(r === o, "r != o"); o.x = 1; - todo_wine. ok(!("x" in o), "x property added to o"); try { Object.defineProperty(o, "y", { value: true }); @@ -960,6 +959,13 @@ sync_test("preventExtensions", function() { r = Object.preventExtensions(o); ok(r === o, "r != o"); + function Constr() {} + o = Object.preventExtensions(new Constr()); + Constr.prototype.prop = 1; + ok(o.prop === 1, "o.prop = " + o.prop); + o.prop = 2; + ok(o.prop === 1, "o.prop = " + o.prop); + ok(Object.preventExtensions.length === 1, "Object.preventExtensions.length = " + Object.preventExtensions.length); });

1 0

Jacek Caban : jscript: Add Object.isExtensible implementation.
by Alexandre Julliard 01 Apr '21

01 Apr '21

Module: wine Branch: master Commit: 7dadcaf81904265ea76055cb12fe7577922e8c36 URL: https://source.winehq.org/git/wine.git/?a=commit;h=7dadcaf81904265ea76055cb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Apr 1 18:18:47 2021 +0200 jscript: Add Object.isExtensible implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/object.c | 22 ++++++++++++++++++++++ dlls/mshtml/tests/es5.js | 15 +++++++++++++++ 2 files changed, 37 insertions(+) diff --git a/dlls/jscript/object.c b/dlls/jscript/object.c index da91b744128..8ff040853ef 100644 --- a/dlls/jscript/object.c +++ b/dlls/jscript/object.c @@ -693,12 +693,34 @@ static HRESULT Object_preventExtensions(script_ctx_t *ctx, vdisp_t *jsthis, WORD return S_OK; } +static HRESULT Object_isExtensible(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, unsigned argc, jsval_t *argv, jsval_t *r) +{ + jsdisp_t *obj; + + if(!argc || !is_object_instance(argv[0]) || !get_object(argv[0])) { + WARN("argument is not an object\n"); + return JS_E_OBJECT_EXPECTED; + } + + TRACE("(%s)\n", debugstr_jsval(argv[0])); + + obj = to_jsdisp(get_object(argv[0])); + if(!obj) { + FIXME("Non-JS object\n"); + return E_NOTIMPL; + } + + if(r) *r = jsval_bool(obj->extensible); + return S_OK; +} + static const builtin_prop_t ObjectConstr_props[] = { {L"create", Object_create, PROPF_ES5|PROPF_METHOD|2}, {L"defineProperties", Object_defineProperties, PROPF_ES5|PROPF_METHOD|2}, {L"defineProperty", Object_defineProperty, PROPF_ES5|PROPF_METHOD|2}, {L"getOwnPropertyDescriptor", Object_getOwnPropertyDescriptor, PROPF_ES5|PROPF_METHOD|2}, {L"getPrototypeOf", Object_getPrototypeOf, PROPF_ES5|PROPF_METHOD|1}, + {L"isExtensible", Object_isExtensible, PROPF_ES5|PROPF_METHOD|1}, {L"keys", Object_keys, PROPF_ES5|PROPF_METHOD|1}, {L"preventExtensions", Object_preventExtensions, PROPF_ES5|PROPF_METHOD|1}, }; diff --git a/dlls/mshtml/tests/es5.js b/dlls/mshtml/tests/es5.js index 3eccc38fada..8d38070b387 100644 --- a/dlls/mshtml/tests/es5.js +++ b/dlls/mshtml/tests/es5.js @@ -958,6 +958,8 @@ sync_test("preventExtensions", function() { r = Object.preventExtensions(o); ok(r === o, "r != o"); + r = Object.isExtensible(o); + ok(r === false, "isExtensible(o) returned " + r); function Constr() {} o = Object.preventExtensions(new Constr()); @@ -965,8 +967,21 @@ sync_test("preventExtensions", function() { ok(o.prop === 1, "o.prop = " + o.prop); o.prop = 2; ok(o.prop === 1, "o.prop = " + o.prop); + r = Object.isExtensible(o); + ok(r === false, "isExtensible(o) returned " + r); + + r = Object.isExtensible({}); + ok(r === true, "isExtensible(o) returned " + r); + + try { + Object.isExtensible(1); + ok(false, "exception expected"); + }catch(e) { + ok(e.name === "TypeError", "got " + e.name + " exception"); + } ok(Object.preventExtensions.length === 1, "Object.preventExtensions.length = " + Object.preventExtensions.length); + ok(Object.isExtensible.length === 1, "Object.isExtensible.length = " + Object.isExtensible.length); }); sync_test("head_setter", function() {

1 0

Jacek Caban : jscript: Support non-extensible objects in jsdisp_define_property.
by Alexandre Julliard 01 Apr '21

01 Apr '21

1 0

Jacek Caban : jscript: Add Object.freeze implementation.
by Alexandre Julliard 01 Apr '21

01 Apr '21

Module: wine Branch: master Commit: 0d3634b3ffb1fe4d58a883244c9ab74b370f8aa1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0d3634b3ffb1fe4d58a88324… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Apr 1 18:19:09 2021 +0200 jscript: Add Object.freeze implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/dispex.c | 13 +++++++++++++ dlls/jscript/jscript.h | 1 + dlls/jscript/object.c | 24 ++++++++++++++++++++++++ dlls/mshtml/tests/es5.js | 41 +++++++++++++++++++++++++++++++++++++++++ 4 files changed, 79 insertions(+) diff --git a/dlls/jscript/dispex.c b/dlls/jscript/dispex.c index 9b11f607791..57f5d09daee 100644 --- a/dlls/jscript/dispex.c +++ b/dlls/jscript/dispex.c @@ -2628,6 +2628,19 @@ HRESULT jsdisp_define_data_property(jsdisp_t *obj, const WCHAR *name, unsigned f return jsdisp_define_property(obj, name, &prop_desc); } +void jsdisp_freeze(jsdisp_t *obj) +{ + unsigned int i; + + for(i = 0; i < obj->prop_cnt; i++) { + if(obj->props[i].type == PROP_JSVAL) + obj->props[i].flags &= ~PROPF_WRITABLE; + obj->props[i].flags &= ~PROPF_CONFIGURABLE; + } + + obj->extensible = FALSE; +} + HRESULT jsdisp_get_prop_name(jsdisp_t *obj, DISPID id, jsstr_t **r) { dispex_prop_t *prop = get_prop(obj, id); diff --git a/dlls/jscript/jscript.h b/dlls/jscript/jscript.h index 750158aabd9..d4ebb3246dd 100644 --- a/dlls/jscript/jscript.h +++ b/dlls/jscript/jscript.h @@ -317,6 +317,7 @@ HRESULT jsdisp_define_property(jsdisp_t*,const WCHAR*,property_desc_t*) DECLSPEC HRESULT jsdisp_define_data_property(jsdisp_t*,const WCHAR*,unsigned,jsval_t) DECLSPEC_HIDDEN; HRESULT jsdisp_next_prop(jsdisp_t*,DISPID,BOOL,DISPID*) DECLSPEC_HIDDEN; HRESULT jsdisp_get_prop_name(jsdisp_t*,DISPID,jsstr_t**); +void jsdisp_freeze(jsdisp_t*) DECLSPEC_HIDDEN; HRESULT create_builtin_function(script_ctx_t*,builtin_invoke_t,const WCHAR*,const builtin_info_t*,DWORD, jsdisp_t*,jsdisp_t**) DECLSPEC_HIDDEN; diff --git a/dlls/jscript/object.c b/dlls/jscript/object.c index 8ff040853ef..3dcfd297a9b 100644 --- a/dlls/jscript/object.c +++ b/dlls/jscript/object.c @@ -693,6 +693,29 @@ static HRESULT Object_preventExtensions(script_ctx_t *ctx, vdisp_t *jsthis, WORD return S_OK; } +static HRESULT Object_freeze(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, unsigned argc, + jsval_t *argv, jsval_t *r) +{ + jsdisp_t *obj; + + if(!argc || !is_object_instance(argv[0]) || !get_object(argv[0])) { + WARN("argument is not an object\n"); + return JS_E_OBJECT_EXPECTED; + } + + TRACE("(%s)\n", debugstr_jsval(argv[0])); + + obj = to_jsdisp(get_object(argv[0])); + if(!obj) { + FIXME("Non-JS object\n"); + return E_NOTIMPL; + } + + jsdisp_freeze(obj); + if(r) *r = jsval_obj(jsdisp_addref(obj)); + return S_OK; +} + static HRESULT Object_isExtensible(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, unsigned argc, jsval_t *argv, jsval_t *r) { jsdisp_t *obj; @@ -718,6 +741,7 @@ static const builtin_prop_t ObjectConstr_props[] = { {L"create", Object_create, PROPF_ES5|PROPF_METHOD|2}, {L"defineProperties", Object_defineProperties, PROPF_ES5|PROPF_METHOD|2}, {L"defineProperty", Object_defineProperty, PROPF_ES5|PROPF_METHOD|2}, + {L"freeze", Object_freeze, PROPF_ES5|PROPF_METHOD|1}, {L"getOwnPropertyDescriptor", Object_getOwnPropertyDescriptor, PROPF_ES5|PROPF_METHOD|2}, {L"getPrototypeOf", Object_getPrototypeOf, PROPF_ES5|PROPF_METHOD|1}, {L"isExtensible", Object_isExtensible, PROPF_ES5|PROPF_METHOD|1}, diff --git a/dlls/mshtml/tests/es5.js b/dlls/mshtml/tests/es5.js index 5851278e983..ae73d435456 100644 --- a/dlls/mshtml/tests/es5.js +++ b/dlls/mshtml/tests/es5.js @@ -983,6 +983,47 @@ sync_test("preventExtensions", function() { ok(Object.isExtensible.length === 1, "Object.isExtensible.length = " + Object.isExtensible.length); }); +sync_test("freeze", function() { + ok(Object.freeze.length === 1, "Object.freeze.length = " + Object.freeze.length); + try { + Object.freeze(1); + ok(false, "exception expected"); + }catch(e) { + ok(e.name === "TypeError", "got " + e.name + " exception"); + } + + function f() {} + + var o = {}, r; + o.prop = 1; + o.func = f; + Object.defineProperty(o, "accprop", { + get: function() { + return r; + }, + set: function(v) { + r = v; + } + }); + + test_own_data_prop_desc(o, "prop", true, true, true); + r = Object.freeze(o); + ok(r === o, "r != o"); + test_own_data_prop_desc(o, "prop", false, true, false); + test_own_data_prop_desc(o, "func", false, true, false); + ok(!Object.isExtensible(o), "o is still extensible"); + o.prop = false; + o.func = false; + ok(o.prop === 1, "o.prop = " + o.prop); + ok(o.func === f, "o.func = " + o.func); + + r = 1; + o.accprop = 2; + ok(r === 2, "r = " + r); + r = 3; + ok(o.accprop === 3, "o.accprop = " + o.accprop); +}); + sync_test("head_setter", function() { document.head = ""; ok(typeof(document.head) === "object", "typeof(document.head) = " + typeof(document.head));

1 0

Jacek Caban : jscript: Add Object.seal implementation.
by Alexandre Julliard 01 Apr '21

01 Apr '21

Module: wine Branch: master Commit: 51742d80cd6d4ca9d07aab4f76abd8da0898b4d4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=51742d80cd6d4ca9d07aab4f… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Apr 1 18:19:16 2021 +0200 jscript: Add Object.seal implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/dispex.c | 4 ++-- dlls/jscript/jscript.h | 2 +- dlls/jscript/object.c | 26 +++++++++++++++++++++++++- dlls/mshtml/tests/es5.js | 41 +++++++++++++++++++++++++++++++++++++++++ 4 files changed, 69 insertions(+), 4 deletions(-) diff --git a/dlls/jscript/dispex.c b/dlls/jscript/dispex.c index 57f5d09daee..ca404f92fe5 100644 --- a/dlls/jscript/dispex.c +++ b/dlls/jscript/dispex.c @@ -2628,12 +2628,12 @@ HRESULT jsdisp_define_data_property(jsdisp_t *obj, const WCHAR *name, unsigned f return jsdisp_define_property(obj, name, &prop_desc); } -void jsdisp_freeze(jsdisp_t *obj) +void jsdisp_freeze(jsdisp_t *obj, BOOL seal) { unsigned int i; for(i = 0; i < obj->prop_cnt; i++) { - if(obj->props[i].type == PROP_JSVAL) + if(!seal && obj->props[i].type == PROP_JSVAL) obj->props[i].flags &= ~PROPF_WRITABLE; obj->props[i].flags &= ~PROPF_CONFIGURABLE; } diff --git a/dlls/jscript/jscript.h b/dlls/jscript/jscript.h index d4ebb3246dd..3999f04a970 100644 --- a/dlls/jscript/jscript.h +++ b/dlls/jscript/jscript.h @@ -317,7 +317,7 @@ HRESULT jsdisp_define_property(jsdisp_t*,const WCHAR*,property_desc_t*) DECLSPEC HRESULT jsdisp_define_data_property(jsdisp_t*,const WCHAR*,unsigned,jsval_t) DECLSPEC_HIDDEN; HRESULT jsdisp_next_prop(jsdisp_t*,DISPID,BOOL,DISPID*) DECLSPEC_HIDDEN; HRESULT jsdisp_get_prop_name(jsdisp_t*,DISPID,jsstr_t**); -void jsdisp_freeze(jsdisp_t*) DECLSPEC_HIDDEN; +void jsdisp_freeze(jsdisp_t*,BOOL) DECLSPEC_HIDDEN; HRESULT create_builtin_function(script_ctx_t*,builtin_invoke_t,const WCHAR*,const builtin_info_t*,DWORD, jsdisp_t*,jsdisp_t**) DECLSPEC_HIDDEN; diff --git a/dlls/jscript/object.c b/dlls/jscript/object.c index 3dcfd297a9b..86c656a719a 100644 --- a/dlls/jscript/object.c +++ b/dlls/jscript/object.c @@ -711,7 +711,30 @@ static HRESULT Object_freeze(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, uns return E_NOTIMPL; } - jsdisp_freeze(obj); + jsdisp_freeze(obj, FALSE); + if(r) *r = jsval_obj(jsdisp_addref(obj)); + return S_OK; +} + +static HRESULT Object_seal(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, unsigned argc, + jsval_t *argv, jsval_t *r) +{ + jsdisp_t *obj; + + if(!argc || !is_object_instance(argv[0]) || !get_object(argv[0])) { + WARN("argument is not an object\n"); + return JS_E_OBJECT_EXPECTED; + } + + TRACE("(%s)\n", debugstr_jsval(argv[0])); + + obj = to_jsdisp(get_object(argv[0])); + if(!obj) { + FIXME("Non-JS object\n"); + return E_NOTIMPL; + } + + jsdisp_freeze(obj, TRUE); if(r) *r = jsval_obj(jsdisp_addref(obj)); return S_OK; } @@ -747,6 +770,7 @@ static const builtin_prop_t ObjectConstr_props[] = { {L"isExtensible", Object_isExtensible, PROPF_ES5|PROPF_METHOD|1}, {L"keys", Object_keys, PROPF_ES5|PROPF_METHOD|1}, {L"preventExtensions", Object_preventExtensions, PROPF_ES5|PROPF_METHOD|1}, + {L"seal", Object_seal, PROPF_ES5|PROPF_METHOD|1}, }; static const builtin_info_t ObjectConstr_info = { diff --git a/dlls/mshtml/tests/es5.js b/dlls/mshtml/tests/es5.js index ae73d435456..6099c4090f3 100644 --- a/dlls/mshtml/tests/es5.js +++ b/dlls/mshtml/tests/es5.js @@ -1024,6 +1024,47 @@ sync_test("freeze", function() { ok(o.accprop === 3, "o.accprop = " + o.accprop); }); +sync_test("seal", function() { + ok(Object.seal.length === 1, "Object.seal.length = " + Object.seal.length); + try { + Object.seal(1); + ok(false, "exception expected"); + }catch(e) { + ok(e.name === "TypeError", "got " + e.name + " exception"); + } + + function f() {} + + var o = {}, r; + o.prop = 1; + o.func = f; + Object.defineProperty(o, "accprop", { + get: function() { + return r; + }, + set: function(v) { + r = v; + } + }); + + test_own_data_prop_desc(o, "prop", true, true, true); + r = Object.seal(o); + ok(r === o, "r != o"); + test_own_data_prop_desc(o, "prop", true, true, false); + test_own_data_prop_desc(o, "func", true, true, false); + ok(!Object.isExtensible(o), "o is still extensible"); + o.prop = false; + o.func = false; + ok(o.prop === false, "o.prop = " + o.prop); + ok(o.func === false, "o.func = " + o.func); + + r = 1; + o.accprop = 2; + ok(r === 2, "r = " + r); + r = 3; + ok(o.accprop === 3, "o.accprop = " + o.accprop); +}); + sync_test("head_setter", function() { document.head = ""; ok(typeof(document.head) === "object", "typeof(document.head) = " + typeof(document.head));

1 0