NS_NewByteArrayInputStream is now redundant with NS_NewByteInputStream. Let's get rid of it to minimize code/maintenance.

The difference between the two is that ByteArrayInputStream owns the buffer, while ByteInputStream doesn't. That makes them different. For stream converter purposes - the stream converter can be long gone at the time the actual stream is accessed by the consumer

Can we just add a flag to one of them that tells them whether they own the buffer or not? Or make the policy be that the thing always owns the buffer, so the caller has to call memcpy if necessary.

The stream is good. I don't want to remove it.

okay, vrfy wontfix unless warren disagrees...