[filebrowser] Enable fileBrowserHandlers only for files, not directories.

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

 // Copyright (c) 2011 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.
 
 // Setting the src of an img to an empty string can crash the browser, so we
 // use an empty 1x1 gif instead.
 const EMPTY_IMAGE_URI = 'data:image/gif;base64,'
         + 'R0lGODlhAQABAPABAP///wAAACH5BAEKAAAALAAAAAABAAEAAAICRAEAOw%3D%3D';
 
 var g_slideshow_data = null;
@@ skipping 211 matching lines @@
   };
 
   const previewArt = {
     'audio': 'images/filetype_large_audio.png',
     'folder': 'images/filetype_large_folder.png',
     'unknown': 'images/filetype_large_generic.png',
     'video': 'images/filetype_large_video.png'
   };
 
   /**
+   * Regexp for archive files. Used to show mount-archive task.
+   */
+  const ARCHIVES_REGEXP = /.zip$/;
+
+  /**
    * Return a translated string.
    *
    * Wrapper function to make dealing with translated strings more concise.
    * Equivilant to localStrings.getString(id).
    *
    * @param {string} id The id of the string to return.
    * @return {string} The translated string.
    */
   function str(id) {
     return localStrings.getString(id);
@@ skipping 1283 matching lines @@
       }
 
       if (pendingFiles.length) {
         cacheEntrySize(pendingFiles.pop(), cacheNextFile);
       } else {
         self.dispatchEvent(new cr.Event('selection-summarized'));
       }
     };
 
     if (this.dialogType_ == FileManager.DialogType.FULL_PAGE) {
-      // Since unmount task cannot be defined in terms of file patterns,
-      // we manually include it here, if all selected items are mount points.
       this.taskButtons_.innerHTML = '';
-      chrome.fileBrowserPrivate.getFileTasks(
-          selection.urls,
-          this.onTasksFound_.bind(this,
-                                  this.shouldShowUnmount_(selection.urls)));
+      // Some internal tasks cannot be defined in terms of file patterns,
+      // so we pass selection to check for them manually.
+      if (selection.directoryCount == 0 && selection.fileCount > 0) {
+        // Only files, not directories, are supported for external tasks.
+        chrome.fileBrowserPrivate.getFileTasks(
+            selection.urls,
+            this.onTasksFound_.bind(this, selection));
+      } else {
+        // There may be internal tasks for directories.
+        this.onTasksFound_(selection, []);
+      }
     }
 
     cacheNextFile();
   };
 
   FileManager.prototype.onMetadataResult_ = function(fileURL, metadata) {
     var observers = this.metadataCache_[fileURL];
     if (!observers || !(observers instanceof Array)) {
       console.error('Missing or invalid metadata observers: ' + fileURL + ': ' +
                     observers);
@@ skipping 43 matching lines @@
     if (!(methodName in this)) {
       console.log('Unknown message from metadata reader: ' + data.verb, data);
       return;
     }
 
     this[methodName].apply(this, data.arguments);
   };
 
   /**
    * Callback called when tasks for selected files are determined.
-   * @param {boolean} unmount Whether unmount task should be included.
+   * @param {Object} selection Selection is passed here, since this.selection
+   *     can change before tasks were found, and we should be accurate.
    * @param {Array.<Task>} tasksList The tasks list.
    */
-  FileManager.prototype.onTasksFound_ = function(unmount, tasksList) {
-    if (unmount) {
-      tasksList.push({
-          taskId: this.getExtensionId_() + '|unmount-archive',
-          iconUrl: '',
-          title: ''
-      });
-    }
+  FileManager.prototype.onTasksFound_ = function(selection, tasksList) {
     for (var i = 0; i < tasksList.length; i++) {
       var task = tasksList[i];
 
       // Tweak images, titles of internal tasks.
       var task_parts = task.taskId.split('|');
       if (task_parts[0] == this.getExtensionId_()) {
         if (task_parts[1] == 'preview') {
           // TODO(serya): This hack needed until task.iconUrl get working
           //              (see GetFileTasksFileBrowserFunction::RunImpl).
           task.iconUrl =
               chrome.extension.getURL('images/icon_preview_16x16.png');
           task.title = str('PREVIEW_IMAGE');
         } else if (task_parts[1] == 'edit') {
           task.iconUrl =
               chrome.extension.getURL('images/icon_preview_16x16.png');
           task.title = 'Edit';
           if (!IMAGE_EDITOR_ENABLED) continue;  // Skip the button creation.
         } else if (task_parts[1] == 'play') {
           task.iconUrl =
               chrome.extension.getURL('images/icon_play_16x16.png');
           task.title = str('PLAY_MEDIA').replace("&", "");
         } else if (task_parts[1] == 'enqueue') {
           task.iconUrl =
               chrome.extension.getURL('images/icon_add_to_queue_16x16.png');
           task.title = str('ENQUEUE');
         } else if (task_parts[1] == 'mount-archive') {
           task.iconUrl =
               chrome.extension.getURL('images/icon_mount_archive_16x16.png');
           task.title = str('MOUNT_ARCHIVE');
-        } else if (task_parts[1] == 'unmount-archive') {
-          task.iconUrl =
-              chrome.extension.getURL('images/icon_unmount_archive_16x16.png');
-          task.title = str('UNMOUNT_ARCHIVE');
         }
       }
       this.renderTaskButton_(task);
     }
-    // This needs to be done in sparate function, as check requires
+    this.maybeRenderUnmountTask_(selection);
+    // This needs to be done in separate function, as check requires
     // asynchronous function calls.
-    this.maybeRenderFormattingTask_();
+    this.maybeRenderFormattingTask_(selection);
   };
 
   FileManager.prototype.renderTaskButton_ = function(task) {
     var button = this.document_.createElement('button');
     button.addEventListener('click', this.onTaskButtonClicked_.bind(this));
     button.className = 'task-button';
     button.task = task;
 
     var img = this.document_.createElement('img');
     img.src = task.iconUrl;
 
     button.appendChild(img);
     button.appendChild(this.document_.createTextNode(task.title));
 
     this.taskButtons_.appendChild(button);
   };
 
   /**
+   * Checks whether unmount task should be displayed and if the answer is
+   * affirmative renders it.
+   * @param {Object} selection Selected files object.
+   */
+  FileManager.prototype.maybeRenderUnmountTask_ = function(selection) {
+    for (var index = 0; index < selection.urls.length; ++index) {
+      // Each url should be a mount point.
+      var path = selection.urls[index];
+      if (!this.mountPoints_.hasOwnProperty(path) ||
+          this.mountPoints_[path].type != 'file')
+        return;
+    }
+    this.renderTaskButton_({
+        taskId: this.getExtensionId_() + '|unmount-archive',
+        iconUrl:
+            chrome.extension.getURL('images/icon_unmount_archive_16x16.png'),
+        title: str('UNMOUNT_ARCHIVE')
+    });
+   };
+
+  /**
    * Checks whether formatting task should be displayed and if the answer is
    * affirmative renders it. Includes asynchronous calls, so it's splitted into
    * three parts.
+   * @param {Object} selection Selected files object.
    */
-  FileManager.prototype.maybeRenderFormattingTask_ = function() {
-    // Not to make unnecesary getMountPoints() call we doublecheck if there is
+  FileManager.prototype.maybeRenderFormattingTask_ = function(selection) {
+    // Not to make unnecessary getMountPoints() call we doublecheck if there is
     // only one selected entry.
-    if (this.selection.entries.length != 1)
+    if (selection.entries.length != 1)
       return;
     var self = this;
     function onMountPointsFound(mountPoints) {
       self.mountPoints_ = mountPoints;
 
       function normalize(x) {
         if (x[0] == '/')
           return x.slice(1);
         else
           return x;
       }
 
       function onVolumeMetadataFound(volumeMetadata) {
         if (volumeMetadata.deviceType == "flash") {
-          if (self.selection.entries.length != 1 ||
-              normalize(self.selection.entries[0].fullPath) !=
+          if (selection.entries.length != 1 ||
+              normalize(selection.entries[0].fullPath) !=
               normalize(volumeMetadata.mountPath)) {
             return;
           }
           var task = {
             taskId: self.getExtensionId_() + '|format-device',
             iconUrl: chrome.extension.getURL('images/filetype_generic.png'),
             title: str('FORMAT_DEVICE')
           };
           self.renderTaskButton_(task);
         }
       }
 
-      if (self.selection.entries.length != 1)
+      if (selection.entries.length != 1)
         return;
-      var selectedPath = self.selection.entries[0].fullPath;
+      var selectedPath = selection.entries[0].fullPath;
       for (var i = 0; i < mountPoints.length; i++) {
         if (mountPoints[i].mountType == "device" &&
             normalize(mountPoints[i].mountPath) == normalize(selectedPath)) {
           chrome.fileBrowserPrivate.getVolumeMetadata(mountPoints[i].sourceUrl,
               onVolumeMetadataFound);
           return;
         }
       }
     }
 
@@ skipping 73 matching lines @@
       for (var index = 0; index < urls.length; ++index) {
         chrome.fileBrowserPrivate.removeMount(urls[index]);
       }
     } else if (id == 'format-device') {
       this.confirm.show(str('FORMATTING_WARNING'), function() {
         chrome.fileBrowserPrivate.formatDevice(urls[0]);
       });
     }
   };
 
-  /**
-   * Determines whether unmount task should present for selected files.
-   */
-  FileManager.prototype.shouldShowUnmount_ = function(urls) {
-    for (var index = 0; index < urls.length; ++index) {
-      // Each url should be a mount point.
-      var path = urls[index];
-      if (!this.mountPoints_.hasOwnProperty(path) ||
-          this.mountPoints_[path].type != 'file') {
-        return false;
-      }
-    }
-    return true;
-  };
-
   FileManager.prototype.openImageEditor_ = function(entry) {
     var self = this;
 
     var editorFrame = this.document_.createElement('iframe');
     editorFrame.className = 'overlay-pane';
     editorFrame.scrolling = 'no';
 
     editorFrame.onload = function() {
       self.cacheMetadata_(entry, function(metadata) {
         editorFrame.contentWindow.ImageEditor.open(
@@ skipping 1306 matching lines @@
 
     if (msg) {
       console.log('no no no');
       this.alert.show(msg, onAccept);
       return false;
     }
 
     return true;
   };
 })();