StandaloneRoomView.shouldRenderRemoteVideo: unexpected roomState: room-gather

RESOLVED FIXED in Firefox 42

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: pdehaan, Assigned: standard8)

Tracking

unspecified
mozilla42
Points:
1

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Using FF40.

Was doing a call with the handsome Mr :rpapa and saw this error float by in my dev tools console:

> StandaloneRoomView.shouldRenderRemoteVideo: unexpected roomState:  room-gather
> standaloneRoomViews.js:413:

Which is roughly https://github.com/mozilla/loop-client/blob/2b2c696eb524fd4428724c08c85c3ad91d724f41/content/js/standaloneRoomViews.jsx#L390 (not sure why the line number mis-match, but maybe they repos are out of sync).
(Assignee)

Comment 1

3 years ago
Created attachment 8637417 [details] [diff] [review]
Fix a warning in StandaloneRoomView.shouldRenderRemoteVideo when the roomState is gather.

This fixes the warning, there's no harm to it as the default is the same.
Attachment #8637417 - Flags: review?(mdeboer)
(Assignee)

Updated

3 years ago
Assignee: nobody → standard8
Iteration: --- → 42.2 - Jul 27
Points: --- → 1
Comment on attachment 8637417 [details] [diff] [review]
Fix a warning in StandaloneRoomView.shouldRenderRemoteVideo when the roomState is gather.

Review of attachment 8637417 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM!
Attachment #8637417 - Flags: review?(mdeboer) → review+
https://hg.mozilla.org/mozilla-central/rev/24d4e869eec9
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox42: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.