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

[Yuri Nesterenko]

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.
>