feat: Remove getCanvasManager, export CanvasManager class directly

This simplifies the code a bit by exporting the CanvasManager directly. With [ReplayCanvas](getsentry/sentry-javascript#10112), we can rely on it for complex setup, but keeps it simple for our users.

Author: billyvg

packages/rrweb/src/index.ts

@@ -31,5 +31,7 @@ export {
   mirror,
   freezePage,
   addCustomEvent,
-  getCanvasManager,
 } from './record';
+
+export {CanvasManager} from './record/observers/canvas/canvas-manager';
+export type {CanvasManagerConstructorOptions} from './record/observers/canvas/canvas-manager';

packages/rrweb/src/record/index.ts

@@ -3,7 +3,6 @@ import {
   MaskInputOptions,
   SlimDOMOptions,
   createMirror,
-  DataURLOptions,
 } from '@sentry-internal/rrweb-snapshot';
 import { initObservers, mutationBuffers } from './observer';
 import {
@@ -41,7 +40,6 @@ import {
   ShadowDomManagerNoop,
 } from './shadow-dom-manager';
 import {
-  CanvasManager,
   CanvasManagerConstructorOptions,
   CanvasManagerInterface,
   CanvasManagerNoop,
@@ -340,6 +338,18 @@ function record<T = eventWithTime>(
   const canvasManager: CanvasManagerInterface = _getCanvasManager(
     getCanvasManager,
     {
+      mirror,
+      win: window,
+      mutationCb: (p: canvasMutationParam) =>
+        wrappedEmit(
+          wrapEvent({
+            type: EventType.IncrementalSnapshot,
+            data: {
+              source: IncrementalSource.CanvasMutation,
+              ...p,
+            },
+          }),
+        ),
       recordCanvas,
       blockClass,
       blockSelector,
@@ -717,14 +727,6 @@ export function takeFullSnapshot(isCheckout?: boolean) {
   _takeFullSnapshot(isCheckout);
 }
 
-function wrappedEmit(e: eventWithTime) {
-  if (!_wrappedEmit) {
-    return;
-  }
-
-  _wrappedEmit(e);
-}
-
 // record.addCustomEvent is removed because Sentry Session Replay does not use it
 // record.freezePage is removed because Sentry Session Replay does not use it
 
@@ -734,48 +736,18 @@ record.takeFullSnapshot = takeFullSnapshot;
 
 export default record;
 
-type PrivateOptions = 'mutationCb' | 'win' | 'mirror';
-type PublicGetCanvasManagerOptions = Omit<
-  CanvasManagerConstructorOptions,
-  PrivateOptions
->;
-
-interface PrivateGetCanvasManagerOptions
-  extends PublicGetCanvasManagerOptions,
-    Pick<CanvasManagerConstructorOptions, PrivateOptions> {}
-
 function _getCanvasManager(
   getCanvasManagerFn:
     | undefined
-    | ((options: PrivateGetCanvasManagerOptions) => CanvasManagerInterface),
-  options: PublicGetCanvasManagerOptions,
+    | ((options: Partial<CanvasManagerConstructorOptions>) => CanvasManagerInterface),
+  options: CanvasManagerConstructorOptions,
 ) {
   try {
     return getCanvasManagerFn
-      ? getCanvasManagerFn({
-          ...options,
-          mirror,
-          win: window,
-          mutationCb: (p: canvasMutationParam) =>
-            wrappedEmit(
-              wrapEvent({
-                type: EventType.IncrementalSnapshot,
-                data: {
-                  source: IncrementalSource.CanvasMutation,
-                  ...p,
-                },
-              }),
-            ),
-        })
+      ? getCanvasManagerFn(options)
       : new CanvasManagerNoop();
   } catch {
     console.warn('Unable to initialize CanvasManager');
     return new CanvasManagerNoop();
   }
 }
-
-export function getCanvasManager(
-  options: PublicGetCanvasManagerOptions,
-): CanvasManagerInterface {
-  return new CanvasManager(options as CanvasManagerConstructorOptions);
-}