<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    The fix looks good to me.<br>
    <br>
    Thanks,<br>
    Alexandr.<br>
    <br>
    <div class="moz-cite-prefix">On 7/6/2016 3:15 PM, Alexander
      Zvegintsev wrote:<br>
    </div>
    <blockquote cite="mid:577CF66D.7020101@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      <a moz-do-not-send="true" class="moz-txt-link-freetext"
        href="http://cr.openjdk.java.net/%7Eazvegint/jdk/9/8159374/01/">http://cr.openjdk.java.net/~azvegint/jdk/9/8159374/01/</a><br>
      <br>
      It is possible, it can be checked on the peer level only
      partially(as long as we have API for icon badge with a String as a
      parameter only).<br>
      If this is not the acceptable solution, we might consider to add <tt>setIconBadge(long
        badge)</tt> method.<br>
      <pre class="moz-signature" cols="72">--
Thanks,
Alexander.</pre>
      <div class="moz-cite-prefix">On 07/06/2016 10:25 AM, Alexandr
        Scherbatiy wrote:<br>
      </div>
      <blockquote
        cite="mid:8e94f295-8444-75be-caa6-93c661da0961@oracle.com"
        type="cite">
        <meta content="text/html; charset=utf-8"
          http-equiv="Content-Type">
        On 7/5/2016 5:54 PM, Alexander Zvegintsev wrote:<br>
        <blockquote
          cite="mid:498e5517-9cbe-8b43-a67d-065b8cc9b52e@oracle.com"
          type="cite">
          <meta content="text/html; charset=utf-8"
            http-equiv="Content-Type">
          <p>We have two cases:</p>
          <ol>
            <li>Underlying system(Linux) supports only numbers as
              parameter (adding this check will block Linux).<br>
            </li>
            <li>Underlying system(OS X) supports strings as parameter,
              but since we can always convert a number to a string
              numbers are supported too.</li>
          </ol>
          <p>So if system doesn't support numbers it doesn't support
            strings too, thus this exception will be thrown in this
            case.</p>
        </blockquote>
           Is it possible that on Linux using a string parameter does
        not throw the specified exception? Could it be checked on the
        peer level?<br>
        <br>
          Thanks,<br>
          Alexandr. <br>
        <br>
        <blockquote
          cite="mid:498e5517-9cbe-8b43-a67d-065b8cc9b52e@oracle.com"
          type="cite"> <br>
          <div class="moz-cite-prefix">On 7/5/16 5:32 PM, Alexandr
            Scherbatiy wrote:<br>
          </div>
          <blockquote
            cite="mid:de436de3-94d9-e8ee-508e-1c239443c7de@oracle.com"
            type="cite">On 7/5/2016 5:04 PM, Alexander Zvegintsev wrote:
            <br>
            <blockquote type="cite">Hello, <br>
              <br>
              please review the fix <br>
              <a moz-do-not-send="true" class="moz-txt-link-freetext"
                href="http://cr.openjdk.java.net/%7Eazvegint/jdk/9/8159374/00">http://cr.openjdk.java.net/~azvegint/jdk/9/8159374/00</a>
              <br>
              for the issue <br>
              <a moz-do-not-send="true" class="moz-txt-link-freetext"
                href="https://bugs.openjdk.java.net/browse/JDK-8159374">https://bugs.openjdk.java.net/browse/JDK-8159374</a>
              <br>
            </blockquote>
              Should the  "checkFeatureSupport(Feature.ICON_BADGE_TEXT)"
            be added to the setIconBadge() method? <br>
            <br>
              Thanks, <br>
              Alexandr. <br>
            <br>
            <br>
          </blockquote>
          <br>
        </blockquote>
        <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>