<AWT Dev> [9] review request for 8072769: System tray icon title freezes java

[Semyon Sadetsky]

Thanks, Sergey!

tray.remove at the end is accepted.
Below is about your proposal to run reg test on all other platforms and 
scan parameters lengths:
--
I think it's more like bug regression test policy. Since I'm a novice in 
the project maybe you or somebody can advise me the right process.
Lets see what we have here:
The fix is connected to the windows platform native code only. It is a 
platform specific corner case. I would not say it contains "magic 
numbers", those numbers a not magic for the native layer. But in the 
test we call native code from java where in truth string parameters 
maximum lengths are not specified.
I think it make sense to run the test on all other platforms test 
iterating strings lengths up to 1000 or more just to ensure that there 
no similar issues there. We can do this once or have an option to switch 
this scenario on.

But I'm not 100% sure that such test scenario should be included in the 
regular regression run because sending messages to the system tray is 
not very fast and the scan can take noticeable amount of time. In my 
opinion such issue seeking scenario is reasonable to run time to time or 
as a part of stress testing profile, but it does not make sense to run 
it during the regular regression because it tests nothing specific on 
platforms other than Windows and even in Windows there is only one 
specific lengths combination. It's not worth to do this potential issue 
seek it's time expensive but will not bring us a lot of value really.

--Semyon

On 3/4/2015 2:13 PM, Sergey Bylokhov wrote:
> Hi, Semyon.
> I suggest to remove stuff related to windows platform from the test. 
> Also it will be good to test some reasonable range of data instead of 
> magic constant, wrap setTooltip/displayMessage in a loop.
> Please add "tray.remove(trayIcon);" at the end of the test, otherwise 
> it can hang, when will be run w/o jtreg.
>
> On 03.03.2015 15:32, Semyon Sadetsky wrote:
>> accepted.
>> http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/8072769/webrev.03/
>>
>>
>> On 3/3/2015 1:16 PM, Alexander Scherbatiy wrote:
>>>
>>>  Just few comments about the test:
>>>   - The test sets WindowsLookAndFeel and seems fails on non Windows 
>>> platforms.
>>>   - There is the second SystemTray.isSupported() check on line 49. 
>>> Does it depends on L&F?
>>>   - The copyright should be updated to 2015.
>>>
>>>  Thanks,
>>>  Alexandr.
>>>
>>> On 3/2/2015 4:01 PM, Semyon Sadetsky wrote:
>>>> Hello,
>>>>
>>>> Test was added. Please review.
>>>> webrev: 
>>>> http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/8072769/webrev.02/
>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8072769
>>>>
>>>>
>>>> On 2/26/2015 10:46 AM, Semyon Sadetsky wrote:
>>>>> fix updated: 
>>>>> http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/8072769/webrev.01/ 
>>>>>
>>>>>
>>>>> On 2/24/2015 12:12 PM, Semyon Sadetsky wrote:
>>>>>> Hello,
>>>>>> <AWT Dev> [9] review request for 8061636: Fix for JDK-7079254 
>>>>>> changes behavior of MouseListener, MouseMotionListener
>>>>>> please review the 
>>>>>> fix:http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/8072769/webrev/ 
>>>>>>
>>>>>>
>>>>>> for the issue:https://bugs.openjdk.java.net/browse/JDK-8072769
>>>>>>
>>>>>> System tray icon title freezes java
>>>>>>
>>>>>> This fix contains:
>>>>>> fix corner case: <buffer size> == <length of the string>
>>>>>> for two string parameters:
>>>>>> 1. balloon title string
>>>>>> 2. balloon text string
>>>>>>
>>>>>> -- 
>>>>>> Thanks,
>>>>>>
>>>>>> Semyon.
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>
>
>