On Tue, 28 Jan 2025 17:27:17 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> modules/javafx.graphics/src/main/native-glass/mac/GlassApplication.m line
>> 286:
>>
>>> 284: LOG("-> need to active application");
>>> 285: dispatch_async(dispatch_get_main_queue(), ^{
>>> 286: [NSApp activateIgnoringOtherApps:YES];
>>
>> I see that the CanvasTest mentioned in the JBS issue passes now with this
>> fix, and fails with `[NSApp activate];`
>>
>> I'm not sure why `[NSApp activateIgnoringOtherApps:YES];` works but `[NSApp
>> activate];` doesn't, given that the former is deprecated, as you already
>> mentioned, and the latter is what Apple precisely
>> [recommends](https://developer.apple.com/documentation/appkit/nsapplication/activate(ignoringotherapps:)?language=objc)
>> to use instead.
>> In any case, probably something for
>> https://bugs.openjdk.org/browse/JDK-8348878.
>
> @jperedadnr can you add this comment to
> https://bugs.openjdk.org/browse/JDK-8348878 please?
Done
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1685#discussion_r1932595626
