Parser sync tests for `let` identifiers

Parser sync tests for `let` identifiers

R=marja@chromium.org
BUG=

Committed: https://code.google.com/p/v8/source/detail?r=22324

[marja, 2014-07-10 14:35:13]

lgtm

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

https://codereview.chromium.org/385613002/diff/1/test/cctest/test-parsing.cc#...
test/cctest/test-parsing.cc:1782: static const ParserFlag always_flags[] =
{kAllowArrowFunctions};
Afaics this test doesn't need kAllowArrowFunctions, and the test where you
copied it doesn't need it either (it's there by mistake :/).

RunParserSyncTest(context_data, statement_data, kSuccess); should be enough.
That will run the test both with and without arrow functions.

https://codereview.chromium.org/385613002/diff/1/test/cctest/test-parsing.cc#...
test/cctest/test-parsing.cc:1820: static const ParserFlag always_flags[] =
{kAllowArrowFunctions};
Namely here... we don't need kAllowArrowFunctions.

[rossberg, 2014-07-10 14:39:00]

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

https://codereview.chromium.org/385613002/diff/1/test/cctest/test-parsing.cc#...
test/cctest/test-parsing.cc:1782: static const ParserFlag always_flags[] =
{kAllowArrowFunctions};
On 2014/07/10 14:35:13, marja wrote:
> Afaics this test doesn't need kAllowArrowFunctions, and the test where you
> copied it doesn't need it either (it's there by mistake :/).
> 
> RunParserSyncTest(context_data, statement_data, kSuccess); should be enough.
> That will run the test both with and without arrow functions.

Right, done.

[rossberg, 2014-07-10 14:39:41]

Committed patchset #2 manually as r22324 (presubmit successful).