[wasm] Properly handlify WasmCompiledModule::Clone.

[wasm] Properly handlify WasmCompiledModule::Clone.

This is to avoid a dereference of the unhandlified {this} reference
after handlified allocation functions have been called. Combining
handlified and unhandlified style is inherently unsafe.

R=titzer@chromium.org

Committed: https://crrev.com/2ab7a917c9da03ef352e557c95e50820304fb7de
Cr-Commit-Position: refs/heads/master@{#39921}

[Michael Starzinger, 2016-09-30 13:03:12]

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

[commit-bot: I haz the power, 2016-09-30 13:03:15]

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-09-30 13:31:26]

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

[commit-bot: I haz the power, 2016-09-30 13:31:27]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-09-30 16:21:07]

This is to fix the "mjsunit/wasm/asm-wasm" flakes on the waterfall.

[Michael Starzinger, 2016-09-30 16:22:25]

Description was changed from

==========
[wasm] Properly handlify WasmCompiledModule::Clone.

This is to avoid a derefenrece of the unhandlified {this} reference
after handlified allocation functions have been called. Combining
handlified and unhandlified style is inherently unsafe.

R=titzer@chromium.org
==========

to

==========
[wasm] Properly handlify WasmCompiledModule::Clone.

This is to avoid a dereference of the unhandlified {this} reference
after handlified allocation functions have been called. Combining
handlified and unhandlified style is inherently unsafe.

R=titzer@chromium.org
==========

[titzer, 2016-09-30 16:27:00]

lgtm

[titzer, 2016-09-30 16:29:11]

The CQ bit was checked by titzer@chromium.org

[commit-bot: I haz the power, 2016-09-30 16:29:16]

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-09-30 16:31:19]

Description was changed from

==========
[wasm] Properly handlify WasmCompiledModule::Clone.

This is to avoid a dereference of the unhandlified {this} reference
after handlified allocation functions have been called. Combining
handlified and unhandlified style is inherently unsafe.

R=titzer@chromium.org
==========

to

==========
[wasm] Properly handlify WasmCompiledModule::Clone.

This is to avoid a dereference of the unhandlified {this} reference
after handlified allocation functions have been called. Combining
handlified and unhandlified style is inherently unsafe.

R=titzer@chromium.org
==========

[commit-bot: I haz the power, 2016-09-30 16:31:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-30 16:31:51]

Description was changed from

==========
[wasm] Properly handlify WasmCompiledModule::Clone.

This is to avoid a dereference of the unhandlified {this} reference
after handlified allocation functions have been called. Combining
handlified and unhandlified style is inherently unsafe.

R=titzer@chromium.org
==========

to

==========
[wasm] Properly handlify WasmCompiledModule::Clone.

This is to avoid a dereference of the unhandlified {this} reference
after handlified allocation functions have been called. Combining
handlified and unhandlified style is inherently unsafe.

R=titzer@chromium.org

Committed: https://crrev.com/2ab7a917c9da03ef352e557c95e50820304fb7de
Cr-Commit-Position: refs/heads/master@{#39921}
==========

[commit-bot: I haz the power, 2016-09-30 16:31:51]

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