[icedtea-web] RFC: fix null dtd for swing applications rendering html

[Omair Majid]

Hi,

There has been a regression in netx: applications that render html 
(using Swing components) now fail with a null pointer exception. This 
problem can be reproduced by running the program at [1].

The problem is caused by how HTMLEditorKit/HTMLDocument and 
ParserDelgator interact in the presence of multiple AppContexts. The 
instance of the ParserDelegate is shared between multiple HTMLEditorKit 
in different AppContexts. However, ParserDelegate maintains a 
per-AppContext dtd instance. The dtd is valid in the AppContext that 
created the ParserDelegate but null in all other AppContexts. Using this 
null dtd throws a NullPointerException. This is really a bug in OpenJDK6 
rather than Netx. Fortunately, this bug has been fixed in OpenJDK7 - 
ParserDelegate maintains an instance of the dtd per runtime instead of 
per AppContext.

The attached patch works around this bug. It manually creates a 
ParserDelegate for each AppContext to avoid this problem.

ChangeLog
2010-10-26  Omair Majid  <omajid at redhat.com>

     * netx/net/sourceforge/jnlp/Launcher.java
     (doPerApplicationAppContextHacks): New method.
     (TgThread.run): Call doPerApplicationAppContextHacks.
     * netx/net/sourceforge/jnlp/runtime/JNLPRuntime
     (initialize): Call doMainAppContextHacks.
     (doMainAppContextHacks): New method.

Any comments?

Thanks,
Omair

[1] http://www.ucware.com/apev/jaws/apev.jnlp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: icedtea-web-parser-per-appcontext.patch
Type: text/x-patch
Size: 3176 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20101026/e9837d54/icedtea-web-parser-per-appcontext.patch