Skip to content

Streams: rationalize management of cross realm transforms #37084

@gterzian

Description

@gterzian

Following #36977, the below note can be addressed:

/// Note: it may seem strange to use a pair of options, versus for example an enum.

by merging the two fields into one:

cross_realm_transform: Option<CrossRealmTransform>,

Where CrossRealmTransform would be defined as:

#[derive(Clone, JSTraceable, MallocSizeOf)]
enum CrossRealmTransform {
    Readable(CrossRealmTransformReadable),
    Writable(CrossRealmTransformWritable),
}

and where fixing the code using those concepts should be obvious.

Metadata

Metadata

Assignees

Labels

A-content/scriptRelated to the script threadE-less-complexStraightforward. Recommended for a new contributor.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions