New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 718193 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit 15 days ago
Closed: Jun 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 716320



Sign in to add a comment

Serializing SharedArrayBuffer via history state should throw

Project Member Reported by binji@chromium.org, May 3 2017

Issue description

This desired behavior is similar to with IndexedDB and the notifications APIs.

See https://crbug.com/716320, and the associated web-platform tests here: https://w3c-test.org/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/
 

Comment 1 by binji@chromium.org, May 3 2017

Blocking: 716320
Project Member

Comment 2 by bugdroid1@chromium.org, May 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2c08e4da26268c0297c10e0579f3851d6c922179

commit 2c08e4da26268c0297c10e0579f3851d6c922179
Author: binji <binji@chromium.org>
Date: Fri May 12 00:11:51 2017

History API throws when serializing a 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-history.html

BUG= chromium:718193 

Review-Url: https://codereview.chromium.org/2857303003
Cr-Commit-Position: refs/heads/master@{#471133}

[modify] https://crrev.com/2c08e4da26268c0297c10e0579f3851d6c922179/third_party/WebKit/LayoutTests/virtual/sharedarraybuffer/external/wpt/html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/serialization-via-history-expected.txt
[modify] https://crrev.com/2c08e4da26268c0297c10e0579f3851d6c922179/third_party/WebKit/Source/bindings/core/v8/serialization/SerializedScriptValue.h
[modify] https://crrev.com/2c08e4da26268c0297c10e0579f3851d6c922179/third_party/WebKit/Source/bindings/core/v8/serialization/V8ScriptValueSerializer.cpp
[modify] https://crrev.com/2c08e4da26268c0297c10e0579f3851d6c922179/third_party/WebKit/Source/bindings/scripts/v8_methods.py
[modify] https://crrev.com/2c08e4da26268c0297c10e0579f3851d6c922179/third_party/WebKit/Source/bindings/scripts/v8_types.py
[modify] https://crrev.com/2c08e4da26268c0297c10e0579f3851d6c922179/third_party/WebKit/Source/bindings/tests/results/core/V8TestObject.cpp
[modify] https://crrev.com/2c08e4da26268c0297c10e0579f3851d6c922179/third_party/WebKit/Source/modules/indexeddb/IDBObjectStore.cpp
[modify] https://crrev.com/2c08e4da26268c0297c10e0579f3851d6c922179/third_party/WebKit/Source/modules/notifications/NotificationData.cpp

Comment 3 by binji@chromium.org, Jun 19 2017

Status: Fixed (was: Assigned)

Sign in to add a comment