Fetching of changelist for a push notification not executed while GET_REMAINING_FILE_LIST running |
|||||||||
Issue descriptionChrome Version: 64.0.3278.0 Steps To Reproduce: This is a hypothesis that is not yet confirmed with other accounts than my account. (1) Use an account that has many files. (2) Open about://drive-internals and click "Clear local data". (3) Scroll down and confirm TYPE_GET_REMAINING_FILE_LIST logs are output continuously in the Event Log section. (4) Open the Files app. (5) Open drive.google.com and view the same folder to (4). (6) Create a new directory in drive.google.com. Expected Result: The new folder appears in the Files app window in a few seconds. Actual Result: The new folder doesn't appear in the Files app. If you change to another folder and go back to the original folder in the Files app., the new folder will appear (as a result of the fast fetch).
,
Nov 27 2017
,
Nov 27 2017
Here is the logic that postpones change fetch. https://cs.chromium.org/chromium/src/components/drive/chromeos/change_list_loader.cc?type=cs&q=change_list_loader.cc+IsRefreshing&sq=package:chromium&l=385 I think this is by design, but we'd need to change the design now because a change list fetch can be a very huge task like after installing an account to a new device.
,
Nov 27 2017
Do you have any concrete idea on how to implement that? Just loading and applying the change from rev.X to rev.Y on the fly while the large loading of X doesn't sound good to me. (Unless we really carefully implement, the changelist of X will later overwrite Y. Or, when X is not fully available, the diff of X->Y may not even be possible to apply.) I'm wondering if it might be easier to just ensure some mechanism to cause reload in the Files app UI for this scenario. (Then the fast-fetch will do all the job.)
,
Nov 27 2017
By the way, this bug reminded me of Bug 307868 , which will require a different solution but at least from users' point of view it may look similar.
,
Nov 28 2017
Deletion of files are not reflected even after refreshing the view by changing directories in the app. This would be a separate issue.
,
Feb 20 2018
,
Feb 22 2018
,
Feb 28 2018
,
Feb 28 2018
,
Mar 28 2018
Postponing pending drive update.
,
Sep 27
Closing as this issue is not applicable to the new backend. |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by yamaguchi@chromium.org
, Nov 27 2017