[wasm][asm.js] Don't squish error location because module end not reached.

[wasm][asm.js] Don't squish error location because module end not reached.

We recently started emitting an error because a module's validation
doesn't reach the end (due to lack of exports).
This contained a mistake that prevents many most other types
of errors from actually getting out.

Ensuring export errors (or not reaching then end) doesn't clobber
other errors.

TODO(bradnelson): Figure out a way to structure tests so this gets
caught.

BUG=v8:4203
R=gdeepti@chromium.org

Review-Url: https://codereview.chromium.org/2632593002
Cr-Commit-Position: refs/heads/master@{#42301}
Committed: https://chromium.googlesource.com/v8/v8/+/7d86a478d22c8ea7dbf16789229e9e1c0b8687ec

[bradnelson, 2017-01-13 02:48:04]

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

[commit-bot: I haz the power, 2017-01-13 02:48:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[gdeepti, 2017-01-13 02:49:42]

lgtm

[bradn, 2017-01-13 02:58:14]

The CQ bit was unchecked by bradnelson@google.com

[bradn, 2017-01-13 02:58:15]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2017-01-13 02:58:21]

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, 2017-01-13 03:38:47]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484276295342090, "parent_rev":
"f6929084821d4f021b2c2768c9856472ffa42623", "commit_rev":
"7d86a478d22c8ea7dbf16789229e9e1c0b8687ec"}

[commit-bot: I haz the power, 2017-01-13 03:38:53]

Description was changed from

==========
[wasm][asm.js] Don't squish error location because module end not reached.

We recently started emitting an error because a module's validation
doesn't reach the end (due to lack of exports).
This contained a mistake that prevents many most other types
of errors from actually getting out.

Ensuring export errors (or not reaching then end) doesn't clobber
other errors.

TODO(bradnelson): Figure out a way to structure tests so this gets
caught.

BUG=v8:4203
R=gdeepti@chromium.org
==========

to

==========
[wasm][asm.js] Don't squish error location because module end not reached.

We recently started emitting an error because a module's validation
doesn't reach the end (due to lack of exports).
This contained a mistake that prevents many most other types
of errors from actually getting out.

Ensuring export errors (or not reaching then end) doesn't clobber
other errors.

TODO(bradnelson): Figure out a way to structure tests so this gets
caught.

BUG=v8:4203
R=gdeepti@chromium.org

Review-Url: https://codereview.chromium.org/2632593002
Cr-Commit-Position: refs/heads/master@{#42301}
Committed:
https://chromium.googlesource.com/v8/v8/+/7d86a478d22c8ea7dbf16789229e9e1c0b8...
==========

[commit-bot: I haz the power, 2017-01-13 03:38:54]

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