[wasm] Add test for reexport of the same import twice.

[wasm] Add test for reexport of the same import twice.

R=clemensh@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2592563003
Cr-Commit-Position: refs/heads/master@{#41850}
Committed: https://chromium.googlesource.com/v8/v8/+/e797e0ea6d01ed84a23345ddf077157091d65648

[titzer, 2016-12-20 09:45:23]

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

[commit-bot: I haz the power, 2016-12-20 09:45:33]

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

[Clemens Hammacher, 2016-12-20 09:51:39]

lgtm

https://codereview.chromium.org/2592563003/diff/1/test/mjsunit/wasm/export-ta...
File test/mjsunit/wasm/export-table.js (right):

https://codereview.chromium.org/2592563003/diff/1/test/mjsunit/wasm/export-ta...
test/mjsunit/wasm/export-table.js:149: let instance = builder.instantiate({a:
js, b: js});
b is not needed here

[commit-bot: I haz the power, 2016-12-20 10:11:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-20 10:11:38]

Dry run: This issue passed the CQ dry run.

[titzer, 2016-12-20 10:13:45]

https://codereview.chromium.org/2592563003/diff/1/test/mjsunit/wasm/export-ta...
File test/mjsunit/wasm/export-table.js (right):

https://codereview.chromium.org/2592563003/diff/1/test/mjsunit/wasm/export-ta...
test/mjsunit/wasm/export-table.js:149: let instance = builder.instantiate({a:
js, b: js});
On 2016/12/20 09:51:39, Clemens Hammacher wrote:
> b is not needed here

Actually that was a bug in the test. Good catch. Fixed this and added a second
test for another case.

[titzer, 2016-12-20 10:14:18]

The CQ bit was checked by titzer@chromium.org

[titzer, 2016-12-20 10:14:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
clemensh@chromium.org
Link to the patchset: https://codereview.chromium.org/2592563003/#ps20001
(title: "Added test for second case.")

[commit-bot: I haz the power, 2016-12-20 10:14:31]

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-20 10:43:55]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482228858586170,
"parent_rev": "cbd3b3d0fe2439f8835c81658a6d685a0ce95480", "commit_rev":
"e797e0ea6d01ed84a23345ddf077157091d65648"}

[commit-bot: I haz the power, 2016-12-20 10:43:58]

Description was changed from

==========
[wasm] Add test for reexport of the same import twice.

R=clemensh@chromium.org
BUG=
==========

to

==========
[wasm] Add test for reexport of the same import twice.

R=clemensh@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2592563003
Cr-Commit-Position: refs/heads/master@{#41850}
Committed:
https://chromium.googlesource.com/v8/v8/+/e797e0ea6d01ed84a23345ddf077157091d...
==========

[commit-bot: I haz the power, 2016-12-20 10:43:59]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/e797e0ea6d01ed84a23345ddf077157091d...