Closed Bug 1750353 Opened 2 years ago Closed 2 years ago

Remove NativeUnderlyingSource

Categories

(Core :: DOM: Streams, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
98 Branch
Tracking Status
firefox98 --- fixed

People

(Reporter: evilpie, Assigned: evilpie)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Everything is specialized for BodyStream anyway, so this base class is superfluous.

Assignee: nobody → evilpies
Status: NEW → ASSIGNED

I thought the idea was that NativeUnderlyingSource will be used by other users in the near future.

In theory, yes. In practice, it's insufficiently well proven to support that use right now, and I would rather someone trying to make a new consumer here not be confused into thinking that this is just going to work because it really might not; step one for trying to add a new consumer may well be to revert this removal, but I'm very supportive of removing this until the story is clearer.

Pushed by evilpies@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/0d726177468a
Remove unused NativeUnderlyingSource. r=mgaudet

Jan, you may want to take a look at this change. I assume we'll need something like NativeUnderlyingSource for OPFS.

Flags: needinfo?(jvarga)

Hard to say at this point, we are currently implementing only basic functionality of OPFS. I will know more once we get to implementing stream support.
Anyway, thanks for letting me know.

Flags: needinfo?(jvarga)
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: