Hello Sergey,

<<The change looks fine, but please confirm that the test will fail on the build before JDK-8003400 was fixed

I have verified that the test is failing without the fix done for JDK-8003400

<<BTW the new bugid in the @bug tag is not necessary.

Ok, I will remove it before pushing the change.


Regards,

Pankaj

On 30/06/20 8:08 AM, Sergey Bylokhov wrote:
The change looks fine, but please confirm that the test will fail on the build before JDK-8003400 was fixed.
BTW the new bugid in the @bug tag is not necessary.

On 6/26/20 12:01 pm, Pankaj Bansal wrote:
Hi Sergey/Prasanta,

I have made changes to the test to work on all platforms. I have also made changes to run it on all installed look and feels on the platforms. The test is passing on all L&Fs on all platforms. Link in the JBS.

webrev: http://cr.openjdk.java.net/~pbansal/8197560/webrev01/ <http://cr.openjdk.java.net/%7Epbansal/8197560/webrev01/>


Regards,

Pankaj


On 26/06/20 9:39 PM, Sergey Bylokhov wrote:
On 6/25/20 9:27 pm, Prasanta Sadhukhan wrote:
sting only in windows, then probably we need to move the bugfix to WindowsTreeUI before changing the test to windows-only.

I think that even if the fix could be moved to WindowsTreeUI, the use case which caused the bug is not windows specific, and could potentially exist on other L&F and platforms. SO it is better to update the test to work on all
platforms/L&Fs





Reply via email to