[parser] Reorder traits methods

[parser] Reorder traits methods

This patch simply reorders the elements of classes ParserTraits
and PreParserTraits (mainly method headers), so that in both
classes the elements are defined in the same order.  This is
useful for easily verifying the consistency between these two
classes.

R=adamk@chromium.org, marja@chromium.org
BUG=
LOG=N

Committed: https://crrev.com/848b6278f8e52cc7a88423d9493e83fdb0866e58
Cr-Commit-Position: refs/heads/master@{#38766}

[nickie, 2016-08-19 13:03:40]

This patch only moves methods up and down, only inside the declaration of
classes ParserTraits and PreParserTraits.  In case you feel that this will
unnecessarily clutter git blame (which is true, of course), my excuse is that
the ultimate goal is to have very minimal traits objects, so these methods will
soon change in much more radical ways.

[nickie, 2016-08-19 13:09:19]

The CQ bit was checked by nikolaos@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-19 13:09:29]

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-08-19 13:55:35]

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

[commit-bot: I haz the power, 2016-08-19 13:55:35]

Dry run: This issue passed the CQ dry run.

[adamk, 2016-08-19 17:59:58]

lgtm % nit

https://codereview.chromium.org/2264483003/diff/1/src/parsing/preparser.h
File src/parsing/preparser.h (right):

https://codereview.chromium.org/2264483003/diff/1/src/parsing/preparser.h#new...
src/parsing/preparser.h:989: V8_INLINE PreParserExpression
RewriteYieldStar(PreParserExpression generator,
I'd leave these V8_INLINE changes out of the patch.

[nickie, 2016-08-20 13:33:32]

https://codereview.chromium.org/2264483003/diff/1/src/parsing/preparser.h
File src/parsing/preparser.h (right):

https://codereview.chromium.org/2264483003/diff/1/src/parsing/preparser.h#new...
src/parsing/preparser.h:989: V8_INLINE PreParserExpression
RewriteYieldStar(PreParserExpression generator,
On 2016/08/19 17:59:58, adamk wrote:
> I'd leave these V8_INLINE changes out of the patch.

Done.

[nickie, 2016-08-20 13:35:08]

The CQ bit was checked by nikolaos@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-20 13:35:17]

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-08-20 14:05:10]

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

[commit-bot: I haz the power, 2016-08-20 14:05:11]

Dry run: This issue passed the CQ dry run.

[nickie, 2016-08-20 14:09:02]

The CQ bit was checked by nikolaos@chromium.org

[nickie, 2016-08-20 14:09:02]

The patchset sent to the CQ was uploaded after l-g-t-m from adamk@chromium.org
Link to the patchset: https://codereview.chromium.org/2264483003/#ps20001
(title: "Changes after reviewers' comments")

[commit-bot: I haz the power, 2016-08-20 14:09:11]

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-08-20 14:11:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-20 14:11:39]

Description was changed from

==========
[parser] Reorder traits methods

This patch simply reorders the elements of classes ParserTraits
and PreParserTraits (mainly method headers), so that in both
classes the elements are defined in the same order.  This is
useful for easily verifying the consistency between these two
classes.

R=adamk@chromium.org, marja@chromium.org
BUG=
LOG=N
==========

to

==========
[parser] Reorder traits methods

This patch simply reorders the elements of classes ParserTraits
and PreParserTraits (mainly method headers), so that in both
classes the elements are defined in the same order.  This is
useful for easily verifying the consistency between these two
classes.

R=adamk@chromium.org, marja@chromium.org
BUG=
LOG=N

Committed: https://crrev.com/848b6278f8e52cc7a88423d9493e83fdb0866e58
Cr-Commit-Position: refs/heads/master@{#38766}
==========

[commit-bot: I haz the power, 2016-08-20 14:11:40]

Patchset 2 (id:??) landed as
https://crrev.com/848b6278f8e52cc7a88423d9493e83fdb0866e58
Cr-Commit-Position: refs/heads/master@{#38766}