Re: [8u40] Review request for RT-38058: Pagination leak

2014-10-07 Thread Morris Meyer
Here is my latest webrev based on Jonathan's comments in JIRA for RT-38058. --morris WEBREV - http://cr.openjdk.java.net/~morris/RT-38058.02 On 10/6/14, 3:53 PM, Morris Meyer wrote: Here is an open webrev: WEBREV - http://cr.openjdk.java.net/~morris/RT-38058.01 On 10/6/14, 2:50 PM,

[8u40] Review request for RT-38058: Pagination leak

2014-10-06 Thread Morris Meyer
Jonathan, I've pulled out the following patch from this JIRA issue and used jvisualvm to verify that this indeed does solve our pagination leak using the HelloPagination app. Could you please give this a review? Thanks. --morris BUG - https://javafx-jira.kenai/browse/RT-38058

Re: [8u40] Review request for RT-38058: Pagination leak

2014-10-06 Thread Morris Meyer
Here is an open webrev: WEBREV - http://cr.openjdk.java.net/~morris/RT-38058.01 On 10/6/14, 2:50 PM, Morris Meyer wrote: Jonathan, I've pulled out the following patch from this JIRA issue and used jvisualvm to verify that this indeed does solve our pagination leak using the HelloPagination