Open Bug 1561594 Opened 5 years ago Updated 1 year ago

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

Categories

(Core :: Internationalization, enhancement, P3)

enhancement

Tracking

()

People

(Reporter: hsivonen, Unassigned, NeedInfo)

References

(Blocks 3 open bugs)

Details

(Keywords: dev-doc-needed)

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.

Blocks: 1561840
Blocks: 1561842
Type: defect → enhancement
Priority: -- → P3
Blocks: resab
Depends on: 1606624

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

Blocks: 1563480
No longer blocks: resab

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

Flags: needinfo?(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.

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.