admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove NSApplicationActivateIgnoringOtherApps

Browse Source 
NSApplicationActivateIgnoringOtherApps is about to be deprecated.
The default behavior is good enough.

Tested on Chrome using https://wicg.github.io/conditional-focus/demo/

Bug: webrtc:15511
Change-Id: I1f59aea3d4e7c4942d17ee5c4f1b6c2d398016ee
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/321080
Commit-Queue: Johannes Kron <kron@webrtc.org>
Reviewed-by: Mark Foltz <mfoltz@chromium.org>
Auto-Submit: Johannes Kron <kron@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#40795}
This commit is contained in:
Johannes Kron 2023-09-21 23:13:50 +02:00 committed by WebRTC LUCI CQ
parent 850296b7a4
commit bbf27e0081
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
modules/desktop_capture/window_capturer_mac.mm
View File

@ -127,8 +127,7 @@ bool WindowCapturerMac::FocusOnSelectedSource() {
// TODO(jiayl): this will bring the process main window to the front. We // TODO(jiayl): this will bring the process main window to the front. We
// should find a way to bring only the window to the front. // should find a way to bring only the window to the front.
bool result = bool result =
[[NSRunningApplication runningApplicationWithProcessIdentifier: pid] [[NSRunningApplication runningApplicationWithProcessIdentifier:pid] activateWithOptions:0];
activateWithOptions: NSApplicationActivateIgnoringOtherApps];
CFRelease(window_id_array); CFRelease(window_id_array);
CFRelease(window_array); CFRelease(window_array);