Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(734)

Unified Diff: chrome/browser/resources/file_manager/js/volume_manager.js

Issue 13866009: Remove root resource id aliasing from DriveResourceMetadata. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove other resource id check. Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/chromeos/drive/stale_cache_files_remover_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/file_manager/js/volume_manager.js
diff --git a/chrome/browser/resources/file_manager/js/volume_manager.js b/chrome/browser/resources/file_manager/js/volume_manager.js
index f6306f39be36a8e70e6ec4bea2ec6c59c10861f7..76895526b0999a9bc1846cfdbae610ea53caf5d6 100644
--- a/chrome/browser/resources/file_manager/js/volume_manager.js
+++ b/chrome/browser/resources/file_manager/js/volume_manager.js
@@ -313,12 +313,16 @@ VolumeManager.prototype.waitDriveLoaded_ = function(mountPath, callback) {
chrome.fileBrowserPrivate.requestLocalFileSystem(function(filesystem) {
filesystem.root.getDirectory(mountPath, {},
function(entry) {
- // After introducion of the 'fast-fetch' feature, getting the root
- // entry does not start fetching data. Rather, it starts when the
- // entry is read.
- entry.createReader().readEntries(
- callback.bind(null, true),
- callback.bind(null, false));
+ // After file system is mounted, we need to "read" drive grand root
+ // entry at first. It loads mydrive root entry as a part of
+ // 'fast-fetch' quickly, and starts full feed fetch in parallel.
+ // Without this read, accessing mydrive root will be 'full-fetch'
+ // rather than 'fast-fetch' on the current architecture.
+ // Just "getting" the grand root entry doesn't trigger it. Rather,
+ // it starts when the entry is "read".
+ entry.createReader().readEntries(
+ callback.bind(null, true),
+ callback.bind(null, false));
},
callback.bind(null, false));
});
« no previous file with comments | « chrome/browser/chromeos/drive/stale_cache_files_remover_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698