Here is the new Webrev with the requested changes:
http://cr.openjdk.java.net/~pkbalakr/shashi/8178106/webrev.02/
Thanks and regards,
Shashi
*From:*Sergey Bylokhov
*Sent:* Wednesday, July 26, 2017 10:17 PM
*To:* Prasanta Sadhukhan <prasanta.sadhuk...@oracle.com>
*Cc:* swing-dev@openjdk.java.net; Shashidhara Veerabhadraiah
<shashidhara.veerabhadra...@oracle.com>
*Subject:* Re: <Swing Dev> [10] JDK-8178106: There is no error message
pop up when clicking 'create folder' button
I guess the user will need to set permissions back at the end of the
test? Otherwise jtreg will not be able to delete it(if the folder will
be created in the results).
----- prasanta.sadhuk...@oracle.com
<mailto:prasanta.sadhuk...@oracle.com> wrote:
>
looks fine.
>
Regards
> Prasanta
>
> On 7/26/2017 3:05 PM, Shashidhara Veerabhadraiah wrote:
>
>
Hi, Please find the updated Webrev @
http://cr.openjdk.java.net/~pkbalakr/shashi/8178106/webrev.01/
<http://cr.openjdk.java.net/%7Epkbalakr/shashi/8178106/webrev.01/>
Thanks and regards,
Shashi
>
*From:*Prasanta Sadhukhan
> *Sent:* Wednesday, July 26, 2017 1:52 PM
> *To:* Shashidhara Veerabhadraiah
<shashidhara.veerabhadra...@oracle.com>
<mailto:shashidhara.veerabhadra...@oracle.com>;
swing-dev@openjdk.java.net <mailto:swing-dev@openjdk.java.net>
> *Subject:* Re: <Swing Dev> [10] JDK-8178106: There is no error
message pop up when clicking 'create folder' button
ok. Probably, you can make the test instructions more clear citing
"by changing security permission through Security tab under
Folder->properties menu to deny write permissions to the newly
created folder.."
Regards
> Prasanta
On 7/26/2017 1:47 PM, Shashidhara Veerabhadraiah wrote:
Yes. Now this fix has been tested on window 7 and windows 8.1 as
well. The way to set the folder properties is same and the
expected error message pops up if we tried to create a new
folder under the read-only directory.
Thanks and regards,
Shashi
>
*From:*Prasanta Sadhukhan
> *Sent:* Monday, July 24, 2017 4:06 PM
> *To:* Shashidhara Veerabhadraiah
<shashidhara.veerabhadra...@oracle.com>
<mailto:shashidhara.veerabhadra...@oracle.com>;
swing-dev@openjdk.java.net <mailto:swing-dev@openjdk.java.net>
> *Subject:* Re: <Swing Dev> [10] JDK-8178106: There is no error
message pop up when clicking 'create folder' button
On 7/24/2017 4:04 PM, Prasanta Sadhukhan wrote:
ok. Then what about windows 7? Did you test there as well?
and windows 8.1 since the affected OS is 8.1.
>
>
>
Regards
> Prasanta
On 7/24/2017 3:56 PM, Shashidhara Veerabhadraiah wrote:
Hi Prasanta, I have verified this on Windows 10 itself.
Below is the output of the system info where it was tested:
OS Name:Microsoft Windows 10 Pro
OS Version:10.0.14393 N/A Build 14393
OS Manufacturer:Microsoft Corporation
OS Configuration:Member Workstation
OS Build Type:Multiprocessor Free
BIOS Version:LENOVO R06ET47W (1.21 ),
11/30/2016
Thanks and regards,
> Shashi
>
*From:*Prasanta Sadhukhan
> *Sent:* Monday, July 24, 2017 3:43 PM
> *To:* Shashidhara Veerabhadraiah
<shashidhara.veerabhadra...@oracle.com>
<mailto:shashidhara.veerabhadra...@oracle.com>;
swing-dev@openjdk.java.net
<mailto:swing-dev@openjdk.java.net>
> *Cc:* Philip Race <philip.r...@oracle.com>
<mailto:philip.r...@oracle.com>; Sergey Bylokhov
<sergey.bylok...@oracle.com>
<mailto:sergey.bylok...@oracle.com>
> *Subject:* Re: <Swing Dev> [10] JDK-8178106: There is no
error message pop up when clicking 'create folder'
button
Hi Shashi,
Which windows version you tested?
> Could you also verify with windows 10 (if not already
tested) if this way of making folder read-only is valid
or doable (if not, then the instructions may need to be
changed)?
>
> Regards
> Prasanta
On 7/24/2017 3:28 PM, Shashidhara Veerabhadraiah wrote:
Hi All,
Please review a fix for the JDK-8178106 where there
was no error message being popped up when creating a
new folder.
Issue: When creating a new folder under a read-only
set folder causes no pop up with respect to the error.
Resolution: The test instruction does not elaborate
on how to make the folder read-only and hence the
failure. Windows offers to set this via the security
option of the folder properties and deny the write
operation for the folder apart from the read-only
attribute being set while creating the folder
itself. The test instructions are modified to
indicate a way to set the read-only attribute of the
folder.
Test output: After setting folder permission
correctly, the below output being seen as expected.
Bug:
https://bugs.openjdk.java.net/browse/JDK-8178106
Webrev:
http://cr.openjdk.java.net/~aghaisas/shashi/8178106/webrev.00/
<http://cr.openjdk.java.net/%7Eaghaisas/shashi/8178106/webrev.00/>
Thanks and regards,
Shashi
