On Wed, 10 May 2023 13:04:42 GMT, Kevin Rushforth <[email protected]> wrote:
>> Updated according to your comments. >> I still left `revokeScreenCapturePermission` here for now, so that its >> documentation can be easily compared and aligned with the other >> documentation changes. >> After finalization it will be moved to #13803 (if we decide to keep the >> method) > > I agree with Phil (and Sergey, who mentioned the same earlier). I'm not sure > I see enough motivation for this method to justify adding it. It seems more > like a workaround for an implementation quirk of Wayland to me. Like I wrote elsewhere, I am not sure how the app would know when to call this. And it is easy enough to use the preferences API to clear this out whilst we think more on this. BTW "reset" or "clear" might be better than "revoke" .. for all we know the current token denies access anyway and what we really mean is more like those other suggestions. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13809#discussion_r1190318644
