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

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Fri Sep 4 14:43:29 UTC 2015


   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
>



More information about the awt-dev mailing list