[webkit-changes] [WebKit/WebKit] bc1ade: rdar://141787428 (Screen sharing pause button does...

youennf noreply at github.com
Thu Jan 30 10:34:10 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: bc1adebed36b72d32614972ce0d59ade6c68ccc3
      https://github.com/WebKit/WebKit/commit/bc1adebed36b72d32614972ce0d59ade6c68ccc3
  Author: Youenn Fablet <youenn at apple.com>
  Date:   2025-01-30 (Thu, 30 Jan 2025)

  Changed paths:
    M Source/WebKit/UIProcess/API/C/WKPage.cpp
    M Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm
    M Tools/TestWebKitAPI/Tests/WebKitCocoa/GetDisplayMediaWindowAndScreen.mm

  Log Message:
  -----------
  rdar://141787428 (Screen sharing pause button does not pause)

Reviewed by Eric Carlson.

After https://github.com/WebKit/WebKit/commit/a9b4b55, WebCore is only muting window capture using MediaProducerMutedState::WindowCaptureIsMuted.
But WKPage SPI was only setting MediaProducerMutedState::ScreenCaptureIsMuted when trying to mute screen share.
We would then end up muting only screen tracks and not window tracks.
We update WKPage SPI and alos WKWebView SPI to also handle MediaProducerMutedState::WindowCaptureIsMuted and MediaProducerMutedState::SystemAudioCaptureIsMuted.

Manually tested and covered by updated test.

* Source/WebKit/UIProcess/API/C/WKPage.cpp:
(WKPageSetMuted):
* Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _setPageMuted:]):
* Tools/TestWebKitAPI/Tests/WebKitCocoa/GetDisplayMediaWindowAndScreen.mm:
(TestWebKitAPI::TEST(WebKit2, GetDisplayMediaWindowAndScreenPrompt)):

Originally-landed-as: 8da7b03ceb9e. rdar://143592356
Canonical link: https://commits.webkit.org/289565@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