Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1419)

Unified Diff: test/mjsunit/harmony/class-computed-property-names-super.js

Issue 914773002: Fix issue with super and computed property names (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/mjsunit/harmony/class-computed-property-names-super.js
diff --git a/test/mjsunit/harmony/class-computed-property-names-super.js b/test/mjsunit/harmony/class-computed-property-names-super.js
new file mode 100644
index 0000000000000000000000000000000000000000..5a5db67d4884cd7ef95672fda0336165ca4eca5c
--- /dev/null
+++ b/test/mjsunit/harmony/class-computed-property-names-super.js
@@ -0,0 +1,76 @@
+// Copyright 2014 the V8 project authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// Flags: --harmony-computed-property-names --harmony-sloppy
+// Flags: --harmony-classes --allow-natives-syntax
+
+
+function ID(x) {
+ return x;
+}
+
+
+(function TestComputedMethodSuper() {
+ class Base {
+ m() {
+ return ' base m';
+ }
+ }
+ class Derived extends Base {
+ ['a']() { return 'a' + super.m(); }
+ [ID('b')]() { return 'b' + super.m(); }
+ [0]() { return '0' + super.m(); }
+ [ID(1)]() { return '1' + super.m(); }
+ }
+
+ assertSame(Derived.prototype, Derived.prototype.a[%HomeObjectSymbol()]);
+
+ assertEquals('a base m', new Derived().a());
+ assertEquals('b base m', new Derived().b());
+ assertEquals('0 base m', new Derived()[0]());
+ assertEquals('1 base m', new Derived()[1]());
+})();
+
+
+(function TestComputedGetterSuper() {
+ class Base {
+ m() {
+ return ' base m';
+ }
+ }
+ class Derived extends Base {
+ get ['a']() { return 'a' + super.m(); }
+ get [ID('b')]() { return 'b' + super.m(); }
+ get [0]() { return '0' + super.m(); }
+ get [ID(1)]() { return '1' + super.m(); }
+ }
+ assertEquals('a base m', new Derived().a);
+ assertEquals('b base m', new Derived().b);
+ assertEquals('0 base m', new Derived()[0]);
+ assertEquals('1 base m', new Derived()[1]);
+})();
+
+
+(function TestComputedSetterSuper() {
+ var value;
+ class Base {
+ m(name, v) {
+ value = name + ' ' + v;
+ }
+ }
+ class Derived extends Base {
+ set ['a'](v) { super.m('a', v); }
+ set [ID('b')](v) { super.m('b', v); }
+ set [0](v) { super.m('0', v); }
+ set [ID(1)](v) { super.m('1', v); }
+ }
+ new Derived().a = 2;
+ assertEquals('a 2', value);
+ new Derived().b = 3;
+ assertEquals('b 3', value);
+ new Derived()[0] = 4;
+ assertEquals('0 4', value);
+ new Derived()[1] = 5;
+ assertEquals('1 5', value);
+})();

Powered by Google App Engine
This is Rietveld 408576698