Re: [9] Review request: JDK-8170979: Update for Jigsaw etc. the openFX functional tests

2017-02-15 Thread Yuri Nesterenko
Version .06: http://cr.openjdk.java.net/~yan/8170979/webrev.06/ added, a couple more of add-exports for ControlsTests. Updated https://bugs.openjdk.java.net/browse/JDK-8170979 : an attached 'hg diff' for this version presumably works better than a patch generated by webrev. Thanks, -yan On

Re: [9] Review request: JDK-8170979: Update for Jigsaw etc. the openFX functional tests

2017-02-10 Thread Yuri Nesterenko
Version .05: http://cr.openjdk.java.net/~yan/8170979/webrev.05 added, two lines fixing some 300+ Css tests. Updating https://bugs.openjdk.java.net/browse/JDK-8170979 -- I'll add there a plain diff. Perhaps it would be easier to apply. Thank, -yan On 01/31/2017 04:03 PM, Yuri Nesterenko wrote:

Re: [9] Review request: JDK-8170979: Update for Jigsaw etc. the openFX functional tests

2017-01-31 Thread Yuri Nesterenko
And once more: http://cr.openjdk.java.net/~yan/8170979/webrev.04 Added: a port of JDK-8138872; a change in javaclient/shared/screenshots/GoldenImageManager.java to allow it find old golden images. In the index.html there's a list of 10 files to remove after the patch applied. Webrev

Re: [9] Review request: JDK-8170979: Update for Jigsaw etc. the openFX functional tests

2016-12-29 Thread Yuri Nesterenko
And again: http://cr.openjdk.java.net/~yan/8170979/webrev.03 (Discussion in the bug) Unfortunately, webrev doesn't process properly files deleted by 'hg rm' . -yan On 12/22/2016 12:07 PM, Yuri Nesterenko wrote: A new version is here: http://cr.openjdk.java.net/~yan/8170979/webrev.01

Re: [9] Review request: JDK-8170979: Update for Jigsaw etc. the openFX functional tests

2016-12-22 Thread Yuri Nesterenko
A new version is here: http://cr.openjdk.java.net/~yan/8170979/webrev.01 (trailing spaces removed). See also discussion in the bug report: https://bugs.openjdk.java.net/browse/JDK-8170979 thank you, Kevin! -yan On 12/21/2016 12:19 PM, Yuri Nesterenko wrote: A gentle reminder. Without this

Re: [9] Review request: JDK-8170979: Update for Jigsaw etc. the openFX functional tests

2016-12-21 Thread Yuri Nesterenko
A gentle reminder. Without this change, we have to make All Test Run out of the makeshift workspace. Thanks, -yan On 12/09/2016 02:02 PM, Yuri Nesterenko wrote: Colleagues, please review this test bug fix: http://cr.openjdk.java.net/~yan/8170979/webrev.00/ Bug:

[9] Review request: JDK-8170979: Update for Jigsaw etc. the openFX functional tests

2016-12-09 Thread Yuri Nesterenko
Colleagues, please review this test bug fix: http://cr.openjdk.java.net/~yan/8170979/webrev.00/ Bug: https://bugs.openjdk.java.net/browse/JDK-8170979 The changes are mostly in tests/functional/. This is a merge of current testbase in http://hg.openjdk.java.net/openjfx/9-dev/tests and whatever