[Jake Review]: [packager] Module Path Packager Arguments

2016-04-06 Thread Chris Bensen
Kevin,

Please review. This adds new argument to Java Packager and cleans a lot of 
things up. My Packager sandbox will be in sync except for refactoring changes.

https://bugs.openjdk.java.net/browse/JDK-8150991 
http://cr.openjdk.java.net/~cbensen/JDK-8150991/webrev.01
 

Chris

[9] Review request: 8153539: JavaFX build script checking order for jfxrt.jar is non-deterministic

2016-04-06 Thread Kevin Rushforth

Chien or Dave,

Please review the following fix to the gradle dependencies:

https://bugs.openjdk.java.net/browse/JDK-8153539
http://cr.openjdk.java.net/~kcr/8153539/webrev.00/

-- Kevin



[webkit] [9] Review request for 8153151: Call JavaScriptCore GC whenever JVM GC happens

2016-04-06 Thread Arunprasad Rajkumar

Hi Kevin, Alexander, Guru,

Please review the below patch.

JIRA: https://bugs.openjdk.java.net/browse/JDK-8153151

Webrev: http://cr.openjdk.java.net/~arajkumar/8153151/webrev.00

Issue: Few JVM heap objects(ImageDecoder, Font,..) are held by JavaScript 
garbages.

Fix: Add a dummy object to com.sun.webkit.Disposer, DisposerRecord.dispose(..) 
will be called once JVM GC's our dummy object. Call JSC's GC function from 
DisposerRecord.dispose(..).


Regards,
Arun





[9] Review request: 8153479: JavaFX unit tests fail after fix for JDK-8079136

2016-04-06 Thread Kevin Rushforth

Hi Vadim,

Can you review this simple fix to our unit tests?

https://bugs.openjdk.java.net/browse/JDK-8153479
http://cr.openjdk.java.net/~kcr/8153479/webrev.00/

Thanks.

-- Kevin