Browse Source

ref(LargeVideoManager): simplify

Simplify 'updateParticipantConnStatusIndication' by getting rid of
'showMessage' argument.
master
paweldomas 8 years ago
parent
commit
a3ba28f507
1 changed files with 5 additions and 6 deletions
  1. 5
    6
      modules/UI/videolayout/LargeVideoManager.js

+ 5
- 6
modules/UI/videolayout/LargeVideoManager.js View File

@@ -241,8 +241,7 @@ export default class LargeVideoManager {
241 241
             this.updateParticipantConnStatusIndication(
242 242
                     id,
243 243
                     !overrideAndHide && isConnectionInterrupted,
244
-                    !overrideAndHide && messageKey !== null,
245
-                    messageKey);
244
+                    !overrideAndHide && messageKey);
246 245
 
247 246
             // resolve updateLargeVideo promise after everything is done
248 247
             promise.then(resolve);
@@ -279,19 +278,19 @@ export default class LargeVideoManager {
279 278
      *
280 279
      * @param {string} id the id of remote participant(MUC nickname)
281 280
      * @param {boolean} showProblemsIndication
282
-     * @param {boolean} showMessage
283
-     * @param {string} messageKey the i18n key of the message
281
+     * @param {string|null} messageKey the i18n key of the message which will be
282
+     * displayed on the large video or <tt>null</tt> to hide it.
284 283
      *
285 284
      * @private
286 285
      */
287 286
     updateParticipantConnStatusIndication (
288
-        id, showProblemsIndication, showMessage, messageKey) {
287
+        id, showProblemsIndication, messageKey) {
289 288
 
290 289
         // Apply grey filter on the large video
291 290
         this.videoContainer.showRemoteConnectionProblemIndicator(
292 291
             showProblemsIndication);
293 292
 
294
-        if (!showMessage) {
293
+        if (!messageKey) {
295 294
             // Hide the message
296 295
             this.showRemoteConnectionMessage(false);
297 296
         } else {

Loading…
Cancel
Save