<AWT Dev> <Awt Dev> [9] Review Request for 8014725: closed/java/awt/Clipboard/HTMLTransferTest/HTMLTransferTest.html failed intermittently

[Alexander Zvegintsev]

+1

Thanks,

Alexander.

On 09/04/2015 05:43 PM, Alexander Scherbatiy wrote:
>
>   The fix looks good to me.
>
>   Thanks,
>   Alexandr.
>
> On 9/3/2015 4:21 PM, Semyon Sadetsky wrote:
>> Alexanders? Any questions?
>>
>> On 8/5/2015 3:11 PM, Semyon Sadetsky wrote:
>>> Hello,
>>>
>>> Please review fix for JDK9:
>>>
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8014725
>>> webrev: http://cr.openjdk.java.net/~ssadetsky/8014725/webrev.00/
>>>
>>> The test is moved from closed to open repo. Initially it was a test 
>>> issue: new document html flavor parameters introduced in 7075105 
>>> were not taken into account.
>>> After the test is fixed and run an issue was found in the code. In 
>>> case InputStream is used as transferable object it is converted to a 
>>> number of compatible formats but during this conversion InputStream 
>>> can be read many times without resetting its position after each 
>>> read. As result the InputStream is exhausted during the first flavor 
>>> conversion and then all consequent flavors receive empty content. 
>>> Stream reset was added as solution to this.
>>>
>>> --Semyon
>>
>