[wasm] cctest - explicit WriteOneByte null termination opt-out

[wasm] cctest - explicit WriteOneByte null termination opt-out

Turns out WriteOneByte writes '\0' by default.

BUG=

Committed: https://crrev.com/5657433e575e038fcb6fa75571bd75f230a028ac
Cr-Commit-Position: refs/heads/master@{#40873}

[Mircea Trofin, 2016-11-09 23:48:22]

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

[commit-bot: I haz the power, 2016-11-09 23:48:24]

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

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

[commit-bot: I haz the power, 2016-11-10 00:15:23]

Dry run: This issue passed the CQ dry run.

[Mircea Trofin, 2016-11-10 00:33:10]

Description was changed from

==========
[wasm] cctest - explicit WriteOneByte null termination opt-out

Turns out WriteOneByte writes '\0' by default.

BUG=
==========

to

==========
[wasm] cctest - explicit WriteOneByte null termination opt-out

Turns out WriteOneByte writes '\0' by default.

BUG=
==========

[Mircea Trofin, 2016-11-10 00:33:10]

mtrofin@chromium.org changed reviewers:
+ bradnelson@chromium.org

[Mircea Trofin, 2016-11-10 00:34:47]

PTAL

Deepti - Patch 1 has your change, too, validating mine is a fix. Patch 2
is what I want to commit. Feel free to commit your change (2490693002) if you
need to.

[gdeepti, 2016-11-10 04:37:24]

gdeepti@chromium.org changed reviewers:
+ gdeepti@chromium.org

[gdeepti, 2016-11-10 04:37:24]

lgtm

Thanks for fixing this! The earlier CL does not need to be committed, as the
current tests do not allocate large chunks of memory, sent it out only as a
reproducible test case that I ran into for a different change.

[Mircea Trofin, 2016-11-10 04:51:19]

The CQ bit was checked by mtrofin@chromium.org

[commit-bot: I haz the power, 2016-11-10 04:51:28]

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 05:17:30]

Description was changed from

==========
[wasm] cctest - explicit WriteOneByte null termination opt-out

Turns out WriteOneByte writes '\0' by default.

BUG=
==========

to

==========
[wasm] cctest - explicit WriteOneByte null termination opt-out

Turns out WriteOneByte writes '\0' by default.

BUG=
==========

[commit-bot: I haz the power, 2016-11-10 05:17:31]

Committed patchset #2 (id:20001)

[bradn, 2016-11-10 05:29:08]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com

[bradn, 2016-11-10 05:29:09]

lgtm

[commit-bot: I haz the power, 2016-11-17 22:28:07]

Description was changed from

==========
[wasm] cctest - explicit WriteOneByte null termination opt-out

Turns out WriteOneByte writes '\0' by default.

BUG=
==========

to

==========
[wasm] cctest - explicit WriteOneByte null termination opt-out

Turns out WriteOneByte writes '\0' by default.

BUG=

Committed: https://crrev.com/5657433e575e038fcb6fa75571bd75f230a028ac
Cr-Commit-Position: refs/heads/master@{#40873}
==========

[commit-bot: I haz the power, 2016-11-17 22:28:08]

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