[parser] Fix bug with non-static name method/property

[parser] Fix bug with non-static name method/property

Without this patch, the tests on lines 410, 414, 418 and 422 in
function testNonStaticName of test/mjsunit/es6/function-name.js
would all fail.  The bug caused non-static "name" methods and
properties to be mistaken for static ones.

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

Committed: https://crrev.com/f39665e36086361a1643d7c43b147bc3894a3bca
Cr-Commit-Position: refs/heads/master@{#41692}

[nickie, 2016-12-13 22:14:44]

https://codereview.chromium.org/2567343004/diff/1/test/mjsunit/es6/function-n...
File test/mjsunit/es6/function-name.js (right):

https://codereview.chromium.org/2567343004/diff/1/test/mjsunit/es6/function-n...
test/mjsunit/es6/function-name.js:389: 
Guys, please check that these two sets of tests actually make sense.
I won't pretend I know what this damned property is...  :-)

[adamk, 2016-12-14 07:17:53]

lgtm, thanks for the fix!

[nickie, 2016-12-14 08:43:09]

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

[commit-bot: I haz the power, 2016-12-14 08:43: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-12-14 08:43:19]

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

[commit-bot: I haz the power, 2016-12-14 08:43:19]

Dry run: The author nikolaos@chromium.org has not signed Google Contributor
License Agreement. Please visit https://cla.developers.google.com to sign and
manage CLA.

[nickie, 2016-12-14 08:48:45]

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

[commit-bot: I haz the power, 2016-12-14 08:48:47]

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-12-14 09:42:10]

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

[commit-bot: I haz the power, 2016-12-14 09:42:11]

Dry run: This issue passed the CQ dry run.

[nickie, 2016-12-14 12:09:14]

The CQ bit was checked by nikolaos@chromium.org

[commit-bot: I haz the power, 2016-12-14 12:09:19]

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-12-14 12:11:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481717354424320, "parent_rev":
"5b1f936d139c5c5e6d68f6d7d40127c13ec539d6", "commit_rev":
"e625be69866dc5b1d1da1aa0217bd769a56d2b3f"}

[commit-bot: I haz the power, 2016-12-14 12:11:21]

Description was changed from

==========
[parser] Fix bug with non-static name method/property

Without this patch, the tests on lines 410, 414, 418 and 422 in
function testNonStaticName of test/mjsunit/es6/function-name.js
would all fail.  The bug caused non-static "name" methods and
properties to be mistaken for static ones.

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

to

==========
[parser] Fix bug with non-static name method/property

Without this patch, the tests on lines 410, 414, 418 and 422 in
function testNonStaticName of test/mjsunit/es6/function-name.js
would all fail.  The bug caused non-static "name" methods and
properties to be mistaken for static ones.

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

Review-Url: https://codereview.chromium.org/2567343004
==========

[commit-bot: I haz the power, 2016-12-14 12:11:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-14 12:12:04]

Description was changed from

==========
[parser] Fix bug with non-static name method/property

Without this patch, the tests on lines 410, 414, 418 and 422 in
function testNonStaticName of test/mjsunit/es6/function-name.js
would all fail.  The bug caused non-static "name" methods and
properties to be mistaken for static ones.

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

Review-Url: https://codereview.chromium.org/2567343004
==========

to

==========
[parser] Fix bug with non-static name method/property

Without this patch, the tests on lines 410, 414, 418 and 422 in
function testNonStaticName of test/mjsunit/es6/function-name.js
would all fail.  The bug caused non-static "name" methods and
properties to be mistaken for static ones.

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

Committed: https://crrev.com/f39665e36086361a1643d7c43b147bc3894a3bca
Cr-Commit-Position: refs/heads/master@{#41692}
==========

[commit-bot: I haz the power, 2016-12-14 12:12:05]

Patchset 1 (id:??) landed as
https://crrev.com/f39665e36086361a1643d7c43b147bc3894a3bca
Cr-Commit-Position: refs/heads/master@{#41692}