Fix bug that caused screenview to stay off when screenshare was toggled

aws · Nov 26, 2019 · fedfe4d · fedfe4d
1 parent d24c5d0
commit fedfe4d
Show file tree

Hide file tree

Showing 5 changed files with 39 additions and 3 deletions.
diff --git a/src/screenviewing/context/DefaultScreenViewingComponentContext.ts b/src/screenviewing/context/DefaultScreenViewingComponentContext.ts
@@ -85,9 +85,11 @@ export default class DefaultScreenViewingComponentContext implements ScreenViewi
       providers
     );
     signalingSession.registerObserver({
-      streamDidStart(_screenMessageDetail: ScreenMessageDetail): void {},
+      streamDidStart(_screenMessageDetail: ScreenMessageDetail): void {
+        deltaRenderer.revealViewport();
+      },
       streamDidStop(_screenMessageDetail: ScreenMessageDetail): void {
-        deltaRenderer.close();
+        deltaRenderer.hideViewport();
       },
     });
 

diff --git a/src/screenviewing/deltarenderer/DefaultScreenViewingDeltaRenderer.ts b/src/screenviewing/deltarenderer/DefaultScreenViewingDeltaRenderer.ts
@@ -188,6 +188,18 @@ export default class DefaultScreenViewingDeltaRenderer implements ScreenViewingD
     }
   }
 
+  hideViewport(): void {
+    if (this.viewport) {
+      this.viewport.style.display = 'none';
+    }
+  }
+
+  revealViewport(): void {
+    if (this.viewport) {
+      this.viewport.style.display = 'block';
+    }
+  }
+
   changePresentationPolicy(policy: PresentationPolicy): void {
     this.policy = policy;
     this.updatePresentation();

diff --git a/src/screenviewing/deltarenderer/ScreenViewingDeltaRenderer.ts b/src/screenviewing/deltarenderer/ScreenViewingDeltaRenderer.ts
@@ -27,6 +27,10 @@ export default interface ScreenViewingDeltaRenderer {
 
   setViewport(viewport: HTMLElement): void;
 
+  hideViewport(): void;
+
+  revealViewport(): void;
+
   changePresentationPolicy(policy: PresentationPolicy): void;
 
   close(): void;

diff --git a/test/screenviewing/context/DefaultScreenViewingComponentContext.test.ts b/test/screenviewing/context/DefaultScreenViewingComponentContext.test.ts
@@ -66,7 +66,8 @@ describe('DefaultScreenViewingComponentContext', () => {
         {
           deltaRendererProvider: () => ({
             ...Substitute.for(),
-            close(): void {
+            hideViewport(): void {},
+            revealViewport(): void {
               done();
             },
           }),

diff --git a/test/screenviewing/deltarenderer/DefaultScreenViewingDeltaRenderer.test.ts b/test/screenviewing/deltarenderer/DefaultScreenViewingDeltaRenderer.test.ts
@@ -434,4 +434,21 @@ describe('DefaultScreenViewingDeltaRenderer', () => {
       deltaRenderer.zoomReset();
     });
   });
+
+  describe('hideViewport and revealViewport', () => {
+    it('hides and reveals the viewport', () => {
+      const deltaRenderer: ScreenViewingDeltaRenderer = new DefaultScreenViewingDeltaRenderer(
+        controller,
+        logger,
+        Substitute.for(),
+        Substitute.for(),
+        Substitute.for()
+      );
+      deltaRenderer.hideViewport();
+      deltaRenderer.revealViewport();
+      deltaRenderer.setViewport(Substitute.for());
+      deltaRenderer.hideViewport();
+      deltaRenderer.revealViewport();
+    });
+  });
 });