1354847 - Allow creating SimpleChannel instances from JS

Status: RESOLVED WONTFIX

(Core :: Networking, enhancement, P3)

Description

[Kris Maglione [:kmag]]

Bug 1351163 added a SimpleChannel class to allow creating simple stream-backed channels from C++. There are a few places where we need to do the same in JS, and currently need to resort to InputStreamChannels instead.

Comment 1

[Firefox Bug Husbandry Bot]

Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258

Comment 2

[Sylvestre Ledru [:Sylvestre]]

Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information

Comment 3

[Randell Jesup [:jesup] (needinfo me)]

From the previous bug:

> Sorry for the question, is there a solution also for js implementations, or
> are we supposed to only use cpp for these more complex cases?

I had considered that, but I didn't want to implement it as part of this bug. It would be easy enough to support JS implementations with a [function] interface for the callbacks. I can implement that after this lands if you have a use for it.

there's a little more discussion of JS interfaces there too

Comment 4

[Kershaw Chang [:kershaw]]

Marco, do you know if we still need this?

Comment 5

[Marco Bonardo [:mak]]

(In reply to Kershaw Chang [:kershaw] from comment #4)

Marco, do you know if we still need this?

This was originally filed for some js implemented protocols like page-icon, but from then the protocol has been converted to cpp, so I personally don't have any other use cases. I don't know if others do, but eventually they can file a new bug for them. Imo this could be closed until that use-case is found.

Comment 6

[Kershaw Chang [:kershaw]]

Thanks. I agree that we can close this one.