<AWT Dev> [8] Review request for 8021815: Add regression test for JDK-8007267

[Sergey Bylokhov]

Hi, Leonid.
Fix looks fine.
On 29.07.2013 22:34, Leonid Romanov wrote:
> Here is a second version of the fix:
> http://cr.openjdk.java.net/~leonidr/8021815/webrev.01/
>
> On Jul 29, 2013, at 9:21 PM, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
>
>> On 29.07.2013 21:10, Leonid Romanov wrote:
>>> I see, thanks… Could you point me to some test that does it in a right way, so I can copy from it?
>>>
>>> On Jul 29, 2013, at 9:08 PM, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
>>>
>>>> Hi, Leonid.
>>>> Are you sure that the test can be compiled on the platforms other than mac?
>>>> import com.apple.eawt.*;
>>>>
>>>> you need to use a reflection.
>>>>
>>>> On 29.07.2013 21:03, Leonid Romanov wrote:
>>>>> Hello,
>>>>> Please review a fix 8021815: Add regression test for JDK-8007267. This fix simply adds a regression test for 8007267: [macosx] com.apple.eawt.Application.setDefaultMenuBar is not working
>>>>> .
>>>>>
>>>>> Bug: http://bugs.sun.com/view_bug.do?bug_id=8021815 (might not be available yet)
>>>>> Webrev: http://cr.openjdk.java.net/~leonidr/8021815/webrev.00/
>>>>>
>>>>> Thanks,
>>>>> Leonid.
>>>> -- 
>>>> Best regards, Sergey.
>>>>
>>
>> -- 
>> Best regards, Sergey.
>>


-- 
Best regards, Sergey.