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, 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 app.


Could you please give this a review?  Thanks.

--morris

BUG - https://javafx-jira.kenai/browse/RT-38058
WEBREV - http://javaweb.us.oracle.com/~mameyer/webrevs/01/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 app.


Could you please give this a review?  Thanks.

--morris

BUG - https://javafx-jira.kenai/browse/RT-38058
WEBREV - http://javaweb.us.oracle.com/~mameyer/webrevs/01/RT-38058