798037 - There is no way to release the local device (e.g. video, audio) aquired via mozGetUserMedia() through the LocalMediaStream object

Status: RESOLVED DUPLICATE of bug 803976

(Core :: WebRTC: Audio/Video, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Right now it's not possible to release the local device (e.g. camera) once the code below has been ran:

    navigator.mozGetUserMedia({video:true}, function (stream) {
      // Will fail because we do not return a LocalMediaStream
      stream.stop();
    }

It also applies to all other types of media.

I will attach a Mochitest which should be checked-in together with the patch in a bit.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Possible Mochitest

This is a potential Mochitest for this specific bug. I'm not sure which interface Randell was referring to on IRC so I haven't implemented a specific check here yet.

Anant, can you please give feedback in how this bug can be solved?

Comment 2

[Anant Narayanan [:anant]]

In your code snippet, the stream isn't actually assigned anywhere, so the camera should not turn on. Is that not the case?

In order to release the local device, just de-assign the stream from any active <video>/<audio> element. We should support stop() too, of course, but don't currently.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

(In reply to Anant Narayanan [:anant] from comment #2)
> In your code snippet, the stream isn't actually assigned anywhere, so the
> camera should not turn on. Is that not the case?

Right, that's what I see here. The green LED on my MBP gets green.

> In order to release the local device, just de-assign the stream from any
> active <video>/<audio> element. We should support stop() too, of course, but
> don't currently.

So probably we have two issues here?

Comment 4

[Anant Narayanan [:anant]]

Yes, sounds like we have 2 bugs that need to be addressed.

Comment 5

[Anant Narayanan [:anant]]

I recall there was already a bug open about the camera turning on early, Jason?

Comment 6

[Timothy B. Terriberry (:derf)]

(In reply to Anant Narayanan [:anant] from comment #5)
> I recall there was already a bug open about the camera turning on early,
> Jason?

Bug 782191.

Comment 7

[Jason Smith [:jsmith]]

Updating title. It is technically possible to release the local device if you add and remove the stream from a HTMLMediaElement, although that's a different bug. The real issue here is that you can't do it through the stream object.

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Not sure why bug 803976 has been filed separately but I would suggest to keep that open for a specific mochitest. If you disagree we can also take bug 803976 for it.

Comment 9

[Jason Smith [:jsmith]]

This will be fixed by bug 803976. I'm duping on that bug.