8378417: Printing All pages results in NPE for 1.1 PrintJob

Reviewed-by: prr, aivanov
This commit is contained in:
Prasanta Sadhukhan 2026-03-09 01:21:34 +00:00
parent b159addb80
commit 8c4c8a1617
4 changed files with 70 additions and 5 deletions

View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2000, 2025, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2000, 2026, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@ -526,8 +526,10 @@ public class PrintJobDelegate implements Printable, Runnable {
}
PageRanges range = (PageRanges)attributes.get(PageRanges.class);
int[][] members = range.getMembers();
jobAttributes.setPageRanges(members);
if (range != null) {
int[][] members = range.getMembers();
jobAttributes.setPageRanges(members);
}
SheetCollate collation =
(SheetCollate)attributes.get(SheetCollate.class);

View File

@ -1734,7 +1734,9 @@ public final class WPrinterJob extends RasterPrinterJob
attributes.add(new PageRanges(from, to));
setPageRange(from, to);
} else {
attributes.remove(PageRanges.class);
// Sets default values for PageRange attribute and setPageRange
attributes.add(new PageRanges(1,
Integer.MAX_VALUE));
setPageRange(Pageable.UNKNOWN_NUMBER_OF_PAGES,
Pageable.UNKNOWN_NUMBER_OF_PAGES);
}

View File

@ -0,0 +1,61 @@
/*
* Copyright (c) 2026, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/
import java.awt.Frame;
import java.awt.JobAttributes;
import java.awt.PrintJob;
import java.awt.Robot;
import java.awt.Toolkit;
import java.awt.event.KeyEvent;
/*
* @test
* @bug 8378417
* @key headful printer
* @summary Verifies No Exception is thrown when Printing "All" pages
* @run main TestPrintNoException
*/
public class TestPrintNoException {
public static void main(String[] args) throws Exception {
Robot robot = new Robot();
Thread t = new Thread (() -> {
robot.delay(5000);
robot.keyPress(KeyEvent.VK_ENTER);
robot.keyRelease(KeyEvent.VK_ENTER);
robot.waitForIdle();
});
Frame testFrame = new Frame("print");
try {
t.start();
PrintJob pj = Toolkit.getDefaultToolkit().getPrintJob(testFrame, null, null);
if (pj != null) {
pj.end();
}
} finally {
testFrame.dispose();
}
}
}

View File

@ -23,7 +23,7 @@
/*
* @test
* @bug 6575331 8297191 8373239
* @bug 6575331 8297191 8373239 8378417
* @key printer
* @summary The specified pages should be printed.
* @library /java/awt/regtesthelpers