Remove fullPath from util.isSameEntry().

Full paths will not work after migrating to the separate file systems per volume. Therefore, for comparing urls should be used instead.

TEST=Tested manually, partly by browser_tests.
BUG=320967

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

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

chrome/browser/resources/file_manager/background/js/volume_manager.js

@@ -42,25 +42,29 @@ function VolumeInfo(
       fullPath: RootDirectory.DRIVE + '/' + DriveSubRootDirectory.ROOT,
       isDirectory: true,
       rootType: RootType.DRIVE,
-      label: PathUtil.getRootLabel(RootType.DRIVE)
+      label: PathUtil.getRootLabel(RootType.DRIVE),
+      toURL: function() { return 'fake-entry://' + this.fullPath; }
     };
     this.fakeEntries[RootType.DRIVE_OFFLINE] = {
       fullPath: RootDirectory.DRIVE_OFFLINE,
       isDirectory: true,
       rootType: RootType.DRIVE_OFFLINE,
-      label: PathUtil.getRootLabel(RootType.DRIVE_OFFLINE)
+      label: PathUtil.getRootLabel(RootType.DRIVE_OFFLINE),
+      toURL: function() { return 'fake-entry://' + this.fullPath; }
     };
     this.fakeEntries[RootType.DRIVE_SHARED_WITH_ME] = {
       fullPath: RootDirectory.DRIVE_SHARED_WITH_ME,
       isDirectory: true,
       rootType: RootType.DRIVE_SHARED_WITH_ME,
-      label: PathUtil.getRootLabel(RootType.DRIVE_SHARED_WITH_ME)
+      label: PathUtil.getRootLabel(RootType.DRIVE_SHARED_WITH_ME),
+      toURL: function() { return 'fake-entry://' + this.fullPath; }
     };
     this.fakeEntries[RootType.DRIVE_RECENT] = {
       fullPath: RootDirectory.DRIVE_RECENT,
       isDirectory: true,
       rootType: RootType.DRIVE_RECENT,
-      label: PathUtil.getRootLabel(RootType.DRIVE_RECENT)
+      label: PathUtil.getRootLabel(RootType.DRIVE_RECENT),
+      toURL: function() { return 'fake-entry://' + this.fullPath; }
     };
   }
 

chrome/browser/resources/file_manager/common/js/util.js

@@ -1106,11 +1106,14 @@ util.createFileError = function(code) {
 util.isSameEntry = function(entry1, entry2) {
   // Currently, we can assume there is only one root.
   // When we support multi-file system, we need to look at filesystem, too.
-  return (entry1 && entry2 && entry1.fullPath === entry2.fullPath) ||
+  return (entry1 && entry2 && entry1.toURL() === entry2.toURL()) ||
       (!entry1 && !entry2);
 };
 
 /**
+ * TODO(mtomasz, yoshiki): Deprecated. Only used in directory_tree.js. Will
+ *     be removed soon.
+ *
  * @param {Entry|Object} parent The parent entry. Can be a fake.
  * @param {Entry|Object} child The child entry. Can be a fake.
  * @return {boolean} True if parent entry is actualy the parent of the child