[RFC][netx]: jnlp app closing issue

Denis Lila dlila at redhat.com
Thu Jun 9 14:54:26 PDT 2011


I would like to push the above two patches.

The first fixes a problem where
javaws http://www.crossftp.com/crossftp.jnlp
will keep running in the background even after there is an
error while loading.

The second is just an improvement (IMO) in the jnlp splashscreen.
It is motivated by the fact that the splashscreen had to be
closed manually in the above example.

ChangeLog (for the first patch):

2011-06-09  Denis Lila  <dlila at redhat.com>

    * netx/net/sourceforge/jnlp/cache/DefaultDownloadIndicator.java:
    (frameMutex): New mutex to synchronize accesses to "frame".
    (getListener): Make almost all of it synchronized on frameMutex.
    (disposeListener): Sync hider's body around frameMutex and call
    dispose on the frame so that the awt threads die when they should.
    (addProgressPanel): Sync "frame" usage.

Second patch:

2011-06-09  Denis Lila  <dlila at redhat.com>

    * netx/net/sourceforge/jnlp/GuiLaunchHandler.java:
    (mutex): New mutex for synchronizing splashScreen.
    (closeSplashScreen): New method to hide and dispose
    (launchStarting): Call closeSplashScreen instead of doing it
    (launchInitialized): Sync splashScreen creation.
    (validationError, launchError): Call closeSplashScreen.

Any comments are appreciated.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: app-hangs-around-after-error.patch
Type: text/x-patch
Size: 3867 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20110609/d0fe6239/app-hangs-around-after-error.patch 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: improve-splashscreen-closing.patch
Type: text/x-patch
Size: 2742 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20110609/d0fe6239/improve-splashscreen-closing.patch 

More information about the distro-pkg-dev mailing list