Re: Code review request: 8026929: remove accelerators from policytool resources

2013-10-28 Thread Alexander Potochkin
Hello Leif Okay, I approve Thanks alexp On 10/22/2013 8:12 PM, Leif Samuelsson wrote: Hi Alexander, On 2013-10-22 04:25, Alexander Potochkin wrote: Hello Leif There was a special logic for keyStrokes on MacOS inPolicyTool.addMenuItem(), could you make sure that the new code correctly set

Re: [8] Review Request: JDK-8027157 [SwingNode] needs explicit expose for JWindow

2013-10-28 Thread Artem Ananiev
Thanks. Artem On 10/25/2013 5:45 PM, Anton V. Tarasov wrote: Hi Artem, On 10/24/13 6:59 PM, Artem Ananiev wrote: Hi, Anton, it looks much better now. Is it possible to create a regression test for this change? Ok, I did that. Here's the test: http://cr.openjdk.java.net/~ant/RT-32570.test