Notification API throws when serializing SharedArrayBuffer

Notification API throws when serializing SharedArrayBuffer

See https://html.spec.whatwg.org/multipage/infrastructure.html#safe-passing-of-structured-data

And the web platform test here:
https://github.com/w3c/web-platform-tests/blob/master/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/serialization-via-notifications-api.any.js

BUG=chromium:718192
Review-Url: https://codereview.chromium.org/2860873002
Cr-Commit-Position: refs/heads/master@{#469225}
Committed: https://chromium.googlesource.com/chromium/src/+/fd0cc89538d0cacf1d4f225e0e6077d1dbd4b966

[binji, 2017-05-03 22:22:05]

Description was changed from

==========
Notification API throws when serializing SharedArrayBuffer

BUG=chromium:718192
==========

to

==========
Notification API throws when serializing SharedArrayBuffer

See
https://html.spec.whatwg.org/multipage/infrastructure.html#safe-passing-of-st...

And the web platform test here:
https://github.com/w3c/web-platform-tests/blob/master/html/infrastructure/saf...

BUG=chromium:718192
==========

[binji, 2017-05-03 22:22:26]

Description was changed from

==========
Notification API throws when serializing SharedArrayBuffer

See
https://html.spec.whatwg.org/multipage/infrastructure.html#safe-passing-of-st...

And the web platform test here:
https://github.com/w3c/web-platform-tests/blob/master/html/infrastructure/saf...

BUG=chromium:718192
==========

to

==========
Notification API throws when serializing SharedArrayBuffer

See
https://html.spec.whatwg.org/multipage/infrastructure.html#safe-passing-of-st...

And the web platform test here:
https://github.com/w3c/web-platform-tests/blob/master/html/infrastructure/saf...

BUG=chromium:718192
==========

[binji, 2017-05-03 22:22:47]

binji@chromium.org changed reviewers:
+ peter@chromium.org

[Peter Beverloo, 2017-05-03 22:35:49]

lgtm

Looking at V8ScriptValueSerializer, this flag is only used for shared arrag
buffers, so that SGTM :). Thanks!

[binji, 2017-05-03 22:39:59]

On 2017/05/03 at 22:35:49, peter wrote:
> lgtm
> 
> Looking at V8ScriptValueSerializer, this flag is only used for shared arrag
buffers, so that SGTM :). Thanks!

I think it will likely be used for WebAssembly modules as well, but for now just
SABs.

[binji, 2017-05-03 23:32:15]

The CQ bit was checked by binji@chromium.org

[commit-bot: I haz the power, 2017-05-03 23:33:14]

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-05-04 01:10:45]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493854335186720, "parent_rev":
"e27ff10e692d0cf3d43e7e2a1c8190cbb62cdbb2", "commit_rev":
"fd0cc89538d0cacf1d4f225e0e6077d1dbd4b966"}

[commit-bot: I haz the power, 2017-05-04 01:11:00]

Description was changed from

==========
Notification API throws when serializing SharedArrayBuffer

See
https://html.spec.whatwg.org/multipage/infrastructure.html#safe-passing-of-st...

And the web platform test here:
https://github.com/w3c/web-platform-tests/blob/master/html/infrastructure/saf...

BUG=chromium:718192
==========

to

==========
Notification API throws when serializing SharedArrayBuffer

See
https://html.spec.whatwg.org/multipage/infrastructure.html#safe-passing-of-st...

And the web platform test here:
https://github.com/w3c/web-platform-tests/blob/master/html/infrastructure/saf...

BUG=chromium:718192

Review-Url: https://codereview.chromium.org/2860873002
Cr-Commit-Position: refs/heads/master@{#469225}
Committed:
https://chromium.googlesource.com/chromium/src/+/fd0cc89538d0cacf1d4f225e0e60...
==========

[commit-bot: I haz the power, 2017-05-04 01:11:01]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fd0cc89538d0cacf1d4f225e0e60...