RFR: 8345142: Remove uses of SecurityManager in Printing related classes

[Alexander Zvegintsev]

On Thu, 28 Nov 2024 00:30:27 GMT, Phil Race <prr at openjdk.org> wrote:

> Remove uses of and references to SecurityManager from the printing code

Marked as reviewed by azvegint (Reviewer).

src/java.desktop/share/classes/sun/print/RasterPrinterJob.java line 1658:

> 1656:                                        dest);
> 1657:         } catch (SecurityException se) {
> 1658:             //There is already file read/write access so at this point

Few other comments seems obsolete now

line 948:


/* A security check has already been performed in the
 * java.awt.print.printerJob.getPrinterJob method.
 * So by the time we get here, it is OK for the current thread
 * to print either to a file (from a Dialog we control!) or
 * to a chosen printer.
 *
 * We raise privilege when we put up the dialog, to avoid
 * the "warning applet window" banner.
 */

(Same for the `PSPrinterJob.java:508`)


line 1809:

     * The caller must have security permission to read system properties.

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

PR Review: https://git.openjdk.org/jdk/pull/22422#pullrequestreview-2468198406
PR Review Comment: https://git.openjdk.org/jdk/pull/22422#discussion_r1862208582