Add parsing for class declarations

Add parsing for class declarations

This patch supports:

-  type parameters,
-  the "implements" clause,
-  variable member declarations, and
-  index members.

This is based on https://codereview.chromium.org/1839393002

R=rossberg@chromium.org
BUG=
NOTRY=true
NOPRESUBMIT=true

[rossberg, 2016-04-18 11:03:08]

lgtm

https://codereview.chromium.org/1850663002/diff/1/src/parsing/parser-base.h
File src/parsing/parser-base.h (right):

https://codereview.chromium.org/1850663002/diff/1/src/parsing/parser-base.h#n...
src/parsing/parser-base.h:1667: // i.e., class C { [42](x) {} } does not work in
typed mode.
Interesting. That's probably going to be a problem for TypeScript, too. I wonder
how they'll resolve it...

https://codereview.chromium.org/1850663002/diff/1/test/cctest/test-parsing.cc
File test/cctest/test-parsing.cc (right):

https://codereview.chromium.org/1850663002/diff/1/test/cctest/test-parsing.cc...
test/cctest/test-parsing.cc:7892: "class F implements I<> {}",
class implements {}
class C implements {}
class implements C {}

[nickie, 2016-04-19 14:55:30]

https://codereview.chromium.org/1850663002/diff/1/src/parsing/parser-base.h
File src/parsing/parser-base.h (right):

https://codereview.chromium.org/1850663002/diff/1/src/parsing/parser-base.h#n...
src/parsing/parser-base.h:1667: // i.e., class C { [42](x) {} } does not work in
typed mode.
On 2016/04/18 11:03:08, rossberg wrote:
> Interesting. That's probably going to be a problem for TypeScript, too. I
wonder
> how they'll resolve it...

I suppose index member declarations can be distinguished from computed
properties if you look ahead past the identifier.  The former have a colon
there, which is not legal in the latter.  (I don't believe that an
AssignmentExpression can begin by an identifier followed by a colon.)

We can do it here too, but we need 3 look-ahead symbols, unless we somehow merge
this with the ParsePropertyName in line 1711 (this will become ugly, IMO).

https://codereview.chromium.org/1850663002/diff/1/test/cctest/test-parsing.cc
File test/cctest/test-parsing.cc (right):

https://codereview.chromium.org/1850663002/diff/1/test/cctest/test-parsing.cc...
test/cctest/test-parsing.cc:7892: "class F implements I<> {}",
On 2016/04/18 11:03:08, rossberg wrote:
> class implements {}
> class C implements {}
> class implements C {}

Done.

[nickie, 2016-04-20 13:55:10]

Patchset 2 landed MANUALLY in experimental/nikolaos/types as:
48fb4065d12bd5df2ee08cb8ce602b4d9cd1bf5d
https://chromium.googlesource.com/v8/v8/+/48fb4065d12bd5df2ee08cb8ce602b4d9cd...

[nickie, 2016-04-20 13:55:23]

Description was changed from

==========
Add parsing for class declarations

This patch supports:

-  type parameters,
-  the "implements" clause,
-  variable member declarations, and
-  index members.

This is based on https://codereview.chromium.org/1839393002

R=rossberg@chromium.org
BUG=
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
Add parsing for class declarations

This patch supports:

-  type parameters,
-  the "implements" clause,
-  variable member declarations, and
-  index members.

This is based on https://codereview.chromium.org/1839393002

R=rossberg@chromium.org
BUG=
NOTRY=true
NOPRESUBMIT=true
==========