[parser] Clean up ParserBase typedefs

[parser] Clean up ParserBase typedefs

The ParserTypes class has typedefs provided by the preparser or the
parser (e.g. PreParserExpression vs Expression*). Some of those typedefs
had gotten stale and were getting in the way, especially on gcc. We
also were making inconsistent use of aliases in parser-base.h; now their
use is much more consistent.

Committed: https://crrev.com/5c8c6fa160374d393c1d5575300c876b97332949
Cr-Commit-Position: refs/heads/master@{#39101}

[bakkot, 2016-09-01 00:43:11]

Description was changed from

==========
Clean up ParserBase typedefs.

The ParserTypes class has typedefs provided by the preparser or the
parser (e.g. PreParserExpression vs Expression*). Some of those typedefs
have gotten stale and were getting in the way, especially on GCC. We
also were making inconsistent use of aliases in parser-base; now their
use is much more consistent.
==========

to

==========
Clean up ParserBase typedefs.

The ParserTypes class has typedefs provided by the preparser or the
parser (e.g. PreParserExpression vs Expression*). Some of those typedefs
had gotten stale and were getting in the way, especially on GCC. We
also were making inconsistent use of aliases in parser-base; now their
use is much more consistent.
==========

[bakkot, 2016-09-01 00:43:11]

bakkot@google.com changed reviewers:
+ littledan@chromium.org, marja@chromium.org, nikolaos@chromium.org

[bakkot, 2016-09-01 00:43:18]

The CQ bit was checked by bakkot@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-01 00:43:24]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bakkot, 2016-09-01 00:58:24]

Description was changed from

==========
Clean up ParserBase typedefs.

The ParserTypes class has typedefs provided by the preparser or the
parser (e.g. PreParserExpression vs Expression*). Some of those typedefs
had gotten stale and were getting in the way, especially on GCC. We
also were making inconsistent use of aliases in parser-base; now their
use is much more consistent.
==========

to

==========
[parser] Clean up ParserBase typedefs

The ParserTypes class has typedefs provided by the preparser or the
parser (e.g. PreParserExpression vs Expression*). Some of those typedefs
had gotten stale and were getting in the way, especially on GCC. We
also were making inconsistent use of aliases in parser-base; now their
use is much more consistent.
==========

[bakkot, 2016-09-01 00:59:30]

Description was changed from

==========
[parser] Clean up ParserBase typedefs

The ParserTypes class has typedefs provided by the preparser or the
parser (e.g. PreParserExpression vs Expression*). Some of those typedefs
had gotten stale and were getting in the way, especially on GCC. We
also were making inconsistent use of aliases in parser-base; now their
use is much more consistent.
==========

to

==========
[parser] Clean up ParserBase typedefs

The ParserTypes class has typedefs provided by the preparser or the
parser (e.g. PreParserExpression vs Expression*). Some of those typedefs
had gotten stale and were getting in the way, especially on gcc. We
also were making inconsistent use of aliases in parser-base.h; now their
use is much more consistent.
==========

[commit-bot: I haz the power, 2016-09-01 01:06:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-01 01:06:18]

Dry run: This issue passed the CQ dry run.

[bakkot, 2016-09-01 01:18:57]

PTAL. I'm not sure what the source of these typedefs was, but removing them
doesn't seem to break v8, and the presence of the ClassLiteral typedef was
causing trouble for another patch when using gcc.

[marja, 2016-09-01 07:34:32]

lgtm. Probably these were needed in some previous state but not anymore. Thanks
for the cleanup!

[nickie, 2016-09-01 10:25:29]

lgtm

[bakkot, 2016-09-01 17:27:01]

The CQ bit was checked by bakkot@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-01 17:27:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-01 18:02:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-01 18:02:34]

Dry run: This issue passed the CQ dry run.

[bakkot, 2016-09-01 18:23:44]

The CQ bit was checked by bakkot@google.com

[bakkot, 2016-09-01 18:23:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
nikolaos@chromium.org, marja@chromium.org
Link to the patchset: https://codereview.chromium.org/2305523002/#ps40001
(title: "document Block typedef")

[commit-bot: I haz the power, 2016-09-01 18:23:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-01 18:26:11]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-01 18:26:50]

Description was changed from

==========
[parser] Clean up ParserBase typedefs

The ParserTypes class has typedefs provided by the preparser or the
parser (e.g. PreParserExpression vs Expression*). Some of those typedefs
had gotten stale and were getting in the way, especially on gcc. We
also were making inconsistent use of aliases in parser-base.h; now their
use is much more consistent.
==========

to

==========
[parser] Clean up ParserBase typedefs

The ParserTypes class has typedefs provided by the preparser or the
parser (e.g. PreParserExpression vs Expression*). Some of those typedefs
had gotten stale and were getting in the way, especially on gcc. We
also were making inconsistent use of aliases in parser-base.h; now their
use is much more consistent.

Committed: https://crrev.com/5c8c6fa160374d393c1d5575300c876b97332949
Cr-Commit-Position: refs/heads/master@{#39101}
==========

[commit-bot: I haz the power, 2016-09-01 18:26:51]

Patchset 3 (id:??) landed as
https://crrev.com/5c8c6fa160374d393c1d5575300c876b97332949
Cr-Commit-Position: refs/heads/master@{#39101}