commit d2534b3914f1b3c6a1aa3683e111fd76b8ff4893
author Andreas Pehrson <apehrson@mozilla.com> Tue Sep 03 09:20:38 2024
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Mar 24 18:19:38 2025
tree de7813f2bca92ffb7c60fb2d6fd0ce36557fc475
parent 48ccc95b0f76af263e64dac5a97a61f8c8aeeba9

In SCK backend move dpi to and set may_contain_cursor in frame callback

Bug: webrtc:367915807
Change-Id: Id29b979ab7909d21eb92bc926f9decdae3345381
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/365084
Reviewed-by: Johannes Kron <kron@webrtc.org>
Reviewed-by: Alexander Cooper <alcooper@chromium.org>
Commit-Queue: Andreas Pehrson <apehrson@mozilla.com>
Cr-Commit-Position: refs/heads/main@{#44204}

modules/desktop_capture/mac/screen_capturer_sck.mm

diff --git a/modules/desktop_capture/mac/screen_capturer_sck.mm b/modules/desktop_capture/mac/screen_capturer_sck.mm
index c8f5411..7825d85 100644
--- a/modules/desktop_capture/mac/screen_capturer_sck.mm
+++ b/modules/desktop_capture/mac/screen_capturer_sck.mm
@@ -171,7 +171,6 @@
     MutexLock lock(&latest_frame_lock_);
     if (latest_frame_) {
       frame = latest_frame_->Share();
-      frame->set_dpi(DesktopVector(latest_frame_dpi_, latest_frame_dpi_));
       if (frame_is_dirty_) {
         frame->mutable_updated_region()->AddRect(
             DesktopRect::MakeSize(frame->size()));
@@ -357,6 +356,9 @@
 
   if (dirty) {
     MutexLock lock(&latest_frame_lock_);
+    frame->set_dpi(DesktopVector(latest_frame_dpi_, latest_frame_dpi_));
+    frame->set_may_contain_cursor(capture_options_.prefer_cursor_embedded());
+
     frame_is_dirty_ = true;
     std::swap(latest_frame_, frame);
   }