[rfc] [icedtea-web] Fix EDT hanging on OpenJDK 11

[Jiri Vanek]

On 10/2/18 11:37 AM, Mario Torre wrote:
> On Tue, Oct 2, 2018 at 10:56 AM Laurent Bourgès
> <bourges.laurent at gmail.com> wrote:
>>
>> Hi,
>>
>>>
>>> The patch is pushed to both head and 1.7
>>
>>
>> Excellent, I will upgrade my local copy.
>>
>>>
>>> During the applying, I noticed following:
>>> ThreadCheckingRepaintManager was returned, however is unused.  Is that intentional?
>>> Together with it, acordingly, +    private static final boolean DEBUG_EDT =
>>> System.getProperty("icedtea-web.edt.debug", "false").equalsIgnoreCase("true"); was removed.
>>>
>>> I think one without another do not have sense. I would vote for make it enable-able. If not then it
>>> should be removed, as it is now dead code.
>>
>>
>> I can restore its optional usage ( DEBUG_EDT ) byresurecting SwingUtils.setup() if you want.
>>
>>>
>>> Is the next step to rmeove ThreadCheckingRepaintManager or  to make it anebale-able?
>>
>>
>> I will re-enable it if you and Mario agree.
> 
> I'm not sure what Jiri means by "re-enable", it should not be the
> default imho, however this is his call really ;)
> 

Nope, not default, via the   DEBUG_EDT = System.getProperty("icedtea-web.edt.debug", 
"false").equalsIgnoreCase("true");


Tahnx!
   J.


-- 
Jiri Vanek
Senior QE engineer, OpenJDK QE lead, Mgr.
Red Hat Czech
jvanek at redhat.com    M: +420775390109