MediaStream stop() is deprecated

[manuelodelain]

Use MediaStreamTrack stop() instead of MediaStream stop().

https://developers.google.com/web/updates/2015/07/mediastream-deprecations

[jhuckaby]

Thanks for catching this. However, the MediaStream stop() thing was added specifically for a Firefox bug (that's why it is wrapped in a try/catch). I don't want to remove that code, as I think Firefox hasn't yet updated to the recommended MediaStreamTrack implementation, and it would break things in current browser versions. So maybe we can add both implementations?

try { this.stream.stop(); } catch (e) {;}
try { this.stream.getTracks()[0].stop(); } catch (e) {;}

Seem ok? Even after MediaStream stop() is removed, it won't cause problems if it is wrapped in a try/catch.

[manuelodelain]

Seems that Firefox implements MediaStreamTrack.stop() since the version 34.
https://developer.mozilla.org/fr/docs/Web/API/MediaStreamTrack/stop

I don't have problem with the latest Firefox version (40.0.3).

What was the bug with Firefox ?

[jhuckaby]

Darn, I can't find the issue now. It was a while ago. I'll go ahead and implement your change, after I test it on all browsers / platforms. Thanks!