1561594 - Enable SharedArrayBuffer arguments for TextEncoder.encodeInto() and TextDecode.decode()

Status: NEW

(Core :: Internationalization, enhancement, P3)

Description

[Henri Sivonen (:hsivonen)]

As the first step, enable shared in WebIDL and make the implementation use memcpySafeWhenRacy so that Web content doesn't need to be responsible for the copy.

Optimizing away the copy is explicitly not in the scope of this bug.

Comment 1

[Anne (:annevk)]

This doesn't block resab, but would be nice to have. Let's track it as part of follow-up.

Comment 2

[Henri Sivonen (:hsivonen)]

Needinfoing self to remind me that the spec has now changed.

Comment 3

[Henri Sivonen (:hsivonen)]

(In reply to Henri Sivonen (:hsivonen) from comment #0)

As the first step, enable shared in WebIDL and make the implementation use memcpySafeWhenRacy so that Web content doesn't need to be responsible for the copy.

We should just go ahead and implement this using and intermediate buffer and memcpySafeWhenRacy.

Optimizing away the copy is explicitly not in the scope of this bug.

I re-read https://github.com/rust-lang/unsafe-code-guidelines/issues/152 today. Getting the formal foundations of this part in order such that the result would be formally correct as opposed to being a YOLO approach is likely to still take years.

Comment 4

[Henri Sivonen (:hsivonen)]

Comment 3 still holds. Clearing needinfo to signal that it's OK to take this bug.