[OpenJDK 2D-Dev] [9] RFR 4885375: Page Ranges 'To Page' field must be populated based on Pageable

Jayathirth D V jayathirth.d.v at oracle.com
Thu Sep 1 07:37:23 UTC 2016

Hi Prasanta,


Changes are working fine.





From: Philip Race 
Sent: Wednesday, August 24, 2016 11:12 PM
To: Prasanta Sadhukhan
Cc: 2d-dev
Subject: Re: [OpenJDK 2D-Dev] [9] RFR 4885375: Page Ranges 'To Page' field must be populated based on Pageable


OK that is what I (sort of) suspected.



On 8/24/16, 10:08 AM, Prasanta Sadhukhan wrote: 

Modified testcase
HYPERLINK "http://cr.openjdk.java.net/%7Epsadhukhan/4885375/webrev.01/"http://cr.openjdk.java.net/~psadhukhan/4885375/webrev.01/

On 8/24/2016 10:27 PM, Prasanta Sadhukhan wrote:

Oh, it should be return 5. I was testing the 2nd issue and forgot to change it back.


On 8/24/2016 10:26 PM, Philip Race wrote:

Why does the test look like this ?

 157     @Override
 158     public int getNumberOfPages() {
 159         return 0;
 160     }

On 8/23/16, 5:05 AM, Prasanta Sadhukhan wrote: 

Hi All, 

Please review a fix whereby it is seen that Page Ranges fields in the "cross-platform" dialog does not get populated based on what pageable's getNumberOfPages() returns. "To" field is always 1. 
However, for native dialog it shows the "To" field correctly based on what getNumberOfPages() returns. 

Bug: https://bugs.openjdk.java.net/browse/JDK-4885375 
webrev: HYPERLINK "http://cr.openjdk.java.net/%7Epsadhukhan/4885375/webrev.00/"http://cr.openjdk.java.net/~psadhukhan/4885375/webrev.00/ 

This is because in cross platform ServiceDialog, for the ToPage to get populated, PageRanges should be present in the attribute set. In this case, there is no PageRanges so it defaults to "from 1 to 1" pages. 
Proposed fix is to set a PageRange attribute to be from 1 to getNumberOfPages. 

The 2nd issue of disabling PageRange fields if getNumberOfPages() returns 0 is not handled as there is no provision of passing PageRanges 0 to ServiceDialog (it resuls in IAE). 



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20160901/031b77be/attachment.html>

More information about the 2d-dev mailing list