<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    Looks good to me<br>
    <br>
    Thanks<br>
    alexp<br>
    <div class="moz-cite-prefix"><br>
    </div>
    <blockquote cite="mid:5567307F.8070409@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      Alexander, thanks for review! <br>
      <br>
      Just found out that the test intermittently fails on Win (due to
      immediate 2nd mouse wheel scroll sometimes missed). <br>
      <br>
      Here is the new version with updated fix (wait for the first
      scroll event then make the second): <br>
      <a moz-do-not-send="true"
        href="http://cr.openjdk.java.net/%7Eanashaty/8041470/webrev.02/">http://cr.openjdk.java.net/~anashaty/8041470/webrev.02/</a><br>
      <br>
      Thanks!<br>
      Anton.<br>
      <br>
      <div class="moz-cite-prefix">On 28.05.2015 16:17, Alexander
        Zvegintsev wrote:<br>
      </div>
      <blockquote cite="mid:5567155C.50101@oracle.com" type="cite">
        <meta content="text/html; charset=utf-8"
          http-equiv="Content-Type">
        the fix looks good to me.<br>
        <pre class="moz-signature" cols="72">Thanks,

Alexander.</pre>
        <div class="moz-cite-prefix">On 05/28/2015 03:32 PM, Anton
          Nashatyrev wrote:<br>
        </div>
        <blockquote cite="mid:55670AE4.2050307@oracle.com" type="cite">
          <meta content="text/html; charset=utf-8"
            http-equiv="Content-Type">
          Hello Alexander, <br>
          <br>
          <div class="moz-cite-prefix">On 27.05.2015 23:04, Alexander
            Zvegintsev wrote:<br>
          </div>
          <blockquote cite="mid:55662348.4030404@oracle.com" type="cite">
            <meta content="text/html; charset=utf-8"
              http-equiv="Content-Type">
            Hello Anton,<br>
            <br>
            As I can see type and wheel_mouse parameters are no longer
            used in getModifiers(),<br>
            so probably we can safely remove them.<br>
          </blockquote>
          <br>
          Sure, please see the new version: <a moz-do-not-send="true"
            href="http://cr.openjdk.java.net/%7Eanashaty/8041470/webrev.01/">http://cr.openjdk.java.net/~anashaty/8041470/webrev.01/</a><br>
          <br>
          Thanks for review!<br>
          Anton.<br>
          <br>
          <blockquote cite="mid:55662348.4030404@oracle.com" type="cite">
            <pre class="moz-signature" cols="72">--
Thanks,
Alexander.</pre>
            <div class="moz-cite-prefix">On 05/27/2015 07:37 PM, Anton
              Nashatyrev wrote:<br>
            </div>
            <blockquote cite="mid:5565F2C7.6020408@oracle.com"
              type="cite">
              <meta http-equiv="content-type" content="text/html;
                charset=utf-8">
              Hello, <br>
                  could you please review the following fix:<br>
              <br>
              fix: <a moz-do-not-send="true"
                href="http://cr.openjdk.java.net/%7Eanashaty/8041470/webrev.00/">http://cr.openjdk.java.net/~anashaty/8041470/webrev.00/</a><br>
              bug: <a moz-do-not-send="true"
                href="https://bugs.openjdk.java.net/browse/JDK-8041470">https://bugs.openjdk.java.net/browse/JDK-8041470</a><br>
              <br>
                  Problem: mouseRelease not called on a component after
              dragging outside and scrolling the wheel <br>
              <br>
                  Fix: correct logical typo <br>
              <br>
              Thanks! <br>
              Anton. <br>
            </blockquote>
            <br>
          </blockquote>
          <br>
        </blockquote>
        <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>