<AWT Dev> Review request: 8005932 : Java 7 on mac os x only provides text clipboard formats
mikhail cherkasov
mikhail.cherkasov at oracle.com
Fri Feb 8 01:43:54 PST 2013
Hello there,
New version, with new test:
http://cr.openjdk.java.net/~mcherkas/8005932/webrev.04/
<http://cr.openjdk.java.net/%7Emcherkas/8005932/webrev.04/>
There I've implemented the test with dnd between two JVM.
Thanks,
Mikhail.
05.02.2013 16:50, mikhail cherkasov пишет:
> Hello Denis, All,
>
> Here is new version:
> http://cr.openjdk.java.net/~mcherkas/8005932/webrev.03/
> <http://cr.openjdk.java.net/%7Emcherkas/8005932/webrev.03/>
> Please review it.
>
> 01.02.2013 19:57, Denis S. Fokin пишет:
>> Hi Mikhail,
>> The fix looks good.
>> As for the test, personally, I would not implement it this way. For
>> AWT functionality we use Swing library. By this reason we use a hack
>> with final array.
>> I would replace the next loop
>> 108 for (int i = 0; i < 2; i++) {
>> 109 DropObject.getInstance().setCurrentDf(i % 2 == 0 ?
>> rtfDataFlavor : htmlDataFlavor);
>> with function invocation where I would directly pass "from" and "to"
>> points instead of swapping variables.
>> I am not sure that I am understand the next lines
>> 203 if (dropObject != null) {
>> 204 dtde.rejectDrop();
>> Are we use the transferrable as an initialization flag?
>> Usually, Transferrable objects contain data that can be converted by
>> demand into particular type.
>> The DropObject always returns html as bytes (for the test purposes
>> this is ok). On the other hand, the condition
>> 109 DropObject.getInstance().setCurrentDf(i % 2 == 0 ?
>> rtfDataFlavor : htmlDataFlavor);
>> should be implemented in transferable and available flavors should be
>> encapsulated.
>> The data transfer part of the test should work as follows:
>> 1. Transferable contains an array with two DataFlavor instances
>> 2. isDataFlavorSupported returns true if the array contains the
>> DataFlavor passed as a parameter
>> 3. getTransferDataFlavors returns the array
>> 4. getTransferData works as you have implemented it
>> The transferable does too many things. It can calculate position of
>> something, draw something on graphics and work as a transferrable.
>> Calculation and drawing are parts of the test not transferable (what
>> is more, we calculate position for a panel that does not have direct
>> relation to the transferred data).
>> On drop I would prefer to use frame disposal instead of System.exit()
>> 67 System.exit(0);
>> The test could be implemented with Clipboard API only. On the other
>> hand, it could be implemented in a two different processes to verify
>> that the data is passed between JVMs.
>> If the current implementation works well on Windows and MacOS X
>> platforms, I am ok with the webrev.
>> Thank you,
>> Denis.
>>
>>
>> On Feb 1, 2013, at 5:59 PM, mikhail cherkasov wrote:
>>
>>> Hello Denis, All,
>>>
>>> There's new version
>>> http://cr.openjdk.java.net/~mcherkas/8005932/webrev.01/
>>> <http://cr.openjdk.java.net/%7Emcherkas/8005932/webrev.01/>
>>> <http://cr.openjdk.java.net/%7Emcherkas/8005932/webrev.01/>
>>> I exclude solaris's changes and also I've rewritten test.
>>>
>>> Thanks,
>>> Mikhail.
>>>
>>> 30.01.2013 18:13, Denis S. Fokin пишет:
>>>> Hi Mikhail,
>>>>
>>>> the fix for macosx looks ok. I would run jtreg and jck dnd and
>>>> datatransfer tests with the change.
>>>>
>>>> Have you gotten any benefits from the solaris change?
>>>>
>>>> Thank you,
>>>> Denis.
>>>>
>>>> On 1/29/2013 6:08 PM, mikhail cherkasov wrote:
>>>>> Hello again,
>>>>>
>>>>> This bug is still waiting for review.
>>>>>
>>>>>
>>>>> Thanks,
>>>>> Mikhail.
>>>>>
>>>>> 25.01.2013 19:41, mikhail cherkasov пишет:
>>>>>> Hello All,
>>>>>>
>>>>>> Please review the fix for the following bug:
>>>>>> http://bugs.sun.com/view_bug.do?bug_id=8005932
>>>>>> Fix:
>>>>>> http://cr.openjdk.java.net/~mcherkas/8005932/webrev.00/
>>>>>> <http://cr.openjdk.java.net/%7Emcherkas/8005932/webrev.00/>
>>>>>> <http://cr.openjdk.java.net/%7Emcherkas/8005932/webrev.00/>
>>>>>>
>>>>>> There's no code changes, I've just added missed mime types to macosx
>>>>>> and linux flavormap.properties.
>>>>>>
>>>>>> Thanks,
>>>>>> Mikhail.
>>>>>>
>>>>>
>>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20130208/cad8c66b/attachment.html
More information about the awt-dev
mailing list