-
Shu-yu Guo authored
This CL has two changes: 1. Remove ValueDeserializer::Delegate::SupportsSharedValues. Only ValueSerializer::Delegate needs to report whether it supports serializing shared values. The ValueDeserializer::Delegate should DCHECK if it gets a shared object tag but it doesn't support it. This better mirrors what happens with SharedArrayBuffer transfers currently. 2. When attempting to serialize a shared object (shared struct, shared array, Atomics.Mutex, or Atomics.Condition) when !SupportsSharedValues(), throw instead of assert. This is for better ergonomics. Bug: v8:12547 Change-Id: I2bb66830393526578016813c4e3488859dd07073 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3866302 Commit-Queue: Camillo Bruni <cbruni@chromium.org> Auto-Submit: Shu-yu Guo <syg@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/main@{#82870}
d855d7f7
Name |
Last commit
|
Last update |
---|---|---|
.. | ||
OWNERS | ||
async-hooks-wrapper.cc | ||
async-hooks-wrapper.h | ||
cov.cc | ||
cov.h | ||
d8-console.cc | ||
d8-console.h | ||
d8-js.cc | ||
d8-platforms.cc | ||
d8-platforms.h | ||
d8-posix.cc | ||
d8-test.cc | ||
d8-windows.cc | ||
d8.cc | ||
d8.h |