RFR: 8335231: [macos] Test java/awt/print/PrinterJob/Cancel/PrinterJobCancel.java failed on macOS because the case didn't get the expected PrintAbortException [v2]

[Phil Race]

On Sun, 28 Jul 2024 22:22:37 GMT, Phil Race <prr at openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Call isCancelled and throw original exception
>
> src/java.desktop/macosx/classes/sun/lwawt/macosx/CPrinterJob.java line 375:
> 
>> 373:                         printLoop(true, firstPage, lastPage);
>> 374:                     } catch (PrinterAbortException pex) {
>> 375:                         throw new PrinterAbortException(pex.getMessage());
> 
> Why not re-throw the original exception ?

Did you miss this question ?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20027#discussion_r1698942876