1249015 - Context is still updated in standalone when sharing is stopped

Status: RESOLVED FIXED

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

Description

[Bogdan Maris, Desktop Test Engineering]

[Affected versions]:
- Latest Aurora 46.0a1
- Latest Nightly 47.0a1

[Affected platforms]:
- Mac OS X 10.10.5
- Ubuntu 14.04 32-bit

[Steps to reproduce]:
1. Start Firefox
2. Install the latest Hello add-on (1.1.2 at this moment)
3. Change the loop.server to https://loop-dev.stage.mozaws.net/v0 in about:config
4. Restart Firefox
5. Start a conversation
6. Join the conversation from another browser/PC
7. Click Stop Sharing on the PC the call was started
8. Switch tabs (make sure tabs contain websites and not about: pages)

[Expected result]:
- Context is not updated in Standalone, only chat can be still used.

[Actual result]:
- Context is updated in Standalone when switching tabs on the first PC.

[Regression range]:
- Not a regression, it reproduces since fix from bug 1215612 was pushed in Standalone.

[Additional notes]:
- Unable to check on Firefox Beta or Release due to the incompatibility of the add-on with 45 or 44 (at this moment)

Comment 1

[Ed Lee :Mardak]

Should probably be able to do something similar to not sending pointer events:

https://github.com/mozilla/loop/pull/134/files#diff-5addb23a0c0fbd56768583e5283792adR662

But from _notifyBrowserSwitch perhaps?

Comment 2

[Ed Lee :Mardak]

Ah perhaps not that easy as we still need to let the sdk know the tab changed.. Next simplest might be to pass the paused state to _handleSwitchBrowserShare as well as the window id?

Comment 3

[Bogdan Maris, Desktop Test Engineering]

Also tested on Windows 10 64-bit, same result (also affected).

Comment 4

[GitHub Autolander Bot]

Attachment: [loop] Mardak:bug-1249015-pause-context > mozilla:master

Comment 5

[Manuel Casas Barrado [:mancas]]

Comment on attachment 8720704 [details] [review]
[loop] Mardak:bug-1249015-pause-context > mozilla:master

Looks good to me! r=me with nit addressed

Thank you

Comment 6

[Ed Lee :Mardak]

https://github.com/mozilla/loop/commit/5f94d79b5fb7d820c8bda195e09f710ec32c703a