Files.app: Add subfolders in the left nav

chrome/browser/resources/file_manager/js/file_manager_commands.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.
 
 var CommandUtil = {};
 
 /**
  * Extracts root on which command event was dispatched.
  *
  * @param {Event} event Command event for which to retrieve root to operate on.
- * @param {cr.ui.List} rootsList Root list to extract root node.
- * @return {DirectoryEntry} Found root.
+ * @param {DirectoryTree} directoryTree Directory tree to extract root node.
+ * @return {?DirectoryEntry} Found root.

[mtomasz, 2013/03/14 02:52:05]

AFAIK objects do not need ? in jsdoc.

[yoshiki, 2013/03/14 07:57:53]

Oops, removed.

  */
-CommandUtil.getCommandRoot = function(event, rootsList) {
-  var result = rootsList.dataModel.item(
-                   rootsList.getIndexOfListItem(event.target)) ||
-               rootsList.selectedItem;
+CommandUtil.getCommandRoot = function(event, directoryTree) {
+  var entry = directoryTree.selectedItem;
 
-  return result;
+  if (entry && PathUtil.isRootPath(entry.fullPath))
+    return entry;
+  else
+    return null;
 };
 
 /**
  * @param {Event} event Command event for which to retrieve root type.
- * @param {cr.ui.List} rootsList Root list to extract root node.
- * @return {string} Found root.
+ * @param {DirectoryTree} directoryTree Directory tree to extract root node.
+ * @return {?string} Found root.
  */
-CommandUtil.getCommandRootType = function(event, rootsList) {
-  var root = CommandUtil.getCommandRoot(event, rootsList);
+CommandUtil.getCommandRootType = function(event, directoryTree) {
+  var root = CommandUtil.getCommandRoot(event, directoryTree);
 
   return root && PathUtil.getRootType(root.fullPath);
 };
 
 /**
  * Checks if command can be executed on drive.
  * @param {Event} event Command event to mark.
  * @param {FileManager} fileManager FileManager to use.
  */
 CommandUtil.canExecuteEnabledOnDriveOnly = function(event, fileManager) {
@@ skipping 82 matching lines @@
   },
   canExecute: function(event, document) {
     event.canExecute = document.queryCommandEnabled(event.command.id);
   }
 };
 
 /**
  * Unmounts external drive.
  */
 Commands.unmountCommand = {
-  execute: function(event, rootsList, fileManager) {
-    var root = CommandUtil.getCommandRoot(event, rootsList);
+  /**
+   * @param {Event} event Command event.
+   * @param {DirectoryTree} directoryTree Target directory tree.
+   */
+  execute: function(event, directoryTree, fileManager) {
+    var root = CommandUtil.getCommandRoot(event, directoryTree);
     if (root)
       fileManager.unmountVolume(PathUtil.getRootPath(root.fullPath));
   },
-  canExecute: function(event, rootsList) {
-    var rootType = CommandUtil.getCommandRootType(event, rootsList);
+  /**
+   * @param {Event} event Command event.
+   * @param {DirectoryTree} directoryTree Target directory tree.
+   */
+  canExecute: function(event, directoryTree) {
+    var rootType = CommandUtil.getCommandRootType(event, directoryTree);
 
-    event.canExecute = (rootType == RootType.ARCHIVE ||
-                        rootType == RootType.REMOVABLE);
+    event.canExecute = (rootType === RootType.ARCHIVE ||

[mtomasz, 2013/03/14 02:52:05]

Is this change necessary? RootType.* values are never 0/null/undefined, so ==
should be always sufficient.

[yoshiki, 2013/03/14 07:57:53]

Done.

+                        rootType === RootType.REMOVABLE);
     event.command.label = rootType == RootType.ARCHIVE ?
         str('CLOSE_ARCHIVE_BUTTON_LABEL') :
         str('UNMOUNT_DEVICE_BUTTON_LABEL');
   }
 };
 
 /**
  * Formats external drive.
  */
 Commands.formatCommand = {
-  execute: function(event, rootsList, fileManager) {
-    var root = CommandUtil.getCommandRoot(event, rootsList);
+  /**
+   * @param {Event} event Command event.
+   * @param {DirectoryTree} directoryTree Target directory tree.
+   */
+  execute: function(event, directoryTree, fileManager) {
+    var root = CommandUtil.getCommandRoot(event, directoryTree);
 
     if (root) {
       var url = util.makeFilesystemUrl(PathUtil.getRootPath(root.fullPath));
       fileManager.confirm.show(
           loadTimeData.getString('FORMATTING_WARNING'),
           chrome.fileBrowserPrivate.formatDevice.bind(null, url));
     }
   },
-  canExecute: function(event, rootsList, fileManager, directoryModel) {
-    var root = CommandUtil.getCommandRoot(event, rootsList);
+  /**
+   * @param {Event} event Command event.
+   * @param {DirectoryTree} directoryTree Target directory tree.
+   */
+  canExecute: function(event, directoryTree, fileManager, directoryModel) {
+    var root = CommandUtil.getCommandRoot(event, directoryTree);
     var removable = root &&
                     PathUtil.getRootType(root.fullPath) == RootType.REMOVABLE;
     var isReadOnly = root && directoryModel.isPathReadOnly(root.fullPath);
     event.canExecute = removable && !isReadOnly;
     event.command.setHidden(!removable);
   }
 };
 
 /**
  * Imports photos from external drive
  */
 Commands.importCommand = {
-  execute: function(event, rootsList) {
-    var root = CommandUtil.getCommandRoot(event, rootsList);
+  /**
+   * @param {Event} event Command event.
+   * @param {DirectoryTree} directoryTree Target directory tree.
+   */
+  execute: function(event, directoryTree) {
+    var root = CommandUtil.getCommandRoot(event, directoryTree);
     if (!root)
       return;
 
     chrome.windows.getCurrent(undefined, function(window) {
       chrome.windows.create(
           { url: chrome.extension.getURL('photo_import.html') +
                  '?' + window.id + '#' + PathUtil.getRootPath(root.fullPath),
             type: 'popup' });
     }.bind(this));
   },
-  canExecute: function(event, rootsList) {
-    event.canExecute =
-        (CommandUtil.getCommandRootType(event, rootsList) != RootType.DRIVE);
+  /**
+   * @param {Event} event Command event.
+   * @param {DirectoryTree} directoryTree Target directory tree.
+   */
+  canExecute: function(event, directoryTree) {
+    var rootType = CommandUtil.getCommandRootType(event, directoryTree);
+    event.canExecute = (rootType != RootType.DRIVE);
   }
 };
 
 /**
  * Initiates new folder creation.
  */
 Commands.newFolderCommand = {
   execute: function(event, fileManager) {
     fileManager.createNewFolder();
   },
@@ skipping 203 matching lines @@
     fileManager.copyManager_.zipSelection(dirEntry, fileManager.isOnDrive(),
                                           selectionEntries);
   },
   canExecute: function(event, fileManager) {
     var selection = fileManager.getSelection();
     event.canExecute = !fileManager.isOnReadonlyDirectory() &&
         !fileManager.isOnDrive() &&
         selection && selection.totalCount > 0;
   }
 };