should have been 8174025 -> Re: RFR [JAXP] JDK-8170192 Regression in XML Transform caused by JDK-8087303

[huizhe wang]

Note that the bug id was incorrect, it should have been 8174025. 8170192 
was a test bug fix.

-Joe

On 2/13/2017 1:35 AM, Frank Yuan wrote:
> Hi Joe and Daniel
>
> Thank you very much for your review!
>
> Frank
>
>
> -----Original Message-----
> From: huizhe wang [mailto:huizhe.wang at oracle.com]
> Sent: Saturday, February 11, 2017 5:52 AM
> To: Daniel Fuchs; Frank Yuan; core-libs-dev at openjdk.java.net
> Subject: Re: RFR [JAXP] JDK-8170192 Regression in XML Transform caused by JDK-8087303
>
> +1 from me too.
>
> Thanks,
> Joe
>
> On 2/10/2017 5:25 AM, Daniel Fuchs wrote:
>> Hi Frank,
>>
>> Thanks for fixing this!
>>
>> I imported your patch and played with it a bit.
>> Also ran the jaxp test.
>>
>> Both issues reported have indeed disappeared.
>>
>> So that's a +1 from me.
>>
>> best regards,
>>
>> -- daniel
>>
>> On 10/02/17 11:03, Frank Yuan wrote:
>>> Hi All
>>>
>>>
>>>
>>> Would you like to review
>>> http://cr.openjdk.java.net/~fyuan/8174025/webrev.00/?
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8174025
>>>
>>>
>>>
>>> JDK-8087303 introduced 2 issues:
>>>
>>> 1.       Flaw when xlst uses disable-output-escaping attribute
>>>
>>> 2.       Eat the whitespace between html inline elements
>>>
>>>
>>>
>>> This patch fixed the issues.
>>>
>>> To fix the second issue, we decide to keep the compatibility with JDK 8
>>> on the whitespace handling, that is only LSSerializer cleans the extra
>>> whitespaces in if pretty-print is on, but XSLT doesn't.
>>>
>>> I modified the behavior of getIndent() method in class ToStream, to make
>>> LSSerializer be sensitive of current state from ToStream. This should be
>>> safe because ToStream is an internal class and getIndent() method is
>>> never used before.
>>>
>>>
>>>
>>> Thanks
>>>
>>> Frank
>>>
>>>
>>>
>