Limit using full paths in the directory model in the Files app.

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

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 'use strict';
 
 // If directory files changes too often, don't rescan directory more than once
 // per specified interval
 var SIMULTANEOUS_RESCAN_INTERVAL = 1000;
 // Used for operations that require almost instant rescan.
@@ skipping 175 matching lines @@
 };
 
 /**
  * @return {DirectoryEntry} Current directory.
  */
 DirectoryModel.prototype.getCurrentDirEntry = function() {
   return this.currentDirContents_.getDirectoryEntry();
 };
 
 /**
- * @return {Array.<string>} File paths of selected files.
+ * @return {Array.<Entry>} Array of selected entries..
  * @private
  */
-DirectoryModel.prototype.getSelectedPaths_ = function() {
+DirectoryModel.prototype.getSelectedEntries_ = function() {
   var indexes = this.fileListSelection_.selectedIndexes;
   var fileList = this.getFileList();
   if (fileList) {
     return indexes.map(function(i) {
-      return fileList.item(i).fullPath;
+      return fileList.item(i);
     });
   }
   return [];
 };
 
 /**
- * @param {Array.<string>} value List of file paths of selected files.
+ * @param {Array.<Entry>} value List of selected entries.
  * @private
  */
-DirectoryModel.prototype.setSelectedPaths_ = function(value) {
+DirectoryModel.prototype.setSelectedEntries_ = function(value) {
   var indexes = [];
   var fileList = this.getFileList();
-
-  var safeKey = function(key) {
-    // The transformation must:
-    // 1. Never generate a reserved name ('__proto__')
-    // 2. Keep different keys different.
-    return '#' + key;
-  };
-
-  var hash = {};
-
-  for (var i = 0; i < value.length; i++)
-    hash[safeKey(value[i])] = 1;
+  var urls = util.entriesToURLs(value);
 
   for (var i = 0; i < fileList.length; i++) {
-    if (hash.hasOwnProperty(safeKey(fileList.item(i).fullPath)))
+    if (urls.indexOf(fileList.item(i).toURL()))

[hirono, 2014/01/17 03:11:29]

!== -1 is needed?

[mtomasz, 2014/01/17 04:16:42]

Oops. Done.

       indexes.push(i);
   }
   this.fileListSelection_.selectedIndexes = indexes;
 };
 
 /**
- * @return {string} Lead item file path.
+ * @return {Entry} Lead entry.
  * @private
  */
-DirectoryModel.prototype.getLeadPath_ = function() {
+DirectoryModel.prototype.getLeadEntry_ = function() {
   var index = this.fileListSelection_.leadIndex;
-  return index >= 0 && this.getFileList().item(index).fullPath;
+  return index >= 0 && this.getFileList().item(index);
 };
 
 /**
- * @param {string} value The name of new lead index.
+ * @param {Entry} value The new lead entry.
  * @private
  */
-DirectoryModel.prototype.setLeadPath_ = function(value) {
+DirectoryModel.prototype.setLeadEntry_ = function(value) {
   var fileList = this.getFileList();
   for (var i = 0; i < fileList.length; i++) {
-    if (fileList.item(i).fullPath === value) {
+    if (util.isSameEntry(fileList.item(i), value)) {
       this.fileListSelection_.leadIndex = i;
       return;
     }
   }
 };
 
 /**
  * Schedule rescan with short delay.
  */
 DirectoryModel.prototype.rescanSoon = function() {
@@ skipping 188 matching lines @@
   dirContents.scan();
 };
 
 /**
  * @param {DirectoryContents} dirContents DirectoryContents instance.
  * @private
  */
 DirectoryModel.prototype.replaceDirectoryContents_ = function(dirContents) {
   cr.dispatchSimpleEvent(this, 'begin-update-files');
   this.updateSelectionAndPublishEvent_(this.fileListSelection_, function() {
-    var selectedPaths = this.getSelectedPaths_();
+    var selectedEntries = this.getSelectedEntries_();
     var selectedIndices = this.fileListSelection_.selectedIndexes;
 
     // Restore leadIndex in case leadName no longer exists.
     var leadIndex = this.fileListSelection_.leadIndex;
-    var leadPath = this.getLeadPath_();
+    var leadEntry = this.getLeadEntry_();
 
     this.currentDirContents_ = dirContents;
     dirContents.replaceContextFileList();
 
-    this.setSelectedPaths_(selectedPaths);
+    this.setSelectedEntries_(selectedEntries);
     this.fileListSelection_.leadIndex = leadIndex;
-    this.setLeadPath_(leadPath);
+    this.setLeadEntry_(leadEntry);
 
     // If nothing is selected after update, then select file next to the
     // latest selection
     var forceChangeEvent = false;
     if (this.fileListSelection_.selectedIndexes.length == 0 &&
         selectedIndices.length != 0) {
       var maxIdx = Math.max.apply(null, selectedIndices);
       this.selectIndex(Math.min(maxIdx - selectedIndices.length + 2,
                                 this.getFileList().length) - 1);
       forceChangeEvent = true;
@@ skipping 227 matching lines @@
         if (entry.isFile) {
           onError(util.createFileError(FileError.TYPE_MISMATCH_ERR));
           return;
         }
         successCallback(entry);
       },
       onError);
 };
 
 /**
- * @param {DirectoryEntry} dirEntry The absolute path to the new directory.
+ * @param {DirectoryEntry} dirEntry The entry of the new directory.
  * @param {function()=} opt_callback Executed if the directory loads
  *     successfully.
  * @private
  */
 DirectoryModel.prototype.changeDirectoryEntrySilent_ = function(dirEntry,
                                                                 opt_callback) {
   var onScanComplete = function() {
     if (opt_callback)
       opt_callback();
     // For tests that open the dialog to empty directories, everything
     // is loaded at this point.
     chrome.test.sendMessage('directory-change-complete');
   };
   this.clearAndScan_(
       DirectoryContents.createForDirectory(this.currentFileListContext_,
                                            dirEntry),
       onScanComplete.bind(this));
 };
 
 /**
  * Change the current directory to the directory represented by a
  * DirectoryEntry.
  *
  * Dispatches the 'directory-changed' event when the directory is successfully
  * changed.
  *
- * @param {DirectoryEntry} dirEntry The absolute path to the new directory.
+ * @param {DirectoryEntry} dirEntry The entry of the new directory.
  * @param {function()=} opt_callback Executed if the directory loads
  *     successfully.
  */
 DirectoryModel.prototype.changeDirectoryEntry = function(
     dirEntry, opt_callback) {
   this.fileWatcher_.changeWatchedDirectory(dirEntry, function(sequence) {
     if (this.changeDirectorySequence_ !== sequence)
       return;
     var previous = this.currentDirContents_.getDirectoryEntry();
     this.clearSearch_();
@@ skipping 294 matching lines @@
   if (this.onSearchCompleted_) {
     this.removeEventListener('scan-completed', this.onSearchCompleted_);
     this.onSearchCompleted_ = null;
   }
 
   if (this.onClearSearch_) {
     this.onClearSearch_();
     this.onClearSearch_ = null;
   }
 };