<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">Approved.<br>
      <br>
      -phil.<br>
      <br>
      On 11/23/2015 01:39 AM, Jayathirth D V wrote:<br>
    </div>
    <blockquote cite="mid:09540635-967a-49a0-8196-607bd4ca85fa@default"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=ISO-8859-1">
      <meta name="Generator" content="Microsoft Word 15 (filtered
        medium)">
      <style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;
        color:black;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:#0563C1;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:#954F72;
        text-decoration:underline;}
span.EmailStyle17
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
      <div class="WordSection1">
        <p class="MsoNormal"><span style="color:#1F497D">Hello All,<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Changed test
            image used in previous webrev. Because I need image with
            similar properties in another fix(JDK - 8041501) but content
            should be white.<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Changed image
            and its name in test file no other code changes.<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <p class="MsoNormal"><u><span style="color:#1F497D">Updated
              Webrev: <o:p></o:p></span></u></p>
        <p class="MsoNormal"><u><span style="color:#1F497D"><o:p><span
                  style="text-decoration:none"> </span></o:p></span></u></p>
        <p class="MsoNormal"><span style="color:#1F497D"><a
              moz-do-not-send="true"
              href="http://cr.openjdk.java.net/%7Erchamyal/jay/8074967/webrev.01/">http://cr.openjdk.java.net/~rchamyal/jay/8074967/webrev.01/</a><o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Please review.<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Thanks,<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Jay<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <div>
          <div style="border:none;border-top:solid #E1E1E1
            1.0pt;padding:3.0pt 0in 0in 0in">
            <p class="MsoNormal"><b><span style="color:windowtext">From:</span></b><span
                style="color:windowtext"> Jayathirth D V <br>
                <b>Sent:</b> Thursday, November 19, 2015 3:19 PM<br>
                <b>To:</b> Philip Race; Prasanta Sadhukhan;
                <a class="moz-txt-link-abbreviated" href="mailto:2d-dev@openjdk.java.net">2d-dev@openjdk.java.net</a><br>
                <b>Subject:</b> <OpenJDK 2D-Dev> Review request
                for JDK-8074967: JPEGImageReader incorrectly identifies
                YCbCr JPEGs as RGB in standard metadata<o:p></o:p></span></p>
          </div>
        </div>
        <p class="MsoNormal"><o:p> </o:p></p>
        <p class="MsoNormal">Hi Phil/Prasanta,<o:p></o:p></p>
        <p class="MsoNormal"> <o:p></o:p></p>
        <p class="MsoNormal"><u>Please review following fix in jdk9:<o:p></o:p></u></p>
        <p class="MsoNormal"> <o:p></o:p></p>
        <p class="MsoNormal">Bug : <a moz-do-not-send="true"
            href="https://bugs.openjdk.java.net/browse/JDK-8074967">https://bugs.openjdk.java.net/browse/JDK-8074967</a><o:p></o:p></p>
        <p class="MsoNormal"> <o:p></o:p></p>
        <p class="MsoNormal">Webrev : <a moz-do-not-send="true"
            href="http://cr.openjdk.java.net/%7Erchamyal/jay/8074967/webrev.00/">http://cr.openjdk.java.net/~rchamyal/jay/8074967/webrev.00/</a><o:p></o:p></p>
        <p class="MsoNormal"> <o:p></o:p></p>
        <p class="MsoNormal">Bug : JPEGImageReader incorrectly
          identifies YCbCr JPEGs as RGB in standard metadata<o:p></o:p></p>
        <p class="MsoNormal"> <o:p></o:p></p>
        <p class="MsoNormal">Root cause : In JPEG image if there is no
          JFIF or EXIF header. We are not checking component ID’s to
          determine color space properly. Third channel component ID (3)
          is matching the length of componentSpecs length and we are not
          setting colorspace as YCbCr. It is set to RGB which is wrong.<o:p></o:p></p>
        <p class="MsoNormal"> <o:p></o:p></p>
        <p class="MsoNormal">Solution : We should check for id greater
          than componentSpecs length and not greater than equal. But
          adding a tighter check to match individual component ID is
          better. So made changes to match individual component ID with
          1, 2 & 3 to set color space as YCbCr.<o:p></o:p></p>
        <p class="MsoNormal"> <o:p></o:p></p>
        <p class="MsoNormal">Thanks,<o:p></o:p></p>
        <p class="MsoNormal">Jay<o:p></o:p></p>
        <p class="MsoNormal"><span style="color:windowtext"><o:p> </o:p></span></p>
      </div>
    </blockquote>
    <br>
  </body>
</html>