Simplify error message logic in ParseImportNames

Simplify error message logic in ParseImportNames

The new logic ensures that the error messages are the same in the
"import { <reserved word> }" and "import { foo as <reserved ord> }"
cases.

Also prepares ParseImportNames for returning both the import and local
names to ParseImportClause.

BUG=v8:1569
LOG=n

Committed: https://crrev.com/8b33567fd37e3151d3ea707f7fb47b67196a5fc2
Cr-Commit-Position: refs/heads/master@{#26863}

[adamk, 2015-02-25 00:07:09]

adamk@chromium.org changed reviewers:
+ arv@chromium.org, rossberg@chromium.org

[rossberg, 2015-02-25 13:41:33]

lgtm

[arv (Not doing code reviews), 2015-02-25 15:33:07]

LGTM

https://codereview.chromium.org/952863006/diff/1/src/parser.cc
File src/parser.cc (right):

https://codereview.chromium.org/952863006/diff/1/src/parser.cc#newcode1422
src/parser.cc:1422: Literal* module = ParseModuleSpecifier(CHECK_OK);
nit: module_specifier

[adamk, 2015-02-25 19:02:33]

https://codereview.chromium.org/952863006/diff/1/src/parser.cc
File src/parser.cc (right):

https://codereview.chromium.org/952863006/diff/1/src/parser.cc#newcode1422
src/parser.cc:1422: Literal* module = ParseModuleSpecifier(CHECK_OK);
On 2015/02/25 15:33:07, arv wrote:
> nit: module_specifier

Done.

[adamk, 2015-02-25 19:04:15]

The CQ bit was checked by adamk@chromium.org

[adamk, 2015-02-25 19:04:16]

The patchset sent to the CQ was uploaded after l-g-t-m from arv@chromium.org,
rossberg@chromium.org
Link to the patchset: https://codereview.chromium.org/952863006/#ps20001 (title:
"Fixed nit")

[commit-bot: I haz the power, 2015-02-25 19:04:53]

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

[commit-bot: I haz the power, 2015-02-25 19:40:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-25 19:41:07]

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