1241427 - New windows should show the correct panel state if the user is already in a conversation

Status: RESOLVED INCOMPLETE

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

Description

[Ada [:adalucinet]]

Reproducible with latest Aurora 45.0a2 and Nightly 46.0a1 (from 2016-01-20)
Affected platforms: Ubuntu 14.04 64-bit, Mac OS X 10.10.5 and Windows 7 64-bit

Note: tested with e10s disabled because of the bug 1225189
Steps to reproduce:
1. Launch Firefox and start a conversation.
2. Send the link to another PC/Fx profile/etc.
3. Detach the conversation windows.
4. Close all the remaining Firefox windows (but not the conversation one!)
5. Open a new window (Ctrl/Cmd + N)
6. Click on Hello button and select the already opened conversation.
7. Repeat step 6 one more time (if “We’re having technical difficulties” is present in the conversation window.)

Expected result: Unable to select an ongoing conversation.

Actual results: 
*'Browse this page with a friend' button is present in Hello panel. 
* Conversation window layout is broken - the shared tab is visible along with a part of the webcam. 
* Chat area vanishes if one line of text is sent.

Additional notes:
1. Screen recording: https://goo.gl/FgbvLs
Note that around 0:40 seconds Firefox hangs and CPU 45%.
2. Also reproducible on stage server.
3. With Nightly 45.0a1 from 2015-11-05 I’m not able to rejoin the same conversation - “We’re having technical difficulties” message is displayed in the conversation window. Will investigate further.
4. Unable to reproduce with 44.0RC build 2 nor 44 beta 9 - I get the same bahavior as with 45.0a1 from step 2. I guess it depends on the option to share tabs automatically when entering a room (bug 1214215 - only on 45 branch)
5. Browser console output with loop.debug.dispatcher and loop.debug.sdk enabled: https://goo.gl/4wVoEn

Comment 1

[Mark Banner (:standard8)]

There's a slightly easier way to reproduce - just open a second window. At which stage you can join the open conversation or start a new one.

Put simply, we're not blocking displaying the correct panel status when opening a new window.

Comment 3

[Mark Banner (:standard8)]

Support for Hello/Loop has been discontinued.

https://support.mozilla.org/kb/hello-status

Hence closing the old bugs. Thank you for your support.