[rfc][icedtea-web] rewritten java console

[Jiri Vanek]

On 12/19/2013 02:12 PM, Jiri Vanek wrote:
>
> hi!
>
> Here is the patch adapted to newest head (especially to config singleton). It is also applied on top
> of "[rfc] [icedtea-web] singletons logic, logs and test cleanup/fixes" which are necessary to make
> the stuff running correctly.
>
>
> There are practicaly only htree differences:
>    - gui of console is created when console is shown
>    - the headers have defaults like "unkknown", as wrongly parsed plugin message could cause npe
> otherwise
>    - the initialisation of log x config get a bit connected, see getConfiguration() method change:
>
> --- a/netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java    Wed Dec 18 17:46:52 2013 +0100
> +++ b/netx/net/sourceforge/jnlp/runtime/JNLPRuntime.java    Thu Dec 19 14:00:34 2013 +0100
> @@ -365,6 +365,21 @@
>                   OutputController.getLogger().log(OutputController.Level.MESSAGE_ALL,
> getMessage("RConfigurationError"));
>                   //mark this exceptionas we can die on it later
>                   config.setLoadingException(ex);
> +            } finally{
> +                new Thread(){
> +                    /*
> +                     * this is crucial. The creation of output logger itself do not need config
> +                     * however processing of messages do. Also loading of properties do log.
> +                     * So the start of processing MUST be done AFTER config is
> +                     * initialised, and in fresh, non waiting thread.
> +                     */
> +                    @Override
> +                    public void run() {
> +                        OutputController.getLogger().startConsumer();
> +                    }
> +
> +                }.start();
> +
>               }
>           }
>           retur
>
>
>
>
> I'm in favour to post those two patches  as much as they are, and let the code evolve....
>
> J.


crap:

         System.out.println(folder);
         System.out.println(cacheDir);

do not belongs to this patch and willnot be pushed.