@azure/communication-react 1.5.1-alpha-202301220014 → 1.5.1-alpha-202301250013

package/dist/dist-cjs/communication-react/index.js

@@ -159,7 +159,7 @@ const _toCommunicationIdentifier = (id) => {
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
 // GENERATED FILE. DO NOT EDIT MANUALLY.
-var telemetryVersion = '1.5.1-alpha-202301220014';
+var telemetryVersion = '1.5.1-alpha-202301250013';
 
 // Copyright (c) Microsoft Corporation.
 /**
@@ -1120,7 +1120,9 @@ var EventNames;
     EventNames["LOCAL_STREAM_RENDERING"] = "LOCAL_STREAM_RENDERING";
     EventNames["REMOTE_STREAM_ALREADY_RENDERED"] = "REMOTE_STREAM_ALREADY_RENDERED";
     EventNames["REMOTE_STREAM_RENDERING"] = "REMOTE_STREAM_RENDERING";
+    EventNames["REMOTE_STREAM_ALREADY_DISPOSED"] = "REMOTE_STREAM_ALREADY_DISPOSED";
     EventNames["REMOTE_STREAM_STOPPING"] = "REMOTE_STREAM_STOPPING";
+    EventNames["REMOTE_CREATED_STREAM_STOPPING"] = "REMOTE_CREATED_STREAM_STOPPING";
     // Error
     EventNames["LOCAL_STREAM_NOT_FOUND"] = "LOCAL_STREAM_NOT_FOUND";
     EventNames["LOCAL_RENDER_INFO_NOT_FOUND"] = "LOCAL_RENDER_INFO_NOT_FOUND";
@@ -2039,8 +2041,15 @@ function createViewRemoteVideo(context, internalContext, callId, participantId,
             });
             return;
         }
+        // "Stopping" only happens if the stream was in "rendering" but `disposeView` was called.
+        // Now that `createView` has been re-called, we can flip the state back to "rendering".
         if (renderInfo.status === 'Stopping') {
-            console.warn('RemoteVideoStream is in the middle of stopping');
+            _logEvent(callingStatefulLogger, {
+                name: EventNames.REMOTE_STREAM_STOPPING,
+                level: 'warning',
+                message: 'RemoteVideoStream was marked as stopping by dispose view. Resetting state to "Rendering".'
+            });
+            internalContext.setRemoteRenderInfo(callId, participantKey, streamId, renderInfo.stream, 'Rendering', renderInfo.renderer);
             return;
         }
         const renderer = new communicationCalling.VideoStreamRenderer(renderInfo.stream);
@@ -2059,6 +2068,8 @@ function createViewRemoteVideo(context, internalContext, callId, participantId,
             internalContext.setRemoteRenderInfo(callId, participantKey, streamId, renderInfo.stream, 'NotRendered', undefined);
             throw e;
         }
+        // Since render could take some time, we need to check if the stream is still valid and if we received a signal to
+        // stop rendering.
         const refreshedRenderInfo = internalContext.getRemoteRenderInfoForParticipant(callId, participantKey, streamId);
         if (!refreshedRenderInfo) {
             // RenderInfo was removed. This should not happen unless stream was removed from the call so dispose the renderer
@@ -2066,7 +2077,7 @@ function createViewRemoteVideo(context, internalContext, callId, participantId,
             _logEvent(callingStatefulLogger, {
                 name: EventNames.REMOTE_RENDER_INFO_NOT_FOUND,
                 level: 'error',
-                message: '`Cannot find remote render info after create the view.',
+                message: 'Cannot find remote render info after create the view.',
                 data: streamLogInfo
             });
             renderer.dispose();
@@ -2076,21 +2087,29 @@ function createViewRemoteVideo(context, internalContext, callId, participantId,
         if (refreshedRenderInfo.status === 'Stopping') {
             // Stop render was called on this stream after we had started rendering. We will dispose this view and do not
             // put the view into the state.
-            renderer.dispose();
-            internalContext.setRemoteRenderInfo(callId, participantKey, streamId, refreshedRenderInfo.stream, 'NotRendered', undefined);
-            context.setRemoteVideoStreamRendererView(callId, participantKey, streamId, undefined);
             _logEvent(callingStatefulLogger, {
-                name: EventNames.REMOTE_VIEW_RENDER_SUCCEED,
-                level: 'info',
-                message: 'Successfully render the remote view.',
+                name: EventNames.REMOTE_CREATED_STREAM_STOPPING,
+                level: 'warning',
+                message: 'Render info status is stopping, dispose renderer.',
                 data: streamLogInfo
             });
+            renderer.dispose();
+            internalContext.setRemoteRenderInfo(callId, participantKey, streamId, refreshedRenderInfo.stream, 'NotRendered', undefined);
+            context.setRemoteVideoStreamRendererView(callId, participantKey, streamId, undefined);
             return;
         }
         // Else the stream still exists and status is not telling us to stop rendering. Complete the render process by
         // updating the state.
         internalContext.setRemoteRenderInfo(callId, participantKey, streamId, refreshedRenderInfo.stream, 'Rendered', renderer);
         context.setRemoteVideoStreamRendererView(callId, participantKey, streamId, convertFromSDKToDeclarativeVideoStreamRendererView(view));
+        _logEvent(callingStatefulLogger, {
+            name: EventNames.REMOTE_VIEW_RENDER_SUCCEED,
+            level: 'info',
+            message: `Successfully render the remote view.`,
+            data: {
+                streamLogInfo
+            }
+        });
         return {
             renderer,
             view
@@ -2187,7 +2206,7 @@ function createViewLocalVideo(context, internalContext, callId, options) {
             _logEvent(callingStatefulLogger, {
                 name: EventNames.LOCAL_CREATED_STREAM_STOPPING,
                 level: 'warning',
-                message: 'Render info status is stopping, dispose renderer. ',
+                message: 'Render info status is stopping, dispose renderer.',
                 data: {
                     callId
                 }
@@ -2301,9 +2320,31 @@ function disposeViewRemoteVideo(context, internalContext, callId, participantId,
         });
         return;
     }
-    // Sets the status and also renderer. I think we need to always set renderer to undefined since in all status when
-    // cleaned up should have renderer as undefined. If the status is 'Rendered' and renderer is not defined it should
-    // be cleaned up below so we can set it to undefined here.
+    // Nothing to dispose of or clean up -- we can safely exit early here.
+    if (renderInfo.status === 'NotRendered') {
+        _logEvent(callingStatefulLogger, {
+            name: EventNames.REMOTE_STREAM_ALREADY_DISPOSED,
+            level: 'info',
+            message: 'RemoteVideoStream is already disposed.',
+            data: streamLogInfo
+        });
+        return;
+    }
+    // Status is already marked as "stopping" so we can exit early here. This is because stopping only occurs
+    // when the stream is being created in createView but hasn't been completed being created yet. The createView
+    // method will see the "stopping" status and perform the cleanup
+    if (renderInfo.status === 'Stopping') {
+        _logEvent(callingStatefulLogger, {
+            name: EventNames.REMOTE_STREAM_STOPPING,
+            level: 'info',
+            message: 'Remote stream is already stopping.',
+            data: streamLogInfo
+        });
+        return;
+    }
+    // If the stream is in the middle of being rendered (i.e. has state "Rendering"), we need the status as
+    // "stopping" without performing any cleanup. This will tell the `createView` method that it should stop
+    // rendering and clean up the state once the view has finished being created.
     if (renderInfo.status === 'Rendering') {
         _logEvent(callingStatefulLogger, {
             name: EventNames.REMOTE_STREAM_STOPPING,
@@ -2312,10 +2353,10 @@ function disposeViewRemoteVideo(context, internalContext, callId, participantId,
             data: streamLogInfo
         });
         internalContext.setRemoteRenderInfo(callId, participantKey, streamId, renderInfo.stream, 'Stopping', undefined);
+        return;
     }
-    else {
-        internalContext.setRemoteRenderInfo(callId, participantKey, streamId, renderInfo.stream, 'NotRendered', undefined);
-    }
+    // Else the state must be in the "Rendered" state, so we can dispose the renderer and clean up the state.
+    internalContext.setRemoteRenderInfo(callId, participantKey, streamId, renderInfo.stream, 'NotRendered', undefined);
     if (renderInfo.renderer) {
         _logEvent(callingStatefulLogger, {
             name: EventNames.DISPOSING_REMOTE_RENDERER,
@@ -12709,7 +12750,7 @@ const useSelector$4 = (selector, selectorProps) => {
  * @public
  */
 const errorBarSelector$1 = reselect.createSelector([getLatestErrors$1, getDiagnostics, getDeviceManager$1, getEnvironmentInfo], (latestErrors, diagnostics, deviceManager, environmentInfo) => {
-    var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o;
+    var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q;
     // The order in which the errors are returned is significant: The `ErrorBar` shows errors on the UI in that order.
     // There are several options for the ordering:
     //   - Sorted by when the errors happened (latest first / oldest first).
@@ -12785,9 +12826,17 @@ const errorBarSelector$1 = reselect.createSelector([getLatestErrors$1, getDiagno
     appendActiveErrorIfDefined(activeErrorMessages, latestErrors, 'Call.stopVideo', 'stopVideoGeneric');
     appendActiveErrorIfDefined(activeErrorMessages, latestErrors, 'Call.mute', 'muteGeneric');
     appendActiveErrorIfDefined(activeErrorMessages, latestErrors, 'Call.stopScreenSharing', 'stopScreenShareGeneric');
-    appendActiveErrorIfDefined(activeErrorMessages, latestErrors, 'Call.startVideo', 'startVideoGeneric');
+    if (((_o = latestErrors['Call.startVideo']) === null || _o === void 0 ? void 0 : _o.message) === 'Call.startVideo: Video operation failure SourceUnavailableError') {
+        appendActiveErrorIfDefined(activeErrorMessages, latestErrors, 'Call.startVideo', 'callCameraAlreadyInUse');
+    }
+    else if (((_p = latestErrors['Call.startVideo']) === null || _p === void 0 ? void 0 : _p.message) === 'Call.startVideo: Video operation failure permissionDeniedError') {
+        appendActiveErrorIfDefined(activeErrorMessages, latestErrors, 'Call.startVideo', 'callCameraAccessDenied');
+    }
+    else {
+        appendActiveErrorIfDefined(activeErrorMessages, latestErrors, 'Call.startVideo', 'startVideoGeneric');
+    }
     appendActiveErrorIfDefined(activeErrorMessages, latestErrors, 'Call.unmute', 'unmuteGeneric');
-    if (((_o = latestErrors['CallAgent.join']) === null || _o === void 0 ? void 0 : _o.message) === 'CallAgent.join: Invalid meeting link') {
+    if (((_q = latestErrors['CallAgent.join']) === null || _q === void 0 ? void 0 : _q.message) === 'CallAgent.join: Invalid meeting link') {
         appendActiveErrorIfDefined(activeErrorMessages, latestErrors, 'CallAgent.join', 'failedToJoinCallInvalidMeetingLink');
     }
     else {