Implement LinkError; import tweaks

Implement LinkError; import tweaks

- Implement new WebAssembly.LinkError exception
- Implement stricter checks for glboal imports
- Add tests
- Refactor handling of import names
- Add TODOs for empty import names

R=titzer@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2584843002
Cr-Commit-Position: refs/heads/master@{#41764}
Committed: https://chromium.googlesource.com/v8/v8/+/16fe4263200532191c4633a96714cec7b6e23510

[rossberg, 2016-12-16 13:34:25]

Description was changed from

==========
Implement LinkError; import tweaks

R=titzer@chromium.org
BUG=
==========

to

==========
Implement LinkError; import tweaks

- Implement new WebAssembly.LinkError exception
- Implement stricter checks for glboal imports
- Add tests
- Refactor handling of import names
- Add TODOs for empty import names

R=titzer@chromium.org
BUG=
==========

[rossberg, 2016-12-16 13:35:03]

@bradnelson: FYI. I signed you up for the empty import name TODO, since the
misbehaviour seems to be due to a hack only relevant to the asm.js pipeline.

[titzer, 2016-12-16 13:37:54]

lgtm

[rossberg, 2016-12-16 13:43:30]

The CQ bit was checked by rossberg@chromium.org

[commit-bot: I haz the power, 2016-12-16 13:43:35]

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-16 14:23:36]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481895810731270, "parent_rev":
"bb76432fde6cee11f35757786c8ca24e1d8ef2ac", "commit_rev":
"16fe4263200532191c4633a96714cec7b6e23510"}

[commit-bot: I haz the power, 2016-12-16 14:23:45]

Description was changed from

==========
Implement LinkError; import tweaks

- Implement new WebAssembly.LinkError exception
- Implement stricter checks for glboal imports
- Add tests
- Refactor handling of import names
- Add TODOs for empty import names

R=titzer@chromium.org
BUG=
==========

to

==========
Implement LinkError; import tweaks

- Implement new WebAssembly.LinkError exception
- Implement stricter checks for glboal imports
- Add tests
- Refactor handling of import names
- Add TODOs for empty import names

R=titzer@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2584843002
Cr-Commit-Position: refs/heads/master@{#41764}
Committed:
https://chromium.googlesource.com/v8/v8/+/16fe4263200532191c4633a96714cec7b6e...
==========

[commit-bot: I haz the power, 2016-12-16 14:23:46]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/16fe4263200532191c4633a96714cec7b6e...

[gsathya, 2016-12-16 18:53:00]

On 2016/12/16 14:23:46, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
>
https://chromium.googlesource.com/v8/v8/+/16fe4263200532191c4633a96714cec7b6e...

Hmm not sure how this cleared the presubmit, my CL caught a bunch of formatting
issues caused by this --
https://codereview.chromium.org/2575313002/