<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 04.03.2015 16:33, Semyon Sadetsky
      wrote:<br>
    </div>
    <blockquote cite="mid:54F709B9.2030700@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      Sergey, <br>
      OK. You can file a request to improve TrayIcon test coverage but
      it's an another story. Here we are discussing regression test for
      a specific bug.<br>
      I cannot agree with the approach you've proposed. In your logic we
      need start to write regression tests for all methods in JDK
      classes those take String parameters by iterating all possible
      parameters lengths to discover potential bugs.<br>
    </blockquote>
    Yes, if we know that there is a high probability to crash the
    method.
    <blockquote cite="mid:54F709B9.2030700@oracle.com" type="cite">
      Tests need to be written by certain rules. Scanning all possible
      values of input parameters is not the best rule in my opinion.<br>
    </blockquote>
    Validation of one number is useless too. Its even does not check all
    corner cases , 0,1,max-1,max,max+1. so the range is better.<br>
    <blockquote cite="mid:54F709B9.2030700@oracle.com" type="cite"> <br>
      --Semyon<br>
      <br>
      <div class="moz-cite-prefix">On 3/4/2015 4:03 PM, Sergey Bylokhov
        wrote:<br>
      </div>
      <blockquote cite="mid:54F702A5.8090304@oracle.com" type="cite">
        <meta content="text/html; charset=utf-8"
          http-equiv="Content-Type">
        <div class="moz-cite-prefix">On 04.03.2015 15:08, Semyon
          Sadetsky wrote:<br>
        </div>
        <blockquote cite="mid:54F6F5A0.2050304@oracle.com" type="cite">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. <br>
          Lets see what we have here: <br>
          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. <br>
          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. <br>
        </blockquote>
        The goal of the test is to catch all possibly related issues. So
        instead of numbers use ranges, cover all platforms if the test
        don't use platform's specific classes, cover all look and feels,
        automatic test is better than manual, make the test generic. But
        it has of course some restrictions like performance and
        stability. Moreover it seems that right now
        <meta http-equiv="content-type" content="text/html;
          charset=utf-8">
        displayMessage/setToolTip are used in the manual tests only.<br>
        <br>
        <blockquote cite="mid:54F6F5A0.2050304@oracle.com" type="cite">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. <br>
          <br>
          --Semyon <br>
          <br>
          On 3/4/2015 2:13 PM, Sergey Bylokhov wrote: <br>
          <blockquote type="cite">Hi, Semyon. <br>
            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. <br>
            Please add "tray.remove(trayIcon);" at the end of the test,
            otherwise it can hang, when will be run w/o jtreg. <br>
            <br>
            On 03.03.2015 15:32, Semyon Sadetsky wrote: <br>
            <blockquote type="cite">accepted. <br>
              <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ealexsch/semyon-sadetsky/8072769/webrev.03/">http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/8072769/webrev.03/</a>
              <br>
              <br>
              <br>
              On 3/3/2015 1:16 PM, Alexander Scherbatiy wrote: <br>
              <blockquote type="cite"> <br>
                 Just few comments about the test: <br>
                  - The test sets WindowsLookAndFeel and seems fails on
                non Windows platforms. <br>
                  - There is the second SystemTray.isSupported() check
                on line 49. Does it depends on L&F? <br>
                  - The copyright should be updated to 2015. <br>
                <br>
                 Thanks, <br>
                 Alexandr. <br>
                <br>
                On 3/2/2015 4:01 PM, Semyon Sadetsky wrote: <br>
                <blockquote type="cite">Hello, <br>
                  <br>
                  Test was added. Please review. <br>
                  webrev: <a moz-do-not-send="true"
                    class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ealexsch/semyon-sadetsky/8072769/webrev.02/">http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/8072769/webrev.02/</a>
                  <br>
                  bug: <a moz-do-not-send="true"
                    class="moz-txt-link-freetext"
                    href="https://bugs.openjdk.java.net/browse/JDK-8072769">https://bugs.openjdk.java.net/browse/JDK-8072769</a>
                  <br>
                  <br>
                  <br>
                  On 2/26/2015 10:46 AM, Semyon Sadetsky wrote: <br>
                  <blockquote type="cite">fix updated: <a
                      moz-do-not-send="true"
                      class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ealexsch/semyon-sadetsky/8072769/webrev.01/">http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/8072769/webrev.01/</a>
                    <br>
                    <br>
                    On 2/24/2015 12:12 PM, Semyon Sadetsky wrote: <br>
                    <blockquote type="cite">Hello, <br>
                      <AWT Dev> [9] review request for 8061636:
                      Fix for JDK-7079254 changes behavior of
                      MouseListener, MouseMotionListener <br>
                      please review the fix:<a moz-do-not-send="true"
                        class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ealexsch/semyon-sadetsky/8072769/webrev/">http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/8072769/webrev/</a>
                      <br>
                      <br>
                      for the issue:<a moz-do-not-send="true"
                        class="moz-txt-link-freetext"
                        href="https://bugs.openjdk.java.net/browse/JDK-8072769">https://bugs.openjdk.java.net/browse/JDK-8072769</a>
                      <br>
                      <br>
                      System tray icon title freezes java <br>
                      <br>
                      This fix contains: <br>
                      fix corner case: <buffer size> == <length
                      of the string> <br>
                      for two string parameters: <br>
                      1. balloon title string <br>
                      2. balloon text string <br>
                      <br>
                      -- <br>
                      Thanks, <br>
                      <br>
                      Semyon. <br>
                      <br>
                      <br>
                      <br>
                      <br>
                    </blockquote>
                    <br>
                  </blockquote>
                  <br>
                </blockquote>
                <br>
              </blockquote>
              <br>
            </blockquote>
            <br>
            <br>
          </blockquote>
          <br>
        </blockquote>
        <br>
        <br>
        <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
      </blockquote>
      <br>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
  </body>
</html>