Closed Bug 994150 Opened 10 years ago Closed 10 years ago

Loop standalone-page does not release camera at the end of a call

Categories

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

Product:
Hello (Loop)
Component:
Client
Platform:
x86
macOS
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla33

People

(Reporter: abr, Unassigned)

References

Details

(Whiteboard: [being fixed by bug 972019]s=mlpnightly2) 

Presently, the link clicker standalone-page keeps the camera open after the other side has closed the connection. This is surprising and somewhat creepy from a user perspective. We should act on the connection-closed event and relinquish the camera.
Blocks: loop_mlp
This is being fixed by bug 972019
Depends on: 972019
Whiteboard: [being fixed by bug 972019]
Blocks: 974875, 972024
No longer blocks: loop_mlp
Priority: -- → P3
Assignee: nobody → nperriault
Hey Niko, I believe this was fixed by the patches you landed for bug 972109.  If so, are you ok with my resolving this as a dup of bug 972109 (with a note that the patches there have the fix for this bug)?
Flags: needinfo?(nperriault)
Priority: P3 → P2
Yes the fix for this has landed along with patches for bug 972019 (not closed atm because other pieces are still being worked out)
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(nperriault)
Resolution: --- → FIXED
Whiteboard: [being fixed by bug 972019] → [being fixed by bug 972019]s=mlpnightly2
Adam, can you please review https://moztrap.mozilla.org/manage/case/14508/ to verify it's sufficient to test this use case?
Flags: needinfo?(adam)
QA Contact: anthony.s.hughes
Whiteboard: [being fixed by bug 972019]s=mlpnightly2 → [being fixed by bug 972019]s=mlpnightly2 [qa?]
Anthony -- this is close, but the hangup sequence is backwards: the desktop user needs to end the call, because the flaw was that the standalone client wasn't relinquishing the camera. Testing steps should be:


> 1. Generate a call URL and share it with a friend
> 2. Have your friend load the call URL and start the call
> 3. Answer the call
> 4. End the call
>
> The call should be disconnected and your friend's camera should be deactivated.
Flags: needinfo?(adam)
Updated, thanks. 

I also confirm this test case passes in Firefox 33 and 34.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Whiteboard: [being fixed by bug 972019]s=mlpnightly2 [qa?] → [being fixed by bug 972019]s=mlpnightly2
Target Milestone: --- → mozilla33
(In reply to Anthony Hughes, QA Mentor (:ashughes) from comment #6)
> Updated, thanks. 

Sorry, I should have been clearer: both the steps and the verification need to be updated. The verification should be changed to "The call should be disconnected and *your* *friend's* camera should be deactivated."
Flags: needinfo?(anthony.s.hughes)
Thanks Adam, fixed.
Flags: needinfo?(anthony.s.hughes)
You need to log in before you can comment on or make changes to this bug.