1288381 - Allow property accessors and calls after super()

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[André Bargull [:anba]]

Test case:
---
class C extends Object {
    constructor() {
        super().foo;
        super()[0];
        super()();
        super()``;
    }
}
---

Expected: Does not throw a SyntaxError
Actual: Throws a SyntaxError

Comment 1

[andrey.a.gubanov]

class A extends B { constructor() { super().foo() } } // throws "SyntaxError: missing ; before statement"

class A extends B { constructor() { super()
    .foo() } } // SyntaxError: expected expression, got '.'

Comment 2

[André Bargull [:anba]]

Attachment: bug1288381.patch

Another simple frontend bug, I'll post another patch which omits whitespace only changes so it's easier to spot the changes.

Comment 3

[André Bargull [:anba]]

Attachment: bug1288381-no-whitespace.patch

Changes to Parser.cpp from the actual patch, but with whitespace only changes suppressed.

Comment 4

[Tooru Fujisawa [:arai]]

Comment on attachment 8797160 [details] [diff] [review]
bug1288381.patch

Review of attachment 8797160 [details] [diff] [review]:
-----------------------------------------------------------------

thanks :)

Comment 5

[André Bargull [:anba]]

Try run:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ffb3438b5f8a07a54364032fcb878a1f15690743

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6dd3d752338c
Allow member expressions after super() call. r=arai

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6dd3d752338c