[7u6] Please review my fix for 7148289: [macosx] Deadlock in sun.lwawt.macosx.CWrapper$NSScreen.visibleFrame

[Alexander Zuev]

Anthony,

   finally all the servers are up and i can present the new webrev, it 
is here:
http://cr.openjdk.java.net/~kizune/7148289/webrev.01

I have renamed methods mentioned that they are nested native event loop 
controllers.

With best regards,
Alex

On 4/28/12 14:52, Anthony Petrov wrote:
> Hi Alexander,
>
> On 4/28/2012 12:16 PM, Alexander Zuev wrote:
>>   the ToolkitThreadBlockerHandler is an interface that is supposed to 
>> be used
>> when program has to wait for the event pushed to the EDT finish its 
>> task and
>> he calls like this:
>>             while (!dispatcher.isDone()) {
>>                 
>> DataTransferer.getInstance().getToolkitThreadBlockedHandler().enter();
>>             }
>> and the event handler in its unregisterEvent() method calls for 
>> handler.exit()
>> The method is supposed to make sure that waiting on the EDT doesn't 
>> create deadlocks.
>> The implementation depends on the toolkit architecture - on some 
>> toolkits it's NoOp since
>> they don't care if EDT is waiting, but in LWCToolkit our AWT 
>> components are backed by Swing
>> delegates so there are a lot of calls from native application thread 
>> to EDT and vice versa so we
>> can't afford to stop both threads. Solution is the code that makes 
>> secondary AppKit event loop
>> to handle native events so waiting on EDT will leave at least one 
>> event loop pumping.
>
> Thanks!
>
>>   The name is correct because despite the fact that we are in AWT 
>> code (LWCToolkit) we don't
>> create nested EDT loop but running nested native (AppKit) loop - 
>> hence the name: enter/exitNativeEventLoop()
>
> Since the main event loop is already spinning, I would still suggest 
> to rename it to, say, enter/exitNestedNativeEventLoop() to stress that 
> this event loop is going to be an inner loop, rather than the main 
> (outer) event loop.
>
> -- 
> best regards,
> Anthony
>
>>
>> With best regards,
>> Alex
>>
>> On 4/27/12 20:54, Anthony Petrov wrote:
>>> Hi Alexander,
>>>
>>> Could you provide some details about the ToolkitThreadBlockedHandler 
>>> interface? How is it used by the DnD code?
>>>
>>> Also, I suggest to replace the word Native with Nested in 
>>> enter/exitNativeEventLoop() method names.
>>>
>>> -- 
>>> best regards,
>>> Anthony
>>>
>>> On 4/26/2012 7:54 PM, Alexander Zuev wrote:
>>>> Hello,
>>>>
>>>>   please review my fix for the CR 7148289: [macosx] Deadlock in 
>>>> sun.lwawt.macosx.CWrapper$NSScreen.visibleFrame
>>>>
>>>>   Bug description is 
>>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7148289
>>>>
>>>>   Fix can be found at 
>>>> http://cr.openjdk.java.net/~kizune/7148289/webrev.00
>>>>
>>>> With best regards,
>>>> Alex
>>