JDK-8102203 : ES2RTTexture.readPixels allocates new byte buffer for each frame
  • Type: Enhancement
  • Component: javafx
  • Sub-Component: graphics
  • Affected Version: 7u6
  • Priority: P2
  • Status: Closed
  • Resolution: Fixed
  • Submitted: 2012-05-23
  • Updated: 2015-06-16
  • Resolved: 2012-06-08
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 7
7u6Fixed
Related Reports
Relates :  
Relates :  
Relates :  
Relates :  
Description
The bug affects JFXPanel (especially with animation) causing very slow performance and occasional OOM exceptions.
Comments
verified in 2.2b18
27-07-2012

Fixed. Changeset: df65c739605a Author: Chien Yang <chien.yang@oracle.com> Date: Fri Jun 08 09:50:12 2012 -0700 URL: http://jfxsrc.us.oracle.com/javafx/2.2/scrum/graphics/rt-closed/rev/df65c739605a
08-06-2012

Compiled and tested on all 3 (Mac, Linux and Windows platforms. Programs tested include the various HelloRenderToImage and HelloSnapshot, modified Brick Breaker and Ensemble. To see the performance improvement run colorcube/JFXPanelColorCube and notice the regular long pauses are now gone. Webrev: http://javaweb.us.oracle.com/~ckyang/reviews/RT-21740/webrev-v1/
07-06-2012

Will work on it for this sprint
23-05-2012

Yes, Oleg and I did sit through to analyze it yesterday.
23-05-2012

I just checked, and ES2NRTTexture does have this same problem.
23-05-2012

If ES2NRTTexture.readPixels has the same issue, then this needs to be fixed for 2.2.
23-05-2012

I could see the problem with the BrickBreaker sample when embedded in JFXPanel. I'm attaching the modified Main.java source file I used. To run in JFXPanel specify -Dembedded=true
23-05-2012