<AWT Dev> [9] Review Request for 8130125: [TEST_BUG] add @modules to the several client tests unaffected by the automated bulk update

[Yuri Nesterenko]

Hi,

could you please take a look at the 2nd version:

http://cr.openjdk.java.net/~yan/8130125/webrev.01

Bug is here:
https://bugs.openjdk.java.net/browse/JDK-8130125

Thanks,
-yan


On 07/02/2015 04:50 PM, Yuri Nesterenko wrote:
> On 07/02/2015 03:57 PM, Sergey Bylokhov wrote:
>> Hi, Yuri.
>>   - bug6980209.java can use some public logging?
>>   - FocusEmbeddedFrameTest/ JButtonInGlassPaneOverlapping/etc why it
>> needs a peer?
> It's UtilInternal needs it, and that helper should be fixed
> in Jake by addExports.
>
>>   - JSplitPaneOverlapping please split the long description
>>   - bug6608456 why it needs com.sun.java.swing?
> That's for SwingUtilities3 (sic)!
>
> I'll be back with logging and description.
>
> Thank you!
>
> -yan
>
>>
>> On 02.07.15 13:55, Yuri Nesterenko wrote:
>>> Colleagues,
>>>
>>> please review this test change:
>>> http://cr.openjdk.java.net/~yan/8130125/webrev.00/
>>>
>>> In the most of the tests just @modules directive added;
>>> other than that, see a short comment in
>>> https://bugs.openjdk.java.net/browse/JDK-8130125
>>>
>>> Also I'd like to piggyback in the push 3 tests
>>> moved from closed part in already approved
>>> http://cr.openjdk.java.net/~yan/8081306/
>>> I forgot to do 'hg add', sorry.
>>>
>>> Thank you!
>>> -yan
>>
>>
>> --
>> Best regards, Sergey.
>>
>