[wasm] Instance exports object should be iterable

[wasm] Instance exports object should be iterable

BUG=v8:5863
R=titzer@chromium.org, dschuff@chromium.org

Review-Url: https://codereview.chromium.org/2640403004
Cr-Commit-Position: refs/heads/master@{#42578}
Committed: https://chromium.googlesource.com/v8/v8/+/2ca39001c8c134fce0bfc6f9f73044ff82671156

[gdeepti, 2017-01-20 01:28:02]

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

[commit-bot: I haz the power, 2017-01-20 01:28:12]

Dry run: 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-20 02:09:13]

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

[commit-bot: I haz the power, 2017-01-20 02:09:14]

Dry run: This issue passed the CQ dry run.

[titzer, 2017-01-20 09:12:02]

lgtm with comment

https://codereview.chromium.org/2640403004/diff/1/test/mjsunit/regress/wasm/r...
File test/mjsunit/regress/wasm/regression-5863.js (right):

https://codereview.chromium.org/2640403004/diff/1/test/mjsunit/regress/wasm/r...
test/mjsunit/regress/wasm/regression-5863.js:1: // Copyright 2017 the V8 project
authors. All rights reserved.
Can you move this file into test/mjsunit/wasm ?

It's more of a feature test than a regression test.

[Derek Schuff, 2017-01-20 16:35:40]

Description was changed from

==========
[wasm] Instance exports object should be iterable

BUG=5863

R=titzer@chromium.org, dschuff@chromium.org
==========

to

==========
[wasm] Instance exports object should be iterable

BUG=v8:5863

R=titzer@chromium.org, dschuff@chromium.org
==========

[gdeepti, 2017-01-20 18:12:10]

https://codereview.chromium.org/2640403004/diff/1/test/mjsunit/regress/wasm/r...
File test/mjsunit/regress/wasm/regression-5863.js (right):

https://codereview.chromium.org/2640403004/diff/1/test/mjsunit/regress/wasm/r...
test/mjsunit/regress/wasm/regression-5863.js:1: // Copyright 2017 the V8 project
authors. All rights reserved.
On 2017/01/20 09:12:02, titzer wrote:
> Can you move this file into test/mjsunit/wasm ?
> 
> It's more of a feature test than a regression test.

Moved to test/mjsunit/wasm/instantiate-module-basic.js.

[gdeepti, 2017-01-20 18:19:18]

The CQ bit was checked by gdeepti@chromium.org

[gdeepti, 2017-01-20 18:19:18]

The patchset sent to the CQ was uploaded after l-g-t-m from titzer@chromium.org
Link to the patchset: https://codereview.chromium.org/2640403004/#ps20001
(title: "Review")

[commit-bot: I haz the power, 2017-01-20 18:19:33]

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-20 19:35:17]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484936358257680,
"parent_rev": "50e0fe29bb4facb6b94cbfb51637daef6cc578b9", "commit_rev":
"2ca39001c8c134fce0bfc6f9f73044ff82671156"}

[commit-bot: I haz the power, 2017-01-20 19:35:22]

Description was changed from

==========
[wasm] Instance exports object should be iterable

BUG=v8:5863

R=titzer@chromium.org, dschuff@chromium.org
==========

to

==========
[wasm] Instance exports object should be iterable

BUG=v8:5863

R=titzer@chromium.org, dschuff@chromium.org

Review-Url: https://codereview.chromium.org/2640403004
Cr-Commit-Position: refs/heads/master@{#42578}
Committed:
https://chromium.googlesource.com/v8/v8/+/2ca39001c8c134fce0bfc6f9f73044ff826...
==========

[commit-bot: I haz the power, 2017-01-20 19:35:23]

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