<AWT Dev> [10] JDK-8190515: java.awt.Desktop.moveToTrash(File) prompts on Windows 7 but not on Mac

[Shashidhara Veerabhadraiah]

Thank you Sergey and Phil for the review.

 

Thanks and regards,

Shashi

 

From: Phil Race 
Sent: Monday, December 11, 2017 10:22 PM
To: Shashidhara Veerabhadraiah <shashidhara.veerabhadraiah at oracle.com>; Sergey Bylokhov <sergey.bylokhov at oracle.com>
Cc: awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> [10] JDK-8190515: java.awt.Desktop.moveToTrash(File) prompts on Windows 7 but not on Mac

 

Ok. I think I was reading things a bit backwards. Looks fine.

-phil.

On 12/11/2017 07:55 AM, Shashidhara Veerabhadraiah wrote:

Hi Phil, Below are my replies below:

 

shashi

 

From: Philip Race 
Sent: Saturday, December 9, 2017 11:30 PM
To: Sergey Bylokhov HYPERLINK "mailto:sergey.bylokhov at oracle.com"<sergey.bylokhov at oracle.com>
Cc: Shashidhara Veerabhadraiah HYPERLINK "mailto:shashidhara.veerabhadraiah at oracle.com"<shashidhara.veerabhadraiah at oracle.com>; HYPERLINK "mailto:awt-dev at openjdk.java.net"awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> [10] JDK-8190515: java.awt.Desktop.moveToTrash(File) prompts on Windows 7 but not on Mac

 

I expect I must be misunderstanding something.
The bug seems to be saying we should eliminate  confirmation dialogs but the fix reads
like it does the opposite since it removes flag bits which suppress the dialogs :

-            fop.fFlags = FOF_ALLOWUNDO;
+            fop.fFlags = FOF_ALLOWUNDO | FOF_NOCONFIRMATION | FOF_NOERRORUI;
 
Does not "NOCONFIRMATION" mean "no confirmation dialog" ?
[Shashi] FOF_NOCONFIRMATION means no confirmation dialog will be popped up.
 
The bug says "does not happen on windows 10". Maybe there was a windows 7 bug ?
So after this fix, what happens on windows 7 and 10 ? What versions was the fix tested on ?
[Shashi] Before the fix, there is a pop up dialog on windows 7 and NO dialog pops up in the case of windows 10 for the same test.
After this fix, windows 7 pop up dialog gets suppressed behaving same as windows 10 now. I think probable reason  is that the implementation looks different between windows 7 and windows 10 for the file operations(SHFileOperation). With this change effective behavior is normalized across the different windows flavors.
 
-phil.



On 12/8/17, 5:40 PM, Sergey Bylokhov wrote: 

Looks fine. 

On 05/12/2017 23:23, Shashidhara Veerabhadraiah wrote: 




Hi All, Please review a fix for the below bug. 

Bug: https://bugs.openjdk.java.net/browse/JDK-8190515 

Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Esveerabhadra/8190515/webrev.00/"http://cr.openjdk.java.net/~sveerabhadra/8190515/webrev.00/ 

Summary: The file delete operation generates pop up dialog in windows7 and not in other higher than windows 7 versions. To keep it consistent across the different flavors, any dialog UI is being suppressed. 

Thanks and regards, 

Shashi 

 

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20171211/a7674055/attachment.html>