1072095 - Double conversation windows when starting a call

Status: RESOLVED WORKSFORME

(Hello (Loop) :: Client, defect)

Description

[u279076]

Tested with two Firefox Aurora 34.0a2 20140923004002 instances running with new profiles on the same system:

1. User X clicks the Hello button and copies the URL
2. User Y loads the copied URL and clicks the start button
> User X sees a single conversation window and hears a ringing notification

3. User X accepts the incoming call then hangs up soon afterwards
4. User X closes the conversation window
5. User X clicks the Hello button and copies the URL again (this URL should be different from the previous)
6. User Y loads the copied URL and clicks the start button
> User X sees two conversation windows appear with a double audible notification

Marking this as a blocker since I think this should block shipping Hello in Firefox 34.

Comment 1

[u279076]

I can reproduce this in Firefox 33.0b3 so I don't think this is a regression.

Comment 2

[u279076]

Maire, I'm needinfoing you here just to increase visibility of this issue. Please let me know if there's something I can do to help debug this further.

Comment 3

[Mark Banner (:standard8)]

I haven't yet reproduced this with nightly. My suspicion would be an interaction between the server and the client, causing multiple call requests to be provided/triggered.

We could probably really do with some logging on the push notifications / MozLoopService to be able to determine what's really happening. I think MattN was looking at adding some of this, but I'm not sure where that work is.

Comment 4

[Matthew N. [:MattN]]

(In reply to Mark Banner (:standard8) from comment #3)
> We could probably really do with some logging on the push notifications /
> MozLoopService to be able to determine what's really happening. I think
> MattN was looking at adding some of this, but I'm not sure where that work
> is.

That's bug 1073027 now.

Comment 5

[Maire Reavy [:mreavy]]

Anthony -- Have you seen this on Nightly (Fx35)?  I'm more interested in seeing if we can repro on Nightly;  we have logging there too.

Comment 6

[u279076]

I just tried my steps in today's Nightly and cannot reproduce so I suspect this can be WONTFIXED for Aurora if we want to.

Comment 7

[Maire Reavy [:mreavy]]

I believe this was fixed by our uplifts.