On Wed, 28 May 2025 10:29:17 GMT, GennadiyKrivoshein <[email protected]> wrote:
> The fix for the https://bugs.openjdk.org/browse/JDK-8251928. > > **Description**. > This PR contains changes to be able to print with DPI higher than 72 on > macOS, set default CPrinterJob DPI is 300 like in the PSPrinterJob. > > As described in the macOS drawing guide, the following steps are required to > draw with high DPI > (https://developer.apple.com/library/archive/documentation/Cocoa/Conceptual/CocoaDrawingGuide/Transforms/Transforms.html#//apple_ref/doc/uid/TP40003290-CH204-BCICIJAJ): > 1. Convert the user-space point, size, or rectangle value to device space > coordinates; > 2. Normalize the value in device space so that it is aligned to the > appropriate pixel boundary; > 3. Convert the normalized value back to user space; > 4. Draw your content using the adjusted value. > > The 1-st step is now implemented in the CPrinterJob, a Graphics provided to > the print method adjusted to a printer's DPI. > The 2-nd step is a drawing process in the java code (without changes). > The 3-rd step is now implemented in the PrinterView.m, the drawing scaled > back to the 72 DPI. > The 4-th step is a drawing process in the native code (without changes). > > **Tests**. > I run all tests from javax.print package and there is no any regression. > New test covers macOS and Linux only because we know its default DPI - 300. This pull request has now been integrated. Changeset: 76a1109d Author: GennadiyKrivoshein <[email protected]> Committer: Prasanta Sadhukhan <[email protected]> URL: https://git.openjdk.org/jdk/commit/76a1109d6fc7baac9ebc7accff800ef8927931bb Stats: 379 lines in 5 files changed: 367 ins; 1 del; 11 mod 8251928: [macos] the printer DPI always be 72, cause some content lost when print out Reviewed-by: psadhukhan, prr ------------- PR: https://git.openjdk.org/jdk/pull/25489
