Files.app: Clean DirectoryModel.

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 55 matching lines @@
 };
 
 /**
  * @return {cr.ui.ArrayDataModel} Files in the current directory.
  */
 DirectoryModel.prototype.getFileList = function() {
   return this.currentFileListContext_.fileList;
 };
 
 /**
- * Sort the file list.
- * @param {string} sortField Sort field.
- * @param {string} sortDirection "asc" or "desc".
- */
-DirectoryModel.prototype.sortFileList = function(sortField, sortDirection) {
-  this.getFileList().sort(sortField, sortDirection);
-};
-
-/**
  * @return {cr.ui.ListSelectionModel|cr.ui.ListSingleSelectionModel} Selection
  * in the fileList.
  */
 DirectoryModel.prototype.getFileListSelection = function() {
   return this.fileListSelection_;
 };
 
 /**
  * @return {?RootType} Root type of current root, or null if not found.
  */
@@ skipping 93 matching lines @@
 };
 
 /**
  * @return {DirectoryEntry} Current directory.
  */
 DirectoryModel.prototype.getCurrentDirEntry = function() {
   return this.currentDirContents_.getDirectoryEntry();
 };
 
 /**
- * @return {Array.<Entry>} Array of selected entries..
+ * @return {Array.<Entry>} Array of selected entries.
  * @private
  */
 DirectoryModel.prototype.getSelectedEntries_ = function() {
   var indexes = this.fileListSelection_.selectedIndexes;
   var fileList = this.getFileList();
   if (fileList) {
     return indexes.map(function(i) {
       return fileList.item(i);
     });
   }
@@ skipping 427 matching lines @@
           this.fileListSelection_.endChange();
           successCallback(newEntry);
         }
       }.bind(this), function(reason) {
         tracker.stop();
         errorCallback(reason);
       });
 };
 
 /**
- * @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 or a fake entry.
  *
  * Dispatches the 'directory-changed' event when the directory is successfully
  * changed.
  *
  * @param {DirectoryEntry|Object} dirEntry The entry of the new directory to
  *     be opened.
  * @param {function()=} opt_callback Executed if the directory loads
  *     successfully.
  */
 DirectoryModel.prototype.changeDirectoryEntry = function(
     dirEntry, opt_callback) {
-  if (util.isFakeEntry(dirEntry)) {
-    this.specialSearch(dirEntry);
-    if (opt_callback)
-      opt_callback();
-    return;
-  }
-
   // Increment the sequence value.
   this.changeDirectorySequence_++;
+  this.clearSearch_();
 
-  this.fileWatcher_.changeWatchedDirectory(dirEntry, function(sequence) {
-    if (this.changeDirectorySequence_ !== sequence)
-      return;
-    var previous = this.currentDirContents_.getDirectoryEntry();
-    this.clearSearch_();
-    this.changeDirectoryEntrySilent_(dirEntry, opt_callback);
+  var promise = new Promise(
+      function(onFulfilled, onRejected) {
+        this.fileWatcher_.changeWatchedDirectory(dirEntry, onFulfilled);
+      }.bind(this)).
 
-    var e = new Event('directory-changed');
-    e.previousDirEntry = previous;
-    e.newDirEntry = dirEntry;
-    this.dispatchEvent(e);
-  }.bind(this, this.changeDirectorySequence_));
+      then(function(sequence) {
+        return new Promise(function(onFulfilled, onRejected) {
+          if (this.changeDirectorySequence_ !== sequence)
+            return;
+
+          var newDirectoryContents = this.createDirectoryContents_(
+              this.currentFileListContext_, dirEntry, '');
+          if (!newDirectoryContents)
+            return;
+
+          var event = new Event('directory-changed');
+          event.previousDirEntry = this.currentDirContents_.getDirectoryEntry();
+          event.newDirEntry = dirEntry;
+
+          this.clearAndScan_(newDirectoryContents, opt_callback);
+          this.dispatchEvent(event);
+        }.bind(this));
+      }.bind(this, this.changeDirectorySequence_));
 };
 
 /**
  * Creates an object which could say whether directory has changed while it has
  * been active or not. Designed for long operations that should be cancelled
  * if the used change current directory.
  * @return {Object} Created object.
  */
 DirectoryModel.prototype.createDirectoryChangeTracker = function() {
   var tracker = {
@@ skipping 61 matching lines @@
 DirectoryModel.prototype.selectIndex = function(index) {
   // this.focusCurrentList_();
   if (index >= this.getFileList().length)
     return;
 
   // If a list bound with the model it will do scrollIndexIntoView(index).
   this.fileListSelection_.selectedIndex = index;
 };
 
 /**
- * Called when VolumeInfoList is updated.
+ * Handles update of VolumeInfoList.
  * @param {Event} event Event of VolumeInfoList's 'splice'.
  * @private
  */
 DirectoryModel.prototype.onVolumeInfoListUpdated_ = function(event) {
   // When the volume where we are is unmounted, fallback to the default volume's
   // root. If current directory path is empty, stop the fallback
   // since the current directory is initializing now.
   if (this.getCurrentDirEntry() &&
       !this.volumeManager_.getVolumeInfo(this.getCurrentDirEntry())) {
     this.volumeManager_.getDefaultDisplayRoot(function(displayRoot) {
       this.changeDirectoryEntry(displayRoot);
     }.bind(this));
   }
 };
 
 /**
+ * Creates directory contents for the entry and query.
+ *
+ * @param {FileListContext} context File list context.
+ * @param {DirectoryEntry} entry Current directory.
+ * @param {opt_query} opt_query Search query string.

[mtomasz, 2014/02/04 02:20:12]

nit: opt_query -> string=

[hirono, 2014/02/04 05:53:11]

Done.

+ * @return {DirectoryEntry} Directory contents.

[mtomasz, 2014/02/04 02:20:12]

nit: DirectoryEntry -> DirectoryContents

[hirono, 2014/02/04 05:53:11]

Done.

+ * @private
+ */
+DirectoryModel.prototype.createDirectoryContents_ =
+    function(context, entry, opt_query) {
+  var query = (opt_query || '').trimLeft();
+  var locationInfo = this.volumeManager_.getLocationInfo(entry);
+  if (!locationInfo)
+    return null;
+  var canUseDriveSearch = this.volumeManager_.getDriveConnectionState().type !==
+      util.DriveConnectionType.OFFLINE &&
+      locationInfo.isDriveBased;
+
+  if (query && canUseDriveSearch) {
+    // Drive search.
+    return DirectoryContents.createForDriveSearch(context, entry, query);
+  } else if (query) {
+    // Local search.
+    return DirectoryContents.createForLocalSearch(context, entry, query);
+  } if (util.isFakeEntry(entry)) {

[mtomasz, 2014/02/04 02:20:12]

nit: isFakeEntry -> locationInfo.isSpecialSearchRoot

[hirono, 2014/02/04 05:53:11]

Done.

+    // Drive special search.
+    var searchType;
+    switch (locationInfo.rootType) {
+      case RootType.DRIVE_OFFLINE:
+        searchType =
+            DriveMetadataSearchContentScanner.SearchType.SEARCH_OFFLINE;
+        break;
+      case RootType.DRIVE_SHARED_WITH_ME:
+        searchType =
+            DriveMetadataSearchContentScanner.SearchType.SEARCH_SHARED_WITH_ME;
+        break;
+      case RootType.DRIVE_RECENT:
+        searchType =
+            DriveMetadataSearchContentScanner.SearchType.SEARCH_RECENT_FILES;
+        break;
+      default:
+        // Unknown special search entry.
+        throw new Error('Unknown special search type.');
+    }
+    return DirectoryContents.createForDriveMetadataSearch(
+        context,
+        entry,
+        searchType);
+  } else {
+    // Local fetch or search.
+    return DirectoryContents.createForDirectory(context, entry);
+  }
+};
+
+/**
  * Performs search and displays results. The search type is dependent on the
  * current directory. If we are currently on drive, server side content search
  * over drive mount point. If the current directory is not on the drive, file
  * name search over current directory will be performed.
  *
  * @param {string} query Query that will be searched for.
  * @param {function(Event)} onSearchRescan Function that will be called when the
  *     search directory is rescanned (i.e. search results are displayed).
  * @param {function()} onClearSearch Function to be called when search state
  *     gets cleared.
  * TODO(olege): Change callbacks to events.
  */
 DirectoryModel.prototype.search = function(query,
                                            onSearchRescan,
                                            onClearSearch) {
-  query = query.trimLeft();
-
   this.clearSearch_();
-
   var currentDirEntry = this.getCurrentDirEntry();
   if (!currentDirEntry) {
     // Not yet initialized. Do nothing.
     return;
   }
 
-  if (!query) {
+  if (!(query || '').trimLeft()) {
     if (this.isSearching()) {
       var newDirContents = DirectoryContents.createForDirectory(
           this.currentFileListContext_,
-          this.currentDirContents_.getLastNonSearchDirectoryEntry());
+          currentDirEntry);
       this.clearAndScan_(newDirContents);
     }
     return;
   }
 
+  var newDirContents = this.createDirectoryContents_(
+      this.currentFileListContext_, currentDirEntry, query);
+  if (!newDirContents)
+    return;
+
   this.onSearchCompleted_ = onSearchRescan;
   this.onClearSearch_ = onClearSearch;
-
   this.addEventListener('scan-completed', this.onSearchCompleted_);
-
-  // If we are offline, let's fallback to file name search inside dir.
-  // A search initiated from directories in Drive or special search results
-  // should trigger Drive search.
-  var newDirContents;
-  var isDriveOffline = this.volumeManager_.getDriveConnectionState().type ===
-      util.DriveConnectionType.OFFLINE;
-  var locationInfo = this.volumeManager_.getLocationInfo(currentDirEntry);
-  if (!isDriveOffline && locationInfo && locationInfo.isDriveBased) {
-    // Drive search is performed over the whole drive, so pass  drive root as
-    // |directoryEntry|.
-    newDirContents = DirectoryContents.createForDriveSearch(
-        this.currentFileListContext_,
-        currentDirEntry,
-        this.currentDirContents_.getLastNonSearchDirectoryEntry(),
-        query);
-  } else {
-    newDirContents = DirectoryContents.createForLocalSearch(
-        this.currentFileListContext_, currentDirEntry, query);
-  }
   this.clearAndScan_(newDirContents);
 };
 
 /**
- * Performs special search and displays results. e.g. Drive files available
- * offline, shared-with-me files, recently modified files.
- * @param {Object} fakeEntry Fake entry representing a special search.
- * @param {string=} opt_query Query string used for the search.
- */
-DirectoryModel.prototype.specialSearch = function(fakeEntry, opt_query) {
-  var query = opt_query || '';
-
-  // Increment the sequence value.
-  this.changeDirectorySequence_++;
-
-  this.clearSearch_();
-  this.onSearchCompleted_ = null;
-  this.onClearSearch_ = null;
-
-  // Obtains a volume information.
-  // TODO(hirono): Obtain the proper profile's volume information.
-  var volumeInfo = this.volumeManager_.getCurrentProfileVolumeInfo(
-      util.VolumeType.DRIVE);
-  if (!volumeInfo) {
-    // It seems that the volume is already unmounted or drive is disable.
-    return;
-  }
-
-  var onDriveDirectoryResolved = function(sequence, driveRoot) {
-    if (this.changeDirectorySequence_ !== sequence)
-      return;
-
-    var locationInfo = this.volumeManager_.getLocationInfo(fakeEntry);
-    if (!locationInfo)
-      return;
-
-    var searchOption;
-    switch (locationInfo.rootType) {
-      case RootType.DRIVE_OFFLINE:
-        searchOption =
-            DriveMetadataSearchContentScanner.SearchType.SEARCH_OFFLINE;
-        break;
-      case RootType.DRIVE_SHARED_WITH_ME:
-        searchOption =
-            DriveMetadataSearchContentScanner.SearchType.SEARCH_SHARED_WITH_ME;
-        break;
-      case RootType.DRIVE_RECENT:
-        searchOption =
-            DriveMetadataSearchContentScanner.SearchType.SEARCH_RECENT_FILES;
-        break;
-      default:
-        // Unknown special search entry.
-        throw new Error('Unknown special search type.');
-    }
-
-    var newDirContents = DirectoryContents.createForDriveMetadataSearch(
-        this.currentFileListContext_,
-        fakeEntry, driveRoot, query, searchOption);
-    var previous = this.currentDirContents_.getDirectoryEntry();
-    this.clearAndScan_(newDirContents);
-
-    var e = new Event('directory-changed');
-    e.previousDirEntry = previous;
-    e.newDirEntry = fakeEntry;
-    this.dispatchEvent(e);
-  }.bind(this, this.changeDirectorySequence_);
-
-  volumeInfo.resolveDisplayRoot(
-      onDriveDirectoryResolved /* success */, function() {} /* failed */);
-};
-
-/**
  * In case the search was active, remove listeners and send notifications on
  * its canceling.
  * @private
  */
 DirectoryModel.prototype.clearSearch_ = function() {
   if (!this.isSearching())
     return;
 
   if (this.onSearchCompleted_) {
     this.removeEventListener('scan-completed', this.onSearchCompleted_);
     this.onSearchCompleted_ = null;
   }
 
   if (this.onClearSearch_) {
     this.onClearSearch_();
     this.onClearSearch_ = null;
   }
 };