[webkit-changes] [WebKit/WebKit] 29cddc: If torch is on on a track, muting and unmuting it ...

youennf noreply at github.com
Fri May 17 23:42:33 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 29cddce2522b69778bd95c60e2d9eb60bbd17714
      https://github.com/WebKit/WebKit/commit/29cddce2522b69778bd95c60e2d9eb60bbd17714
  Author: Youenn Fablet <youenn at apple.com>
  Date:   2024-05-17 (Fri, 17 May 2024)

  Changed paths:
    M Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm

  Log Message:
  -----------
  If torch is on on a track, muting and unmuting it will switch off the torch
https://bugs.webkit.org/show_bug.cgi?id=270840
rdar://124435440

Reviewed by Eric Carlson.

When restarting to capture, we were not setting back torch to what it should be.
This would trigger the issue of having torch on, muting capture, unmuting capture and having torch off.
We fix this by ensuring that torch is updated when (re)starting capture.

We do a drive-by fix in AVVideoCaptureSource::reconfigureIfNeeded.
We need to call startApplyingConstraints/endApplyingConstraints instead of beginConfiguration/commitConfiguration, to keep the balance.
Otherwise updateTorch would do an additional beginConfiguration call without any corresponding commitConfiguration.

* Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm:
(WebCore::AVVideoCaptureSource::setupCaptureSession):
(WebCore::AVVideoCaptureSource::reconfigureIfNeeded):

Canonical link: https://commits.webkit.org/278948@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list