<AWT Dev> [11] Review Request: 8205153 Delete "standard instructions" machinery in the open automated tests

[Sergey Bylokhov]

I have checked all these tests, there are no any new failures caused by 
this change.

On 22/06/2018 14:10, Phil Race wrote:
> I just want to know that all these updated tests still pass ?
> 
> -phil.
> 
> On 06/22/2018 02:08 PM, Sergey Bylokhov wrote:
>> On 22/06/2018 13:56, Phil Race wrote:
>>> I dozed off trying to review this a couple of days ago ;-)
>>> I really just want to know if the tests all pass after the update ?
>>
>> Only the tests which previously sometimes fail because of instructions 
>> windows which were open on top of the test window. I did not change 
>> any other logic in the tests, other than mechanical removing of part 
>> of the code just to simplify webrev. The next step will be drop of 
>> applet usage(also w/o altered the logic of the tests).
>>
>>>
>>> -phil.
>>>
>>>
>>> On 06/22/2018 01:49 PM, Sergey Bylokhov wrote:
>>>> Any volunteers for review?
>>>> =)
>>>>
>>>> On 18/06/2018 12:09, Sergey Bylokhov wrote:
>>>>> Hello.
>>>>> Please review the fix for jdk11.
>>>>>
>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8205153
>>>>> Webrev: http://cr.openjdk.java.net/~serb/8205153/webrev.00
>>>>>
>>>>> This fix is a part of the effort to cleanup and stabilize our 
>>>>> tests. This change will delete "standard instructions" machinery in 
>>>>> the open automated tests, which is just a copy-pasted code to to 
>>>>> show this message to the user:
>>>>> "This is an AUTOMATIC test, simply wait until it is done."
>>>>>
>>>>> In most cases it doubles the amount of code in the test, and make 
>>>>> it difficult to update such tests. Also not all tests assume that 
>>>>> the instruction frame could be opened on top of the test frame, 
>>>>> which make the test unstable.
>>>>>
>>>>
>>>>
>>>
>>
>>
> 


-- 
Best regards, Sergey.