Notification prompts to format a disk even when the device is not writable |
||||||||
Issue descriptionChrome OS Version: ToT Steps To Reproduce: (1) Delete all the partition in an SD card (2) Slide the notch of the SD card to make it read-only (3) Insert the SD card into a device (4) See the "removable device detected" notification message on the right-bottom of the screen. Expected Result: The notification message tells the disk partitions were not recognized, and tell that it cannot be formatted as it's read-only. Actual Result: The notification message tells the disk partitions were not recognized, and shows a link to "format device". How frequently does this problem reproduce? (Always, sometimes, hard to reproduce?) Always What is the impact to the user, and is there a workaround? If so, what is it? If the user attempts to format the drive, the user will end up seeing an error. It does not give any clue why it fails. Note that user can only attempt to format it only when opened the drive directly by the link provided by the notification. If user opens the drive by clicking in the list in Files app, it will correctly gray out the option to format it. This is the origin issue of https://bugs.chromium.org/p/chromium/issues/detail?id=643475 .
,
Sep 5 2016
,
Sep 6 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5c3efcf062d49a2585fc3be33286e91118cd7efc commit 5c3efcf062d49a2585fc3be33286e91118cd7efc Author: yamaguchi <yamaguchi@chromium.org> Date: Tue Sep 06 02:16:46 2016 Hide "Format device" button in notification of unknown device when it is read only. BUG= 644043 Review-Url: https://codereview.chromium.org/2310783004 Cr-Commit-Position: refs/heads/master@{#416576} [modify] https://crrev.com/5c3efcf062d49a2585fc3be33286e91118cd7efc/ui/file_manager/file_manager/background/js/device_handler.js [modify] https://crrev.com/5c3efcf062d49a2585fc3be33286e91118cd7efc/ui/file_manager/file_manager/background/js/device_handler_unittest.js
,
Sep 8 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/01d0c324611df6ceb900aae7766bc15517eb9bca commit 01d0c324611df6ceb900aae7766bc15517eb9bca Author: yamaguchi <yamaguchi@chromium.org> Date: Thu Sep 08 05:44:44 2016 Hide "Format device" button in notification of unknown device when it is read only. BUG= 644043 , 643475 NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2310783004 Cr-Commit-Position: refs/heads/master@{#416576} (cherry picked from commit 5c3efcf062d49a2585fc3be33286e91118cd7efc) Review-Url: https://codereview.chromium.org/2319603002 Cr-Commit-Position: refs/branch-heads/2840@{#228} Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607} [modify] https://crrev.com/01d0c324611df6ceb900aae7766bc15517eb9bca/ui/file_manager/file_manager/background/js/device_handler.js [modify] https://crrev.com/01d0c324611df6ceb900aae7766bc15517eb9bca/ui/file_manager/file_manager/background/js/device_handler_unittest.js
,
Sep 8 2016
,
Oct 7 2016
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/01d0c324611df6ceb900aae7766bc15517eb9bca commit 01d0c324611df6ceb900aae7766bc15517eb9bca Author: yamaguchi <yamaguchi@chromium.org> Date: Thu Sep 08 05:44:44 2016 Hide "Format device" button in notification of unknown device when it is read only. BUG= 644043 , 643475 NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2310783004 Cr-Commit-Position: refs/heads/master@{#416576} (cherry picked from commit 5c3efcf062d49a2585fc3be33286e91118cd7efc) Review-Url: https://codereview.chromium.org/2319603002 Cr-Commit-Position: refs/branch-heads/2840@{#228} Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607} [modify] https://crrev.com/01d0c324611df6ceb900aae7766bc15517eb9bca/ui/file_manager/file_manager/background/js/device_handler.js [modify] https://crrev.com/01d0c324611df6ceb900aae7766bc15517eb9bca/ui/file_manager/file_manager/background/js/device_handler_unittest.js
,
Nov 19 2016
,
Jan 21 2017
,
Jan 25 2017
Test version 9202.4.0, 57.0.2987.8 |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by yamaguchi@chromium.org
, Sep 5 2016