Re: AWT Dev [8] Request for review: 7080109 Dialog.show() lacks doPrivileged() to access system event queue

Hi Sergey, The fix looks good to me. -- best regards, Anthony On 5/4/2012 7:55 PM, Sergey Bylokhov wrote: Hi Everyone. Artem thanks for review. Does anybody has a chance to review the fix? One more reviewer needed. Thanks. 16.04.2012 21:00, Sergey Bylokhov wrote: Hi, Artem. Thanks for

Re: AWT Dev [8] Request for review: 7080109 Dialog.show() lacks doPrivileged() to access system event queue

Hi Everyone. Artem thanks for review. Does anybody has a chance to review the fix? One more reviewer needed. Thanks. 16.04.2012 21:00, Sergey Bylokhov wrote: Hi, Artem. Thanks for view. Scope was changed. Here is an updated version: http://cr.openjdk.java.net/~serb/7080109/webrev.01/

Re: AWT Dev [8] Request for review: 7080109 Dialog.show() lacks doPrivileged() to access system event queue

Approved! Thanks, Artem On 4/16/2012 9:00 PM, Sergey Bylokhov wrote: Hi, Artem. Thanks for view. Scope was changed. Here is an updated version: http://cr.openjdk.java.net/~serb/7080109/webrev.01/ 05.04.2012 14:46, Artem Ananiev wrote: Hi, Sergey, to limit the scope of the added

Re: AWT Dev [8] Request for review: 7080109 Dialog.show() lacks doPrivileged() to access system event queue

Hi, Artem. Thanks for view. Scope was changed. Here is an updated version: http://cr.openjdk.java.net/~serb/7080109/webrev.01/ 05.04.2012 14:46, Artem Ananiev wrote: Hi, Sergey, to limit the scope of the added doPrivileged() block, I would rewrite the code this way: secondaryLoop =

AWT Dev [8] Request for review: 7080109 Dialog.show() lacks doPrivileged() to access system event queue

Hi Everyone, In the fix doPrivileged was added to Dialog.show(). Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7080109 Webrev can be found at: http://cr.openjdk.java.net/~serb/7080109/webrev.00/ -- Best regards, Sergey.