Get rid of fullPath in the metadata cache.

This patch removes the last instance of fullPath from the metadata cache, by using volume manager and its getLocationInformation method.

TEST=Tested manually.
BUG=320967

Review URL: https://codereview.chromium.org/130663002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@243812 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/resources/file_manager/foreground/js/file_manager.js

@@ -659,7 +659,7 @@ var BOTTOM_MARGIN_FOR_PREVIEW_PANEL_PX = 52;
          DialogType.FULL_PAGE]);
 
     // Create the metadata cache.
-    this.metadataCache_ = MetadataCache.createFull();
+    this.metadataCache_ = MetadataCache.createFull(this.volumeManager_);
 
     // Create the root view of FileManager.
     this.ui_ = new FileManagerUI(this.dialogDom_, this.dialogType);

chrome/browser/resources/file_manager/foreground/js/media/audio_player.js

@@ -12,12 +12,12 @@
  */
 function AudioPlayer(container) {
   this.container_ = container;
-  this.metadataCache_ = MetadataCache.createFull();
   this.currentTrack_ = -1;
   this.playlistGeneration_ = 0;
   this.selectedEntry_ = null;
   this.volumeManager_ = new VolumeManagerWrapper(
       VolumeManagerWrapper.DriveEnabledStatus.DRIVE_ENABLED);
+  this.metadataCache_ = MetadataCache.createFull(this.volumeManager_);
 
   this.container_.classList.add('collapsed');
 

chrome/browser/resources/file_manager/foreground/js/media/video_player.js

@@ -141,11 +141,11 @@ function loadVideoPlayer() {
         document.querySelector('#video-container'),
         document.querySelector('#controls'));
 
-    metadataCache = MetadataCache.createFull();
     volumeManager = new VolumeManagerWrapper(
         VolumeManagerWrapper.DriveEnabledStatus.DRIVE_ENABLED);
     volumeManager.addEventListener('externally-unmounted',
                                    onExternallyUnmounted);
+    metadataCache = MetadataCache.createFull(volumeManager);
 
     // If the video player is starting before the first instance of the File
     // Manager then it does not have access to filesystem URLs.

chrome/browser/resources/file_manager/foreground/js/metadata/metadata_cache.js

@@ -108,12 +108,13 @@ MetadataCache.DESCENDANTS = 2;
 MetadataCache.EVICTION_NUMBER = 1000;
 
 /**
+ * @param {VolumeManagerWrapper} volumeManager Volume manager instance.
  * @return {MetadataCache!} The cache with all providers.
  */
-MetadataCache.createFull = function() {
+MetadataCache.createFull = function(volumeManager) {
   var cache = new MetadataCache();
   cache.providers_.push(new FilesystemProvider());
-  cache.providers_.push(new DriveProvider());
+  cache.providers_.push(new DriveProvider(volumeManager));
   cache.providers_.push(new ContentProvider());
   return cache;
 };
@@ -671,11 +672,18 @@ FilesystemProvider.prototype.fetch = function(
  *     drive: { pinned, hosted, present, customIconUrl, etc. }
  *     thumbnail: { url, transform }
  *     streaming: { }
+ * @param {VolumeManagerWrapper} volumeManager Volume manager instance.
  * @constructor
  */
-function DriveProvider() {
+function DriveProvider(volumeManager) {
   MetadataProvider.call(this);
 
+  /**
+   * @type {VolumeManagerWrapper}
+   * @private
+   */
+  this.volumeManager_ = volumeManager;
+
   // We batch metadata fetches into single API call.
   this.entries_ = [];
   this.callbacks_ = [];
@@ -693,8 +701,8 @@ DriveProvider.prototype = {
  * @return {boolean} Whether this provider supports the entry.
  */
 DriveProvider.prototype.supportsEntry = function(entry) {
-  // TODO(mtomasz): Use Entry instead of paths.
-  return PathUtil.isDriveBasedPath(entry.fullPath);
+  var locationInfo = this.volumeManager_.getLocationInfo(entry);
+  return locationInfo && locationInfo.isDriveBased;
 };
 
 /**