<AWT Dev> [9] Review request for 8081787: [macosx] MalformedURLException is thrown during reading data for application/x-java-url; class=java.net.URL flavor

[Alexander Scherbatiy]

   The fix looks good to me.

   Thanks,
   Alexandr.

On 8/13/2015 3:08 PM, Sergey Bylokhov wrote:
> The fix looks fine. Thanks!
>
> On 05.08.15 19:08, mikhail cherkasov wrote:
>> Hi Sergey,
>>
>> Thanks you for examples of manual tests, I've added the test for my fix:
>>
>> http://cr.openjdk.java.net/~mcherkas/8081787.9/webrev.01/
>>
>> Could you please review it?
>>
>> Thanks,
>> Mikhail.
>>
>> On 7/2/2015 22:00, Sergey Bylokhov wrote:
>>> Hi, Mikhail.
>>> The test will be useful, at least for verification. On my osx for 
>>> example the test from the bug description fails with 
>>> UnsupportedFlavorException: application/x-java-url
>>>
>>> On 01.07.15 14:27, mikhail cherkasov wrote:
>>>> Hi there,
>>>>
>>>> Could you please review a fix for:
>>>> 8081787 [macosx] MalformedURLException is thrown during reading 
>>>> data for application/x-java-url;class=java.net.URL flavor
>>>> https://bugs.openjdk.java.net/browse/JDK-8081787
>>>>
>>>> http://cr.openjdk.java.net/~mcherkas/8081787.8/webrev.01/src/macosx/classes/sun/lwawt/macosx/CDataTransferer.java.udiff.html 
>>>>
>>>>
>>>> Mac Os X pasteboard returns a property list that consists of one 
>>>> URL, but our code supposes to get pure URL. I added extracting URL 
>>>> before passing
>>>> text to URL ctor.
>>>>
>>>> I'm not sure how to implement a test for the fix, to test it 
>>>> properly we need a particular content in pasteboard, I don't know 
>>>> how to get this
>>>> without manual coping it.
>>>>
>>>> Thanks,
>>>> Mikhail.
>>>>
>>>
>>>
>>
>
>