<Swing Dev> [10] JDK-8178106: There is no error message pop up when clicking 'create folder' button

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Thu Jul 27 18:06:26 UTC 2017


I guess it will be better to notify the user that the permissions should 
be restored.

On 26.07.2017 23:27, Shashidhara Veerabhadraiah wrote:
> 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.sadhukhan at oracle.com>
> *Cc:* swing-dev at openjdk.java.net; Shashidhara Veerabhadraiah 
> <shashidhara.veerabhadraiah at 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.sadhukhan at oracle.com 
> <mailto:prasanta.sadhukhan at 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.veerabhadraiah at oracle.com>
>     <mailto:shashidhara.veerabhadraiah at oracle.com>;
>     swing-dev at openjdk.java.net <mailto:swing-dev at 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.veerabhadraiah at oracle.com>
>         <mailto:shashidhara.veerabhadraiah at oracle.com>;
>         swing-dev at openjdk.java.net <mailto:swing-dev at 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.veerabhadraiah at oracle.com>
>                 <mailto:shashidhara.veerabhadraiah at oracle.com>;
>                 swing-dev at openjdk.java.net
>                 <mailto:swing-dev at openjdk.java.net>
>                 > *Cc:* Philip Race <philip.race at oracle.com>
>                 <mailto:philip.race at oracle.com>; Sergey Bylokhov
>                 <sergey.bylokhov at oracle.com>
>                 <mailto:sergey.bylokhov at 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
> 
>> 
> 
>> 
> 


-- 
Best regards, Sergey.



More information about the swing-dev mailing list