1406201 - Requesting gUM access with active gUM causes prompt again in http (results in two icons)

Status: RESOLVED DUPLICATE of bug 1335740

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

Description

[Nils Ohlmeier [:drno]]

Attachment: Screen Shot 2017-10-05 at 15.07.23.png

- Open http://mozilla.github.io/webrtc-landing/pc_test.html
- Click the "start" button and "allow" cam access
- Click the "stop" button
- click the "start" button again

Result:
I see a red slow blinking icon for the still active camera access and next to that a grey camera icon with the permission prompt (see attached screen shot).

Expected:
There should not be two icons.

Comment 1

[Nils Ohlmeier [:drno]]

Attachment: Screen Shot 2017-10-05 at 15.13.47.png

To make matters "worse" if I click on the second gUM prompt "Don't Allow" I get permanently a red blinking icon next to a grey crossed out icon. This must be confusing to a user.

Comment 2

[Nils Ohlmeier [:drno]]

This happens only when using HTTP. When HTTPS is used no second prompt appears.

Comment 3

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

We always prompt for every gUM access in http. We opted not to change this in bug 1270572, so we're technically in violation of the spec.

As a result, the UX is a bit confusing, but given the low usage of http, we're likely to remove support first (bug 1335740).

FWIW I think the UX is operating reasonably given those parameters, and this is not a regression. The first red icon reflects the live camera, and the second gray icon reflects that the site is making a second request. When you deny the second request, the gray icon changes to reflect that future requests are "temporarily denied", which you can see and remove by opening the page info drop-down.

It means you're still sharing your camera, yet you're blocking future requests from it at the same time.

Comment 4

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

it=the site

Comment 5

[Florian Quèze [:florian]]

(In reply to Jan-Ivar Bruaroey [:jib] (needinfo? me) from comment #3)

> The first red icon reflects the live camera, and the
> second gray icon reflects that the site is making a second request. When you
> deny the second request, the gray icon changes to reflect that future
> requests are "temporarily denied"

> It means you're still sharing your camera, yet you're blocking future
> requests from it at the same time.

iirc at the time we redesigned this, we debated whether denying a new request means we should stop existing streams from that same domain. We decided we should only do it when the denying is done with "remember this decision" checked (and this is bug 1297047 which was low priority and isn't fixed yet).

Comment 6

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

I think that makes sense.

Comment 7

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Though a corollary to the opposite would be that when a user revokes either mic or camera permission, we revoke both automatically.

Comment 8

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Only happened in http which we no longer support, so we can close this.