[wasm] Change WebAssembly flags option to only cover structured cloning.

[wasm] Change WebAssembly flags option to only cover structured cloning.

BUG=575167
BUG=686457
R=jochen@chromium.org,mtrofin@chromium.org

Review-Url: https://codereview.chromium.org/2642793003
Cr-Commit-Position: refs/heads/master@{#445293}
Committed: https://chromium.googlesource.com/chromium/src/+/2730e3510753fefbc760b09a3c4d3ace5c585a36

[bradn, 2017-01-19 01:23:05]

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

[bradnelson, 2017-01-19 01:23:08]

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

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

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

[Mircea Trofin, 2017-01-19 02:17:09]

lgtm

[bradn, 2017-01-19 02:17:56]

Need LG from jochen for OWNERS

[commit-bot: I haz the power, 2017-01-19 02:59:46]

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

[commit-bot: I haz the power, 2017-01-19 02:59:47]

Dry run: This issue passed the CQ dry run.

[bradnelson, 2017-01-19 17:35:47]

Description was changed from

==========
[wasm] Change WebAssembly flags option to only cover structured cloning.

BUG=575167
R=jochen@chromium.org,mtrofin@chromium.org
==========

to

==========
[wasm] Change WebAssembly flags option to only cover structured cloning.

BUG=575167
R=jochen@chromium.org,mtrofin@chromium.org
==========

[bradnelson, 2017-01-19 17:35:47]

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

[Mircea Trofin, 2017-01-19 22:22:22]

On 2017/01/19 02:17:56, bradn wrote:
> Need LG from jochen for OWNERS

Actually, the change needs to additionally tie, in v8/src/value-serializer.cc,
ValueDeserializer::ReadWasmModule and ValueSerializer::WriteWasmModule, to a new
flag on the v8 side (say, FLAG_wasm_enable_serialization), and then tie that
flag to the Chrome flag. Right now, those APIs ({Read|Write}WasmModule) just
work. (one is tied to FLAG_expose_wasm, but that's on)

[bradnelson, 2017-01-20 00:54:01]

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

[bradn, 2017-01-20 00:54:39]

Revised to disable v8 side of structured cloning.

[commit-bot: I haz the power, 2017-01-20 00:55:36]

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

[bradn, 2017-01-20 00:55:51]

Ping? jochen

[commit-bot: I haz the power, 2017-01-20 02:04:19]

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

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

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2017-01-21 17:23:36]

lgtm

[bradn, 2017-01-21 18:40:57]

The CQ bit was checked by bradnelson@google.com

[bradn, 2017-01-21 18:40:58]

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

[commit-bot: I haz the power, 2017-01-21 18:41:02]

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-21 20:32:25]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485024057644430,
"parent_rev": "0df262f32f773d7655aba8b5990ffbc2d04cc622", "commit_rev":
"2730e3510753fefbc760b09a3c4d3ace5c585a36"}

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

Description was changed from

==========
[wasm] Change WebAssembly flags option to only cover structured cloning.

BUG=575167
R=jochen@chromium.org,mtrofin@chromium.org
==========

to

==========
[wasm] Change WebAssembly flags option to only cover structured cloning.

BUG=575167
R=jochen@chromium.org,mtrofin@chromium.org

Review-Url: https://codereview.chromium.org/2642793003
Cr-Commit-Position: refs/heads/master@{#445293}
Committed:
https://chromium.googlesource.com/chromium/src/+/2730e3510753fefbc760b09a3c4d...
==========

[commit-bot: I haz the power, 2017-01-21 20:33:02]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/2730e3510753fefbc760b09a3c4d...

[bradn, 2017-01-28 18:50:12]

Description was changed from

==========
[wasm] Change WebAssembly flags option to only cover structured cloning.

BUG=575167
R=jochen@chromium.org,mtrofin@chromium.org

Review-Url: https://codereview.chromium.org/2642793003
Cr-Commit-Position: refs/heads/master@{#445293}
Committed:
https://chromium.googlesource.com/chromium/src/+/2730e3510753fefbc760b09a3c4d...
==========

to

==========
[wasm] Change WebAssembly flags option to only cover structured cloning.

BUG=575167
BUG=686457
R=jochen@chromium.org,mtrofin@chromium.org

Review-Url: https://codereview.chromium.org/2642793003
Cr-Commit-Position: refs/heads/master@{#445293}
Committed:
https://chromium.googlesource.com/chromium/src/+/2730e3510753fefbc760b09a3c4d...
==========