[wasm] Use WasmCompiledModule type in interfaces

[wasm] Use WasmCompiledModule type in interfaces

Just a minor refactoring. We have the type for compiled modules, so use
it where possible for better type safety.

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

Committed: https://crrev.com/1b5861413597a2f9dfec98e9bebc0fad457f726d
Cr-Commit-Position: refs/heads/master@{#40156}

[Clemens Hammacher, 2016-10-11 06:54:27]

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

[commit-bot: I haz the power, 2016-10-11 06:54:28]

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, 2016-10-11 07:10:29]

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

[commit-bot: I haz the power, 2016-10-11 07:10:30]

Dry run: Try jobs failed on following builders:
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_avx2_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng_trig...)

[Clemens Hammacher, 2016-10-11 07:31:38]

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

[commit-bot: I haz the power, 2016-10-11 07:31:46]

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

[ahaas, 2016-10-11 07:46:05]

lgtm with nit

https://codereview.chromium.org/2405173002/diff/20001/src/runtime/runtime-tes...
File src/runtime/runtime-test.cc (right):

https://codereview.chromium.org/2405173002/diff/20001/src/runtime/runtime-tes...
src/runtime/runtime-test.cc:766:
WasmCompiledModuleSerializer::DeserializeWasmModule(isolate, &sc);
Couldn't DeserializeWasmModule already return a wasm::WasmCompiledModule?

[commit-bot: I haz the power, 2016-10-11 07:57:37]

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

[commit-bot: I haz the power, 2016-10-11 07:57:38]

Dry run: This issue passed the CQ dry run.

[titzer, 2016-10-11 08:09:46]

On 2016/10/11 07:46:05, ahaas wrote:
> lgtm with nit
> 
>
https://codereview.chromium.org/2405173002/diff/20001/src/runtime/runtime-tes...
> File src/runtime/runtime-test.cc (right):
> 
>
https://codereview.chromium.org/2405173002/diff/20001/src/runtime/runtime-tes...
> src/runtime/runtime-test.cc:766:
> WasmCompiledModuleSerializer::DeserializeWasmModule(isolate, &sc);
> Couldn't DeserializeWasmModule already return a wasm::WasmCompiledModule?

lgtm

[Clemens Hammacher, 2016-10-11 08:19:19]

https://codereview.chromium.org/2405173002/diff/20001/src/runtime/runtime-tes...
File src/runtime/runtime-test.cc (right):

https://codereview.chromium.org/2405173002/diff/20001/src/runtime/runtime-tes...
src/runtime/runtime-test.cc:766:
WasmCompiledModuleSerializer::DeserializeWasmModule(isolate, &sc);
On 2016/10/11 07:46:05, ahaas wrote:
> Couldn't DeserializeWasmModule already return a wasm::WasmCompiledModule?

We would have to include wasm headers in code-serializer.h, which I did not want
to do if not really necessary.

[Clemens Hammacher, 2016-10-11 08:25:13]

The CQ bit was checked by clemensh@chromium.org

[commit-bot: I haz the power, 2016-10-11 08:25:20]

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-10-11 08:28:01]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-11 08:28:55]

Description was changed from

==========
[wasm] Use WasmCompiledModule type in interfaces

Just a minor refactoring. We have the type for compiled modules, so use
it where possible for better type safety.

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

to

==========
[wasm] Use WasmCompiledModule type in interfaces

Just a minor refactoring. We have the type for compiled modules, so use
it where possible for better type safety.

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

Committed: https://crrev.com/1b5861413597a2f9dfec98e9bebc0fad457f726d
Cr-Commit-Position: refs/heads/master@{#40156}
==========

[commit-bot: I haz the power, 2016-10-11 08:28:56]

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