RFR: JDK-8344057 : Remove doPrivileged calls from unix platform sources in the java.desktop module

[Harshitha Onkar]

On Tue, 19 Nov 2024 17:38:19 GMT, Phil Race <prr at openjdk.org> wrote:

>> Well, that too. But the main point is WHERE is the code now that calls PrinterOpener.run() ?
>> The thing that does the actual work. So far as I can see you've deleted the call and done nothing to replace it.
>
> So 2 things to do that I can see.
> 1) after line 526 call po.run();
> 2) Go to the definition of PrinterOpener and remove implements java.security.PrivilegedAction<Object>
> 
> Anything more would be too much refactoring.
> 
> Same idea below for PrinterSpooler.
> 
> How were you testing this ?

> after line 526 call po.run();
> Go to the definition of PrinterOpener and remove implements java.security.PrivilegedAction
> Anything more would be too much refactoring.

Makes sense. Thank you for clarifying.

> How were you testing this ?

Ran CI tests - clientlibs, wayland jobs for these unix file changes as of now. I believe to test this properly we need to run manual printer tests.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22218#discussion_r1848815158