super should be available in object literals

• Component: New Bugs
• Status: RESOLVED FIXED

[Geoffrey Garen]

super should be available in object literals

[Geoffrey Garen]

Created attachment 277107 [details]
Patch

[Saam Barati]

Comment on attachment 277107 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=277107&action=review

r=me with comments.

> Source/JavaScriptCore/ChangeLog:48
> +        about the list of places you can use super.

I agree. I wish our parser added the line/column to the "super not allowed in this context" error message.

> LayoutTests/js/script-tests/object-literal-methods.js:91
> +shouldThrow("{ f() { return super.f(); } }.f()");
> +shouldThrow("new ({ f() { return super(); }.f)");
> +shouldThrow("o = { f() { } }; new ({ __proto__: o, f() { return super(); } }).f");
> +shouldBeTrue("o = { f() { return true; } }; ({ __proto__: o, f() { return super.f(); } }).f()");
> +shouldBeTrue("o = { get p() { return true; } }; ({ __proto__: o, get p() { return super.p; } }).p");
> +shouldBeTrue("o = { set p(p2) { } }; ({ __proto__: o, set p(p2) { super.p = p2; } }).p = true");

Can you also add some tests for arrow functions to make sure they close over 'super' properly in object literal method syntax?

[Geoffrey Garen]

Committed r199927: <http://trac.webkit.org/changeset/199927>