Files app UI: Refresh view when archive file unmounted by errors |
||||||||||||||
Issue descriptionChrome Version: 66.0.3330.0 Steps To Reproduce: (1) Open a zip file. See the file is mounted and the contents is displayed. (2) Remove the zip file. (3) Go to the zip volume. (4) try to open a file/directory in it. Expected Result: The mounted zip volume should disappear from the left pane of all Files app windows after step 2. Actual Result: It only happens after step 4. How frequently does this problem reproduce? (Always, sometimes, hard to reproduce?) 100%
,
Feb 5 2018
,
Feb 8 2018
,
Feb 8 2018
When bug 810205 is fixed, the fix for this could be in file_operation_manager.js: In FileOperationManager.prototype.serviceDeleteTask_, add the following code or similar: var deletedFile = task.entries[0]; var volumeEntries = this.volumeManager_.volumeInfoList; for (var i = 0; i < volumeEntries.length; i++) { // Check if we deleted the archive file, or any ancestor directory. if (util.isSamePath(deletedFile.path, volumeEntries.entry(i).source_path) || util.isPathPrefixOf(volumeEntries.entry(i).source_path, deletedFile.path)) { // Unmount the volume here. } }
,
Feb 15 2018
,
Feb 15 2018
,
Feb 28 2018
,
Mar 1 2018
,
Apr 12 2018
,
Apr 18 2018
Issue 453273 has been merged into this issue.
,
May 24 2018
,
May 24 2018
Hi Noel, assigning to you as new owner of Zip, I've done some work on these so we can catch up about them if needed.
,
Aug 16
,
Oct 19
,
Dec 19
No, I just know how to spell zip, thanks for asking. amistry@ you wanna try one of these? See #4, not sure whether it's a useful hint or not these days. And send it back for re-assignment if you feel that way too.
,
Dec 19
It's tricky to fix with the FSP and so many layers involved. I think the repro steps are unlikely to happen in normal day-to-day usage. It's an edge case and I think this should be low priority. |
||||||||||||||
►
Sign in to add a comment |
||||||||||||||
Comment 1 by yamaguchi@chromium.org
, Jan 31 2018Labels: Hotlist-GoodFirstBug