Revert of change most cases of variable redeclaration from TypeError to SyntaxError

Revert of change most cases of variable redeclaration from TypeError to SyntaxError (patchset #8 id:140001 of https://codereview.chromium.org/2048703002/ )

Reason for revert:
This is going to break the LayoutTest inspector-protocol/console/console-let-const-with-api.html as seen in https://build.chromium.org/p/tryserver.v8/builders/v8_linux_blink_rel/builds/2247 . Please run this test manually, using instructions at https://www.chromium.org/developers/testing/webkit-layout-tests , and fix on the Chrome side if needed before resubmitting this patch.

Original issue's description:
> change most cases of variable redeclaration from TypeError to SyntaxError.
>
> Code like `let a; eval("var a;");` should throw a SyntaxError, not a TypeError
> (this caused a test262 failure.). However, the code `eval("function NaN() {}");`
> should actually throw a TypeError. This patch changes most cases of
> redeclaration errors from TypeError to SyntaxError. See the test
> mjsunit/regress/redeclaration-error-types for a thorough analysis with spec
> references.
>
> The relevant sections of the spec are ES#sec-globaldeclarationinstantiation and
> ES#sec-evaldeclarationinstantiation
>
> BUG=v8:4955
> LOG=y
>
> Committed: https://crrev.com/2b787561763d0f7e8dab698652715a742cf78291
> Cr-Commit-Position: refs/heads/master@{#36940}

TBR=adamk@chromium.org,jwolfe@igalia.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4955
Committed: https://crrev.com/85c2c8d8476df10464b6a6e9462ce8ab6a015ecb
Cr-Commit-Position: refs/heads/master@{#36941}

[Dan Ehrenberg, 2016-06-13 18:21:40]

The CQ bit was checked by littledan@chromium.org

[Dan Ehrenberg, 2016-06-13 18:21:41]

Created Revert of change most cases of variable redeclaration from TypeError to
SyntaxError

[commit-bot: I haz the power, 2016-06-13 18:21:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2064793002/1

[commit-bot: I haz the power, 2016-06-13 18:21:58]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Revert of change most cases of variable redeclaration from TypeError to
SyntaxError (patchset #8 id:140001 of
https://codereview.chromium.org/2048703002/ )

Reason for revert:
This is going to break the LayoutTest
inspector-protocol/console/console-let-const-with-api.html as seen in
https://build.chromium.org/p/tryserver.v8/builders/v8_linux_blink_rel/builds/...
. Please run this test manually, using instructions at
https://www.chromium.org/developers/testing/webkit-layout-tests , and fix on the
Chrome side if needed before resubmitting this patch.

Original issue's description:
> change most cases of variable redeclaration from TypeError to SyntaxError.
> 
> Code like `let a; eval("var a;");` should throw a SyntaxError, not a TypeError
> (this caused a test262 failure.). However, the code `eval("function NaN()
{}");`
> should actually throw a TypeError. This patch changes most cases of
> redeclaration errors from TypeError to SyntaxError. See the test
> mjsunit/regress/redeclaration-error-types for a thorough analysis with spec
> references.
> 
> The relevant sections of the spec are ES#sec-globaldeclarationinstantiation
and
> ES#sec-evaldeclarationinstantiation
> 
> BUG=v8:4955
> LOG=y
> 
> Committed: https://crrev.com/2b787561763d0f7e8dab698652715a742cf78291
> Cr-Commit-Position: refs/heads/master@{#36940}

TBR=adamk@chromium.org,jwolfe@igalia.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4955
==========

to

==========
Revert of change most cases of variable redeclaration from TypeError to
SyntaxError (patchset #8 id:140001 of
https://codereview.chromium.org/2048703002/ )

Reason for revert:
This is going to break the LayoutTest
inspector-protocol/console/console-let-const-with-api.html as seen in
https://build.chromium.org/p/tryserver.v8/builders/v8_linux_blink_rel/builds/...
. Please run this test manually, using instructions at
https://www.chromium.org/developers/testing/webkit-layout-tests , and fix on the
Chrome side if needed before resubmitting this patch.

Original issue's description:
> change most cases of variable redeclaration from TypeError to SyntaxError.
>
> Code like `let a; eval("var a;");` should throw a SyntaxError, not a TypeError
> (this caused a test262 failure.). However, the code `eval("function NaN()
{}");`
> should actually throw a TypeError. This patch changes most cases of
> redeclaration errors from TypeError to SyntaxError. See the test
> mjsunit/regress/redeclaration-error-types for a thorough analysis with spec
> references.
>
> The relevant sections of the spec are ES#sec-globaldeclarationinstantiation
and
> ES#sec-evaldeclarationinstantiation
>
> BUG=v8:4955
> LOG=y
>
> Committed: https://crrev.com/2b787561763d0f7e8dab698652715a742cf78291
> Cr-Commit-Position: refs/heads/master@{#36940}

TBR=adamk@chromium.org,jwolfe@igalia.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4955

Committed: https://crrev.com/85c2c8d8476df10464b6a6e9462ce8ab6a015ecb
Cr-Commit-Position: refs/heads/master@{#36941}
==========

[commit-bot: I haz the power, 2016-06-13 18:24:02]

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