From 333a0f5f90e4de51ee0a92139502c162090b70b0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sa=C3=BAl=20Ibarra=20Corretg=C3=A9?= <saghul@jitsi.org>
Date: Wed, 20 Jun 2018 15:45:22 +0200
Subject: [PATCH 1/2] [RN] Handle presence

Up until now, mobile was oblivious to participants' presence state. Presence
state handling is required (probably, amongst other things) for "call flows".
So, let's add it! This is done by gathering the presence state when a
participant first joins, and handling subsequent changes.
---
 react/features/base/conference/actions.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/react/features/base/conference/actions.js b/react/features/base/conference/actions.js
index fb28781a7..ff86b55cb 100644
--- a/react/features/base/conference/actions.js
+++ b/react/features/base/conference/actions.js
@@ -15,6 +15,7 @@ import {
     participantConnectionStatusChanged,
     participantJoined,
     participantLeft,
+    participantPresenceChanged,
     participantRoleChanged,
     participantUpdated
 } from '../participants';
@@ -146,6 +147,7 @@ function _addConferenceListeners(conference, dispatch) {
             conference,
             id,
             name: user.getDisplayName(),
+            presence: user.getStatus(),
             role: user.getRole()
         })));
     conference.on(
@@ -155,6 +157,9 @@ function _addConferenceListeners(conference, dispatch) {
     conference.on(
         JitsiConferenceEvents.USER_ROLE_CHANGED,
         (...args) => dispatch(participantRoleChanged(...args)));
+    conference.on(
+        JitsiConferenceEvents.USER_STATUS_CHANGED,
+        (...args) => dispatch(participantPresenceChanged(...args)));
 
     conference.addCommandListener(
         AVATAR_ID_COMMAND,

From 7b0a6a2ee58a709c6d79f06518b3d246935e02a8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sa=C3=BAl=20Ibarra=20Corretg=C3=A9?= <saghul@jitsi.org>
Date: Wed, 20 Jun 2018 15:36:28 +0200
Subject: [PATCH 2/2] [RN] Add ability to loop sounds

---
 .../base/media/components/AbstractAudio.js    | 21 +++++++++------
 .../base/media/components/native/Audio.js     | 26 +++++++++++++------
 2 files changed, 31 insertions(+), 16 deletions(-)

diff --git a/react/features/base/media/components/AbstractAudio.js b/react/features/base/media/components/AbstractAudio.js
index 28c3d6853..93319db07 100644
--- a/react/features/base/media/components/AbstractAudio.js
+++ b/react/features/base/media/components/AbstractAudio.js
@@ -10,7 +10,8 @@ export type AudioElement = {
     currentTime?: number,
     pause: () => void,
     play: () => void,
-    setSinkId?: string => void
+    setSinkId?: string => void,
+    stop: () => void
 };
 
 /**
@@ -61,8 +62,6 @@ export default class AbstractAudio extends Component<Props> {
         this.setAudioElementImpl = this.setAudioElementImpl.bind(this);
     }
 
-    pause: () => void;
-
     /**
      * Attempts to pause the playback of the media.
      *
@@ -73,10 +72,8 @@ export default class AbstractAudio extends Component<Props> {
         this._audioElementImpl && this._audioElementImpl.pause();
     }
 
-    play: () => void;
-
     /**
-     * Attempts to being the playback of the media.
+     * Attempts to begin the playback of the media.
      *
      * @public
      * @returns {void}
@@ -106,8 +103,6 @@ export default class AbstractAudio extends Component<Props> {
         typeof setRef === 'function' && setRef(element ? this : null);
     }
 
-    setSinkId: string => void;
-
     /**
      * Sets the sink ID (output device ID) on the underlying audio element.
      * NOTE: Currently, implemented only on Web.
@@ -120,4 +115,14 @@ export default class AbstractAudio extends Component<Props> {
             && typeof this._audioElementImpl.setSinkId === 'function'
             && this._audioElementImpl.setSinkId(sinkId);
     }
+
+    /**
+     * Attempts to stop the playback of the media.
+     *
+     * @public
+     * @returns {void}
+     */
+    stop(): void {
+        this._audioElementImpl && this._audioElementImpl.stop();
+    }
 }
diff --git a/react/features/base/media/components/native/Audio.js b/react/features/base/media/components/native/Audio.js
index 418c75ce7..e9007b2f7 100644
--- a/react/features/base/media/components/native/Audio.js
+++ b/react/features/base/media/components/native/Audio.js
@@ -12,11 +12,10 @@ const logger = require('jitsi-meet-logger').getLogger(__filename);
  * {@link RTCView}.
  */
 export default class Audio extends AbstractAudio {
-
     /**
      * Reference to 'react-native-sound} {@link Sound} instance.
      */
-    _sound: Sound
+    _sound: ?Sound;
 
     /**
      * A callback passed to the 'react-native-sound''s {@link Sound} instance,
@@ -56,9 +55,22 @@ export default class Audio extends AbstractAudio {
      */
     componentWillUnmount() {
         if (this._sound) {
-            this.setAudioElementImpl(null);
             this._sound.release();
             this._sound = null;
+            this.setAudioElementImpl(null);
+        }
+    }
+
+    /**
+     * Attempts to begin the playback of the media.
+     *
+     * @inheritdoc
+     * @override
+     */
+    play() {
+        if (this._sound) {
+            this._sound.setNumberOfLoops(this.props.loop ? -1 : 0);
+            super.play();
         }
     }
 
@@ -81,10 +93,8 @@ export default class Audio extends AbstractAudio {
      * @returns {void}
      */
     stop() {
-        // Currently not implemented for mobile. If needed, a possible
-        // implementation is:
-        // if (this._sound) {
-        //     this._sound.stop();
-        // }
+        if (this._sound) {
+            this._sound.stop();
+        }
     }
 }