Blink-compatible serialization of ArrayBuffer.

Blink-compatible serialization of ArrayBuffer.

Transferral is not included in this CL, nor is SharedArrayBuffer.

BUG=chromium:148757
Committed: https://crrev.com/2646749cbc3a004b6241c699caacd6fbf316049f
Cr-Commit-Position: refs/heads/master@{#38913}

[jbroman, 2016-08-24 18:06:16]

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

[commit-bot: I haz the power, 2016-08-24 18:06:20]

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-08-24 18:08:09]

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

[commit-bot: I haz the power, 2016-08-24 18:08:10]

Dry run: Try jobs failed on following builders:
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/7597)

[jbroman, 2016-08-24 18:12:07]

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

[commit-bot: I haz the power, 2016-08-24 18:12:13]

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-08-24 18:13:40]

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

[commit-bot: I haz the power, 2016-08-24 18:13:40]

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_linux_arm64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/11539)
  v8_linux_gcc_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_gcc_compile_rel/bu...)
  v8_linux_mips64el_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_mipsel_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_linux_nodcheck_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_linux_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/11479)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/22513)

[jbroman, 2016-08-24 21:32:31]

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

[commit-bot: I haz the power, 2016-08-24 21:32:39]

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-08-24 21:39:09]

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

[commit-bot: I haz the power, 2016-08-24 21:39:10]

Dry run: Try jobs failed on following builders:
  v8_win_compile_dbg on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_compile_dbg/builds/2...)

[jbroman, 2016-08-24 21:42:49]

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

[commit-bot: I haz the power, 2016-08-24 21:43:00]

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

[jbroman, 2016-08-24 22:11:26]

jbroman@chromium.org changed reviewers:
+ jkummerow@chromium.org

[commit-bot: I haz the power, 2016-08-24 23:13:17]

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

[commit-bot: I haz the power, 2016-08-24 23:13:18]

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

[Jakob Kummerow, 2016-08-25 08:59:36]

lgtm

https://codereview.chromium.org/2264403004/diff/60001/src/value-serializer.cc
File src/value-serializer.cc (right):

https://codereview.chromium.org/2264403004/diff/60001/src/value-serializer.cc...
src/value-serializer.cc:93: // Array buffer. byteLength:uint32_t, then draw
data.
nit: s/draw/raw/?

https://codereview.chromium.org/2264403004/diff/60001/src/value-serializer.cc...
src/value-serializer.cc:506: return Nothing<bool>();
nit: {}

[jbroman, 2016-08-25 14:14:45]

The CQ bit was checked by jbroman@chromium.org

[jbroman, 2016-08-25 14:14:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/2264403004/#ps100001
(title: "review comments")

[commit-bot: I haz the power, 2016-08-25 14:14:58]

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-08-25 14:43:03]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-08-25 14:44:06]

Description was changed from

==========
Blink-compatible serialization of ArrayBuffer.

Transferral is not included in this CL, nor is SharedArrayBuffer.

BUG=chromium:148757
==========

to

==========
Blink-compatible serialization of ArrayBuffer.

Transferral is not included in this CL, nor is SharedArrayBuffer.

BUG=chromium:148757

Committed: https://crrev.com/2646749cbc3a004b6241c699caacd6fbf316049f
Cr-Commit-Position: refs/heads/master@{#38913}
==========

[commit-bot: I haz the power, 2016-08-25 14:44:07]

Patchset 6 (id:??) landed as
https://crrev.com/2646749cbc3a004b6241c699caacd6fbf316049f
Cr-Commit-Position: refs/heads/master@{#38913}