<AWT Dev> Review request: 8005932 : Java 7 on mac os x only provides text clipboard formats

[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/>
> 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/>
>>>> 
>>>> 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/20130201/65e8208b/attachment.html