Issue metadata
Sign in to add a comment
|
Files>Drive: Sharing a folder is broken |
||||||||||||||||||||||
Issue descriptionChrome Version: Chrome OS 9765.71.0, 61.0.3163.103 beta caroline Steps To Reproduce: 1. Login with User A and go to Gdrive 2. Share a folder containing some files with User B ( using context menu share with others option) OR 3. Goto web drive and share a folder with User B. Expected Result: User B should have the entire folder with the files shared. Actual Result: 1. Web drive User B has the entire shared folder or sometimes the shared folder is displayed empty. 2. Files> Shared with me has the shared folder as empty.
,
Sep 29 2017
fukino@ - Would you be able to take a look at this one?
,
Oct 5 2017
,
Oct 18 2017
,
Dec 5 2017
,
Jan 18 2018
,
Jan 31 2018
,
Feb 5 2018
I examined and found the issue in Comment #1 is an independent bug. Filed Issue 808937 .
,
Feb 16 2018
<files-triage>
,
Feb 16 2018
Oops - changing back to Started.
,
Feb 20 2018
Hi mkarkada@, Is the issue described in the bug description (not in comment #) still reproducible? I was not able to reproduce it on ToT. Do you remember the sizes of files under shared directory? If they are big, uploading them can take a while, it can cause the delay of displaying files in user B.
,
Feb 20 2018
Hi fukino, The issue in the bug description is not reproducible where the shared folder shows empty. But, it does happen that few shared files goes missing from the shared directory and some files/folder gets exposed outside the shared directory. Referring the issue video attached in this comment: Shared folder named 'Audio' should contain 10 files in it but only 8 files appear in this shared folder. Meaning 2 files are missing. The size of entire shared folder was around 4.6 GB and size of 'Audio' folder was around 12 MB. Issue video: https://pantheon.corp.google.com/storage/browser/chromiumos-test-logs/bugfiles/cr/769015/10421.0.0-66.0.3344.0/ To add on, the issue in comment #1 is 100% reproducible. Tested on CrOS version: 10421.0.0, 66.0.3344.0 cave and caroline devices.
,
Feb 28 2018
,
Mar 9 2018
Moving to M-67, please update if incorrect.
,
Mar 9 2018
Fukino-san - as this is marked P-1 for M-67, do you think you'll be able to finish this in time? Are you still working on this?
,
Mar 28 2018
The issue in the bug description is not reproducible according to comment #12. The issue described in #12 (Missing entries in shared folder) can happen when the shared folder is bug and syncing files take time. I'll focus on the issue described in #1: "Chrome Syncable FileSystem" is displayed in "Shared with me" folder. I was able to reproduce the issue, and working on a fix for it.
,
Mar 28 2018
WIP CL for the issue in c#1: https://crrev.com/c/984832
,
Mar 29 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d547564a99138d3861afef3e6540f0733efef728 commit d547564a99138d3861afef3e6540f0733efef728 Author: Naoki Fukino <fukino@chromium.org> Date: Thu Mar 29 01:08:27 2018 Files app: Go to parent folder on BACKSPACE based on the current breadcrumbs. On BACKSPACE, users should be navigated to the parent folder of the current one. We have used Entry.getParent() to identify the parent folder, but it is not safe when the current folder is under fake directories. As we already have the current folder's ascendant folders as breadcrumbs, using it to identify parent folders is safe for consistency between UI and shortcut-key handling. Bug: 769015 Test: Manually tested Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Id3af343dd12921fb14d37f2bed6ce146a478987a Reviewed-on: https://chromium-review.googlesource.com/984832 Reviewed-by: Sasha Morrissey <sashab@chromium.org> Commit-Queue: Naoki Fukino <fukino@chromium.org> Cr-Commit-Position: refs/heads/master@{#546687} [modify] https://crrev.com/d547564a99138d3861afef3e6540f0733efef728/ui/file_manager/file_manager/foreground/js/main_window_component.js [modify] https://crrev.com/d547564a99138d3861afef3e6540f0733efef728/ui/file_manager/file_manager/foreground/js/ui/location_line.js
,
Mar 29 2018
Marking as Fixed. Please verify if the issue described in comment #1 is fixed. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by mkarkada@chromium.org
, Sep 26 2017