Re: RFR: 8263363: Minor cleanup of Lanai code - unused code removal and comments correction [v5]

2021-05-01 Thread Sergey Bylokhov
On Tue, 27 Apr 2021 14:59:57 GMT, Ajit Ghaisas wrote: >> Refer JBS for 3 issues that this PR addresses. >> In addition, I have corrected an erroneous free() call in the same method I >> was cleaning up. > > Ajit Ghaisas has updated the pull request incrementally with one additional > commit

Re: RFR: 8264786: [macos] All Swing/AWT apps cause Allow Notifications prompt to appear when app is launched

2021-05-01 Thread Sergey Bylokhov
On Sat, 1 May 2021 11:26:39 GMT, Prasanta Sadhukhan wrote: > Please confirm if it is working in 10.14 and in multiscreen environment. > Also, I guess a testcase can be written to see if notification center is > coming up for normal awt non-trayicon app(obscuring normal awt window or not) I

Re: RFR: 8264786: [macos] All Swing/AWT apps cause Allow Notifications prompt to appear when app is launched

2021-05-01 Thread Alexander Zuev
On Tue, 27 Apr 2021 01:09:44 GMT, Sergey Bylokhov wrote: > The fix for the JDK-8187639 reimplemented the tray icon messages using the > standard notification center, it worked fine on macOS 10.9. But on the new > macOS, the usage of that API requests permission from the user. Since the > code

RFR: 8264299: Create implementation of native accessibility peer for ScrollPane and ScrollBar Java Accessibility roles

2021-05-01 Thread Pankaj Bansal
Create implementation of native accessibility peer for ScrollPane and ScrollBar Java Accessibility roles. There are no direct protocols for these two roles and there is no protocol in list of role based protocols defined by Apple Accessibility API, which can be used for these roles directly.

Re: RFR: 8264786: [macos] All Swing/AWT apps cause Allow Notifications prompt to appear when app is launched

2021-05-01 Thread Prasanta Sadhukhan
On Tue, 27 Apr 2021 01:09:44 GMT, Sergey Bylokhov wrote: > The fix for the JDK-8187639 reimplemented the tray icon messages using the > standard notification center, it worked fine on macOS 10.9. But on the new > macOS, the usage of that API requests permission from the user. Since the > code