| Summary: | Class body ending with a semicolon throws a SyntaxError | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||
| Component: | JavaScriptCore | Assignee: | Ryosuke Niwa <rniwa> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | commit-queue, gyuyoung.kim, ossy | ||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 140491 | ||||||
| Attachments: |
|
||||||
Created attachment 251720 [details]
Patch
Comment on attachment 251720 [details] Patch Clearing flags on attachment: 251720 Committed r183383: <http://trac.webkit.org/changeset/183383> All reviewed patches have been landed. Closing bug. Committed r183384: <http://trac.webkit.org/changeset/183384> (In reply to comment #4) > Committed r183384: <http://trac.webkit.org/changeset/183384> I'm so sorry. My gardening patch messed up ChangeLog. This test asserts every time: https://build.webkit.org/results/Apple%20Yosemite%20Debug%20WK2%20(Tests)/r183383%20(3800)/js/class-syntax-semicolon-crash-log.txt Ryosuke, are you available to fix this now? Oh oops, let me look into it. Oh it looks like this new test uncovered an existing bug in the code. Added a test expectation in http://trac.webkit.org/changeset/183391. Will investigate the assertion failure in webkit.org/b/144258 |
class A { ; } or class A { foo() {}; } results in a syntax error but it shoulnd't.