[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

review request: 8153657 fix DragDrop builds with ARM

2016-04-06 Thread David Hill 


Kevin,
  could you review the diff in:

https://bugs.openjdk.java.net/browse/JDK-8153657

thanks!

--
David Hill
Java Embedded Development

"A man's feet should be planted in his country, but his eyes should survey the 
world."
-- George Santayana (1863 - 1952)

[9] Review request: 8153645: Mark TaskSwampEventQueueTest as unstable

2016-04-06 Thread Kevin Rushforth 

Dave,

Please review this simple change to mark TaskSwampEventQueueTest as 
unstable. Patch is in JBS:


https://bugs.openjdk.java.net/browse/JDK-8153645

-- 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