Re: RFR: 8342911: Remove calls to doPrivileged in controls [v3]

2024-11-04 Thread Ajit Ghaisas
On Fri, 1 Nov 2024 17:45:55 GMT, Andy Goryachev wrote: >> Removes doPrivileged in the following module: >> >> - javafx.controls >> >> See JDK-8342441 for details. >> >> As a helpful hint for reviewers, I recommend reviewing this using the "Hide >> whitespace" option. > > Andy Goryachev has up

Re: RFR: 8342911: Remove calls to doPrivileged in controls [v3]

2024-11-01 Thread Andy Goryachev
> Removes doPrivileged in the following module: > > - javafx.controls > > See JDK-8342441 for details. > > As a helpful hint for reviewers, I recommend reviewing this using the "Hide > whitespace" option. Andy Goryachev has updated the pull request with a new target base due to a merge or a r

Re: RFR: 8342911: Remove calls to doPrivileged in controls [v2]

2024-10-30 Thread Andy Goryachev
> Removes doPrivileged in the following module: > > - javafx.controls > > See JDK-8342441 for details. > > As a helpful hint for reviewers, I recommend reviewing this using the "Hide > whitespace" option. Andy Goryachev has updated the pull request incrementally with one additional commit sin

Re: RFR: 8342911: Remove calls to doPrivileged in controls [v2]

2024-10-30 Thread Kevin Rushforth
On Wed, 30 Oct 2024 14:36:44 GMT, Andy Goryachev wrote: >> Removes doPrivileged in the following module: >> >> - javafx.controls >> >> See JDK-8342441 for details. >> >> As a helpful hint for reviewers, I recommend reviewing this using the "Hide >> whitespace" option. > > Andy Goryachev has u

Re: RFR: 8342911: Remove calls to doPrivileged in controls

2024-10-29 Thread Kevin Rushforth
On Mon, 28 Oct 2024 15:01:47 GMT, Andy Goryachev wrote: > Removes doPrivileged in the following module: > > - javafx.controls > > See JDK-8342441 for details. > > As a helpful hint for reviewers, I recommend reviewing this using the "Hide > whitespace" option. modules/javafx.controls/src/mai

Re: RFR: 8342911: Remove calls to doPrivileged in controls

2024-10-29 Thread Kevin Rushforth
On Mon, 28 Oct 2024 15:01:47 GMT, Andy Goryachev wrote: > Removes doPrivileged in the following module: > > - javafx.controls > > See JDK-8342441 for details. > > As a helpful hint for reviewers, I recommend reviewing this using the "Hide > whitespace" option. A second pair of eyes would be

Re: RFR: 8342911: Remove calls to doPrivileged in controls

2024-10-29 Thread Kevin Rushforth
On Mon, 28 Oct 2024 15:01:47 GMT, Andy Goryachev wrote: > Removes doPrivileged in the following module: > > - javafx.controls > > See JDK-8342441 for details. > > As a helpful hint for reviewers, I recommend reviewing this using the "Hide > whitespace" option. This looks good to me. I left a

RFR: 8342911: Remove calls to doPrivileged in controls

2024-10-28 Thread Andy Goryachev
Removes doPrivileged in the following module: - javafx.controls See JDK-8342441 for details. As a helpful hint for reviewers, I recommend reviewing this using the "Hide whitespace" option. - Commit messages: - Merge remote-tracking branch 'origin/master' into 8342911.do.privilig