Update ghcr.io/element-hq/element-web:develop Docker digest to 0c39f7b (#4439)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

docker-compose-playwright.yml

@@ -19,7 +19,7 @@ services:
       - ./backend/playwright_homeserver-othersite.yaml:/data/cfg/homeserver.yaml:Z
   element-web:
     # Pin to a SHA so that upstream cannot break our tests. Renovate handles regular updates.
-    image: ghcr.io/element-hq/element-web:develop@sha256:f30be05ae9893a5f8acab96207056d1144ce436b8550b95fb60cdfa838f50699
+    image: ghcr.io/element-hq/element-web:develop@sha256:9e2335e93a6683fff303ca6d14d0978ce21fe78ab9f6eaa4495c614006b8c3a7
   element-web-1:
     # Pin to a SHA so that upstream cannot break our tests. Renovate handles regular updates.
-    image: ghcr.io/element-hq/element-web:develop@sha256:f30be05ae9893a5f8acab96207056d1144ce436b8550b95fb60cdfa838f50699
+    image: ghcr.io/element-hq/element-web:develop@sha256:9e2335e93a6683fff303ca6d14d0978ce21fe78ab9f6eaa4495c614006b8c3a7

sdk/src/LocalMedia.test.ts

@@ -162,7 +162,6 @@ describe("createLocalMedia", () => {
     media.release("camera");
     expect(trackBehind(camera).stop).toHaveBeenCalled();
     expect(media.tracks$.value).toEqual([microphone]);
-    expect(media.requests.map((r) => r.source)).toEqual(["microphone"]);
 
     media.stop();
     expect(trackBehind(microphone).stop).toHaveBeenCalled();

sdk/src/LocalMedia.ts

@@ -107,7 +107,6 @@ export function createLocalMedia(
     if (source !== "microphone" && source !== "camera") return;
     capturedTracks.get(source)?.stop();
     capturedTracks.delete(source);
-    requests.delete(source);
     publishCapturedTracks();
   };
 

src/tracks/BackgroundEffectTransformer.test.ts

@@ -20,7 +20,6 @@ import {
 
 const frame = {} as VideoFrame;
 const controller = {} as TransformStreamDefaultController<VideoFrame>;
-const stats = { processingTimeMs: 1, segmentationTimeMs: 1, filterTimeMs: 0 };
 
 describe("BackgroundEffectTransformer", () => {
   afterEach(() => {
@@ -49,10 +48,9 @@ describe("BackgroundEffectTransformer", () => {
   });
 
   it("reports the first frame carrying an effect, once", async () => {
-    // The library reports every frame it draws.
     vi.spyOn(BackgroundTransformer.prototype, "transform").mockImplementation(
       async function (this: BackgroundTransformer) {
-        this.options.onFrameProcessed?.(stats);
+        this.isFirstFrame = false;
         return Promise.resolve();
       },
     );
@@ -75,37 +73,6 @@ describe("BackgroundEffectTransformer", () => {
     expect(transformer.onFirstFrame).not.toHaveBeenCalled();
   });
 
-  it("keeps reporting drawn frames to whoever asked", async () => {
-    vi.spyOn(BackgroundTransformer.prototype, "transform").mockImplementation(
-      async function (this: BackgroundTransformer) {
-        this.options.onFrameProcessed?.(stats);
-        return Promise.resolve();
-      },
-    );
-    const onFrameProcessed = vi.fn();
-    const transformer = new BackgroundEffectTransformer({ onFrameProcessed });
-
-    await transformer.transform(frame, controller);
-    expect(onFrameProcessed).toHaveBeenCalledWith(stats);
-  });
-
-  it("never sends the first frame on untouched", async () => {
-    // The library copies its first frame through untouched, ahead of the
-    // drawn one, unless told the first frame is past
-    const seen: boolean[] = [];
-    vi.spyOn(BackgroundTransformer.prototype, "transform").mockImplementation(
-      async function (this: BackgroundTransformer) {
-        seen.push(this.isFirstFrame);
-        return Promise.resolve();
-      },
-    );
-    const transformer = new BackgroundEffectTransformer({});
-    expect(transformer.isFirstFrame).toBe(true);
-
-    await transformer.transform(frame, controller);
-    expect(seen).toEqual([false]);
-  });
-
   it("passes frames on untouched once disabled, even after blurring", async () => {
     const transformer = new BackgroundEffectTransformer({});
     await transformer.update({ blurRadius: 15, backgroundDisabled: false });

src/tracks/BackgroundEffectTransformer.ts

@@ -56,30 +56,16 @@ const wasmFileset: WasmFileset = {
 export class BackgroundEffectTransformer extends BackgroundTransformer {
   /** Called once, when the first frame carrying an effect has been drawn. */
   public onFirstFrame: (() => void) | undefined;
-  private drewAFrame = false;
-
-  public constructor(opts: BackgroundOptions) {
-    super(opts);
-    // Called by the library for a frame it drew, never for one it passed on
-    // untouched.
-    this.options.onFrameProcessed = (stats): void => {
-      opts.onFrameProcessed?.(stats);
-      if (this.drewAFrame) return;
-      this.drewAFrame = true;
-      this.onFirstFrame?.();
-    };
-  }
 
   public override async transform(
     frame: VideoFrame,
     controller: TransformStreamDefaultController<VideoFrame>,
   ): Promise<void> {
-    // For the first frame, the library enqueues the raw frame ahead of the
-    // processed one, so that the output shows something while the first
-    // segmentation runs. The raw frame is the room without the effect,
-    // so no frame is allowed to count as the first.
-    this.isFirstFrame = false;
+    const priming = this.isFirstFrame;
     await super.transform(frame, controller);
+    // Cleared only once a frame has been through the segmenter; a frame
+    // passed through untouched leaves it set.
+    if (priming && !this.isFirstFrame) this.onFirstFrame?.();
   }
 
   /**