[wasm] Imported WebAssembly function are never wrapped.

[wasm] Imported WebAssembly function are never wrapped.

According to the spec, import wrappers are only generated for JavaScript
functions, not for WebAssembly function. If an imported WebAssembly
function does not have the expected type, then a type error is thrown.

R=titzer@chromium.org, rossberg@chromium.org
TEST=mjsunit/wasm/test-import-export-wrapper

Committed: https://crrev.com/665a5d17acc17c85d61a99363d61514656d4b6ea
Cr-Commit-Position: refs/heads/master@{#40901}

[ahaas, 2016-11-10 14:14:50]

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

[commit-bot: I haz the power, 2016-11-10 14:15:02]

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

[titzer, 2016-11-10 14:26:23]

On 2016/11/10 14:14:48, ahaas wrote:

lgtm with grammar nazi nits

[ahaas, 2016-11-10 14:27:37]

On 2016/11/10 at 14:26:23, titzer wrote:
> On 2016/11/10 14:14:48, ahaas wrote:
> 
> lgtm with grammar nazi nits

where are the nits?

[titzer, 2016-11-10 14:28:41]

https://codereview.chromium.org/2486943005/diff/1/test/mjsunit/wasm/test-impo...
File test/mjsunit/wasm/test-import-export-wrapper.js (right):

https://codereview.chromium.org/2486943005/diff/1/test/mjsunit/wasm/test-impo...
test/mjsunit/wasm/test-import-export-wrapper.js:131: // In this case,
second_export has less params than first_import,
s/less/fewer

[ahaas, 2016-11-10 14:38:47]

https://codereview.chromium.org/2486943005/diff/1/test/mjsunit/wasm/test-impo...
File test/mjsunit/wasm/test-import-export-wrapper.js (right):

https://codereview.chromium.org/2486943005/diff/1/test/mjsunit/wasm/test-impo...
test/mjsunit/wasm/test-import-export-wrapper.js:131: // In this case,
second_export has less params than first_import,
On 2016/11/10 at 14:28:41, titzer wrote:
> s/less/fewer

Done.

[ahaas, 2016-11-10 14:38:49]

The CQ bit was checked by ahaas@chromium.org

[ahaas, 2016-11-10 14:38:50]

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/2486943005/#ps20001
(title: "typo")

[commit-bot: I haz the power, 2016-11-10 14:38:52]

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-11-10 15:05:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:29:33]

Description was changed from

==========
[wasm] Imported WebAssembly function are never wrapped.

According to the spec, import wrappers are only generated for JavaScript
functions, not for WebAssembly function. If an imported WebAssembly
function does not have the expected type, then a type error is thrown.

R=titzer@chromium.org, rossberg@chromium.org
TEST=mjsunit/wasm/test-import-export-wrapper
==========

to

==========
[wasm] Imported WebAssembly function are never wrapped.

According to the spec, import wrappers are only generated for JavaScript
functions, not for WebAssembly function. If an imported WebAssembly
function does not have the expected type, then a type error is thrown.

R=titzer@chromium.org, rossberg@chromium.org
TEST=mjsunit/wasm/test-import-export-wrapper

Committed: https://crrev.com/665a5d17acc17c85d61a99363d61514656d4b6ea
Cr-Commit-Position: refs/heads/master@{#40901}
==========

[commit-bot: I haz the power, 2016-11-17 22:29:35]

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