Closed Bug 1248407 Opened 8 years ago Closed 8 years ago

Leaving a call on the standalone UI doesn't stop using microphone/camera

Categories

(Hello (Loop) :: Client, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: standard8, Assigned: standard8)

References

Details

(Keywords: regression, Whiteboard: [btpp-fix-later][third-party][fixed by bug 1251357])

STR:

On a standalone page using the latest code:

- Enter the room
- Grant access to devices
- Leave the room.

On both FF & Chrome the permission prompts are left open. On FF, the device light is still on as well.

I've identified the cause as the sdk upgrade to 2.7.2 - bug 1237677.

Michael, can you have a look at this please?
Flags: needinfo?(msander)
Blocks: 1248955
I'm not able to reproduce this in our internal test app and I'm not sure how to build hello with 2.7.2. Are there any errors or warnings in the console?
Flags: needinfo?(msander) → needinfo?(standard8)
Hi Mike, you should be able to reproduce it on this link:

https://loop-webapp-dev.stage.mozaws.net/w-dBc3HTHQo#U7flOlJuwpr6TGlu-rdADg
Flags: needinfo?(standard8) → needinfo?(msander)
Blocks: 1248604
Rank: 15
Priority: -- → P1
Whiteboard: [btpp-fix-now]
I've filed an internal bug. I don't know what I was doing the other day but now it's easy to reproduce every time. Sorry about that.
Flags: needinfo?(msander)
For Chrome, leaving the permission prompt open in the address bar is unrelated to the TokBox SDK. It does the same thing on their gum test page.

1. Open https://webrtc.github.io/samples/src/content/getusermedia/gum/
2. In the console stream.getTracks().forEach(function(t) { t.stop(); })
I've just backed out the upgrade to 2.7.2 in bug 1237677, so that we don't accidentally release it.

Dropping the priority of this bug until we get a fix for it.
Rank: 15 → 35
Priority: P1 → P3
Whiteboard: [btpp-fix-now] → [btpp-fix-later][third-party]
In the 2.7 SDK we started preferring MediaStreamTrack.stop() since MediaStream.stop() has been deprecated. It looks like there is a bug in Nightly.

https://bugzilla.mozilla.org/show_bug.cgi?id=1251357
Bug 1251357 has now landed, and I've just reverified that this is fixed now.

Thanks everyone.
Status: NEW → RESOLVED
Closed: 8 years ago
Depends on: 1251357
Resolution: --- → FIXED
Whiteboard: [btpp-fix-later][third-party] → [btpp-fix-later][third-party][fixed by bug 1251357]
You need to log in before you can comment on or make changes to this bug.