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] [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();
+        }
     }
 }