[webkit-changes] [WebKit/WebKit] 5a41af: Stop failing muted audio capture when AirPods are ...

youennf noreply at github.com
Thu Oct 26 05:38:21 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 5a41af799e85572c94a2b25581e967e6d871e699
      https://github.com/WebKit/WebKit/commit/5a41af799e85572c94a2b25581e967e6d871e699
  Author: Youenn Fablet <youennf at gmail.com>
  Date:   2023-10-26 (Thu, 26 Oct 2023)

  Changed paths:
    M LayoutTests/TestExpectations
    M LayoutTests/fast/mediastream/microphone-change-while-muted-expected.txt
    M LayoutTests/fast/mediastream/microphone-change-while-muted.html
    M LayoutTests/platform/mac/TestExpectations
    M Source/WebCore/platform/mediastream/mac/BaseAudioSharedUnit.cpp
    M Source/WebCore/platform/mediastream/mac/MockAudioSharedUnit.mm

  Log Message:
  -----------
  Stop failing muted audio capture when AirPods are taken out of ears
https://bugs.webkit.org/show_bug.cgi?id=263433
rdar://117252822

Reviewed by Eric Carlson.

When taking AirPods out of ears, the AirPod devices are disappearing and we were failing capture right away.
Given we are muting capture in that case, it is best to wait to fail capture when unmuted instead.

This allows to put AirPods in and out without having the website dealing with restarting capture.
We modify LayoutTests/fast/mediastream/microphone-change-while-muted.html accordingly and mock capture to validate device is available when starting capturing, like done by CoreAudio.

The test is only enabled in MacOS since other ports do not have a way to handle mock capture failure in case of missing device.

* LayoutTests/TestExpectations:
* LayoutTests/fast/mediastream/microphone-change-while-muted-expected.txt:
* LayoutTests/fast/mediastream/microphone-change-while-muted.html:
* LayoutTests/platform/mac/TestExpectations:
* Source/WebCore/platform/mediastream/mac/BaseAudioSharedUnit.cpp:
(WebCore::BaseAudioSharedUnit::devicesChanged):
* Source/WebCore/platform/mediastream/mac/MockAudioSharedUnit.mm:
(WebCore::MockAudioSharedInternalUnit::set):

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




More information about the webkit-changes mailing list