<AWT Dev> [8] Request for review: 7080109 Dialog.show() lacks doPrivileged() to access system event queue
Anthony Petrov
anthony.petrov at oracle.com
Sat May 5 06:57:44 PDT 2012
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 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 = AccessController.doPrivileged(
>>> new PrivilegedAction<SecondaryLoop>() {
>>> public SecondaryLoop run() {
>>> EventQueue eventQueue =
>>> Toolkit.getDefaultToolkit().getSystemEventQueue();
>>> return eventQueue.createSecondaryLoop(
>>> cond, modalFilter, 0);
>>> }
>>> }
>>> );
>>> if (!secondaryLoop.enter()) {
>>> secondaryLoop = null;
>>> }
>>>
>>> Thanks,
>>>
>>> Artem
>>>
>>> On 3/30/2012 8:22 PM, Sergey Bylokhov wrote:
>>>> 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/
>>>>
>>
>>
>
More information about the awt-dev
mailing list