I guess the status is every one has been busy with transitioning to
github [1]
This should be re-sent as a github PR to be considered further.
However I think that one way or another *you* need to provide the test.
Saying a customer tested it or vaguely hoping the community will help
you out is just going
to delay this.
-phil.
[1]
https://mail.openjdk.java.net/pipermail/jdk-dev/2020-September/004691.html
On 9/21/20, 10:40 PM, Vipin Mv1 wrote:
Hi,
May I ask for a status on this please.
-----Vipin Mv1/India/IBM wrote: -----
To: Philip Race<philip.r...@oracle.com>
From: Vipin Mv1/India/IBM
Date: 08/31/2020 05:52PM
Cc: 2d-dev@openjdk.java.net
Subject: Re: [EXTERNAL] Re: [OpenJDK 2D-Dev] RFR: 8234393 [macos] printing
ignores printer tray
Hi Philip,
Thanks for the review comments. The testcase
test/jdk/java/awt/print/PrinterJob/TestMediaTraySelection.java seems to be
intended to test only the MediaTray functionality. Also, it doesn't seems to
have anything that is Linux specific.
Rather than having a new testcase wouldn't it be fine to have os.family ==
"mac" change alone to the existing one.?
Regarding the testing, the patch was already tested by our customer using a
multi tray printer and was found to be working. Owing to WFH factor due to
COVID19, multi tray printer is not accessible to us at this point of time.
May I request if the community can help us do further testing with the multi
tray printer.
Thanks& Regards
Vipin MV
-----Philip Race<philip.r...@oracle.com> wrote: -----
To: Vipin Mv1<vipin...@in.ibm.com>
From: Philip Race<philip.r...@oracle.com>
Date: 08/30/2020 10:02PM
Cc: 2d-dev@openjdk.java.net
Subject: [EXTERNAL] Re: [OpenJDK 2D-Dev] RFR: 8234393 [macos] printing ignores
printer tray
PS there is an existing manual regression test but it is currently
geared towards Linux and in fact is set to run only on Linux
test/jdk/java/awt/print/PrinterJob/TestMediaTraySelection.java
So you still may find it easier to create a new test rather than modify
this one to avoid breaking what it tests.
-phil
On 8/29/20, 11:10 AM, Philip Race wrote:
PS, there's a test case in the bug. Seems like it could be used as the
basis for a manual regression test.
Make sure you use @requires printer as well as adding the manual and
headful keywords
and next you'll have to check for an installed printer with multiple
trays
-phil
On 8/27/20, 11:36 AM, Philip Race wrote:
This looks reasonable but we need to test it first before approving it.
-phil.
On 8/27/20, 6:16 AM, Vipin Mv1 wrote:
Hi,
Please find below a patch for the following issue.
https://bugs.openjdk.java.net/browse/JDK-8234393
http://cr.openjdk.java.net/~aleonard/8234393/webrev.00
Thanks& Regards
Vipin MV