review remarks

GetStream · Jan 16, 2025 · 92e3994 · 92e3994
1 parent a1df414
commit 92e3994
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/packages/react-native-sdk/src/providers/StreamCall/AppStateListener.tsx b/packages/react-native-sdk/src/providers/StreamCall/AppStateListener.tsx
@@ -1,5 +1,5 @@
 import { useCall } from '@stream-io/video-react-bindings';
-import { useEffect, useRef, useState } from 'react';
+import { useEffect, useRef } from 'react';
 import {
   AppState,
   NativeEventEmitter,
@@ -23,8 +23,7 @@ const isAndroid8OrAbove = Platform.OS === 'android' && Platform.Version >= 26;
 export const AppStateListener = () => {
   const call = useCall();
   const appState = useRef(AppState.currentState);
-  const [cameraDisabledByAppState, setCameraDisabledByAppState] =
-    useState<boolean>(false);
+  const cameraDisabledByAppState = useRef<boolean>(false);
 
   // on mount: set initial PiP mode and listen to PiP events
   useEffect(() => {
@@ -69,9 +68,9 @@ export const AppStateListener = () => {
             call?.camera?.enable();
           });
         } else {
-          if (cameraDisabledByAppState) {
+          if (cameraDisabledByAppState.current) {
             call?.camera?.resume();
-            setCameraDisabledByAppState(false);
+            cameraDisabledByAppState.current = false;
           }
         }
         appState.current = nextAppState;
@@ -84,7 +83,7 @@ export const AppStateListener = () => {
           // in PiP mode, we don't want to disable the camera
           const disableCameraIfNeeded = () => {
             if (call?.camera?.state.status === 'enabled') {
-              setCameraDisabledByAppState(true);
+              cameraDisabledByAppState.current = true;
               call?.camera?.disable();
             }
           };
@@ -114,7 +113,7 @@ export const AppStateListener = () => {
           // shouldDisableIOSLocalVideoOnBackgroundRef is false, if local video is enabled on PiP
           if (shouldDisableIOSLocalVideoOnBackgroundRef.current) {
             if (call?.camera?.state.status === 'enabled') {
-              setCameraDisabledByAppState(true);
+              cameraDisabledByAppState.current = true;
               call?.camera?.disable();
             }
           }
@@ -126,7 +125,7 @@ export const AppStateListener = () => {
     return () => {
       subscription.remove();
     };
-  }, [call, cameraDisabledByAppState]);
+  }, [call]);
 
   return null;
 };