1217335 - Context is no longer shown on loading the standalone window

Status: VERIFIED FIXED

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

Description

[Mark Banner (:standard8)]

This is a regression from bug 1208201 - not shipped in production yet.

The context is no longer shown in the text area of the standalone display when loading a room.

Bug 1208201 merged a couple of actions, but didn't update all locations properly. Desktop gets away with this as there's an extra update cycle that happens (and uses the right attribute).

Comment 1

[Mark Banner (:standard8)]

Attachment: Room context is no longer shown when loading the Loop standalone UI.

This makes the necessary adjustments. I've gone with roomContextUrls in more places as that's more descriptive than just "urls" (althought he room information we get from the server is just "urls" which is why we have that in some places.

Comment 2

[Ed Lee :Mardak]

Comment on attachment 8677394 [details] [diff] [review]
Room context is no longer shown when loading the Loop standalone UI.

ui-showcase has an UpdateRoomInfo passing in a "urls":

https://dxr.mozilla.org/mozilla-central/source/browser/components/loop/ui/ui-showcase.jsx#332

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/045a0eb88d6a

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/045a0eb88d6a

Comment 5

[Bogdan Maris, Desktop QA]

We did some exploratory testing on Hello using Firefox 44 beta 2 across platforms (Windows 7 64-bit, Windows 10 64-bit, Mac OS X 10.11.1 and Ubuntu 14.04 64-bit) and can confirm that context showed up in Standalone each time.