fix(ScreenshotStream): do nothing when start() is called after record…

…ing has started

packages/core/src/provenance/screenshot-stream.ts

@@ -51,14 +51,28 @@ export function intitializeScreenshotStream(): ScreenshotStream {
  }
  }
 
+ /**
+ * False if the MediaStream has not been initialized via start(), or has been stopped.
+ * True if we have a MediaStream and can capture screenshots.
+ * @returns Whether a screenshot can be captured.
+ */
+ function canCapture(): boolean {
+ return video !== null;
+ }
+
  /**
  * Starts the media stream needed to capture screenshots on-demand.
  * Will prompt the user for permission to capture the screen.
  * Immediately captures a first screenshot.
+ * Does nothing if the stream is already started.
  * @throws Error if unable to start the recording; usually due to the user denying permission.
  * @param callback Optional callback to run after the stream is started.
  */
  async function start(callback?: () => void): Promise<void> {
+ if (canCapture()) {
+ return;
+ }
+
  video = document.createElement('video');
  video.autoplay = true;
  video.muted = true;
@@ -110,15 +124,6 @@ export function intitializeScreenshotStream(): ScreenshotStream {
  }
  }
 
- /**
- * False if the MediaStream has not been initialized via start(), or has been stopped.
- * True if we have a MediaStream and can capture screenshots.
- * @returns Whether a screenshot can be captured.
- */
- function canCapture(): boolean {
- return video !== null;
- }
-
  /**
  * Captures a screenshot and stores it in the screenshots array.
  * Also pushes the screenshot to the newScreenshotCallback if available.