Commit 319562c8 authored by yawano's avatar yawano Committed by Commit bot

Fix Closure Compiler error which is introduced by crrev.com/1233913010

BUG=509483
TEST=GYP_GENERATORS=ninja tools/gyp/gyp --depth . ui/file_manager/file_manager/foreground/js/compiled_resources.gyp && ninja -C out/Default

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

Cr-Commit-Position: refs/heads/master@{#339847}
parent f5fef066
...@@ -657,75 +657,89 @@ CommandHandler.COMMANDS_['drive-hosted-settings'] = /** @type {Command} */ ({ ...@@ -657,75 +657,89 @@ CommandHandler.COMMANDS_['drive-hosted-settings'] = /** @type {Command} */ ({
* Deletes selected files. * Deletes selected files.
* @type {Command} * @type {Command}
*/ */
CommandHandler.COMMANDS_['delete'] = /** @type {Command} */ ({ CommandHandler.COMMANDS_['delete'] = (function() {
/** /**
* @param {!Event} event Command event. * @constructor
* @param {!FileManager} fileManager FileManager to use. * @implements {Command}
*/ */
execute: function(event, fileManager) { var DeleteCommand = function() {};
var entries = CommandUtil.getCommandEntries(event.target);
// Execute might be called without a call of canExecute method, e.g. called DeleteCommand.prototype = {
// directly from code. Double check here not to delete undeletable entries. /**
if (this.containsFakeOrRootEntry_(entries, fileManager) || * @param {!Event} event Command event.
this.containsReadOnlyEntry_(entries, fileManager)) * @param {!FileManager} fileManager FileManager to use.
return; */
execute: function(event, fileManager) {
var entries = CommandUtil.getCommandEntries(event.target);
// Execute might be called without a call of canExecute method,
// e.g. called directly from code. Double check here not to delete
// undeletable entries.
if (this.containsFakeOrRootEntry_(entries, fileManager) ||
this.containsReadOnlyEntry_(entries, fileManager))
return;
var message = entries.length === 1 ?
strf('GALLERY_CONFIRM_DELETE_ONE', entries[0].name) :
strf('GALLERY_CONFIRM_DELETE_SOME', entries.length);
fileManager.ui.deleteConfirmDialog.show(message, function() {
fileManager.fileOperationManager.deleteEntries(entries);
}, null, null);
},
var message = entries.length === 1 ? /**
strf('GALLERY_CONFIRM_DELETE_ONE', entries[0].name) : * @param {!Event} event Command event.
strf('GALLERY_CONFIRM_DELETE_SOME', entries.length); * @param {!FileManager} fileManager FileManager to use.
*/
canExecute: function(event, fileManager) {
var entries = CommandUtil.getCommandEntries(event.target);
fileManager.ui.deleteConfirmDialog.show(message, function() { // If entries contain fake or root entry, hide delete option.
fileManager.fileOperationManager.deleteEntries(entries); if (this.containsFakeOrRootEntry_(entries, fileManager)) {
}, null, null); event.canExecute = false;
}, event.command.setHidden(true);
/** return;
* @param {!Event} event Command event. }
* @param {!FileManager} fileManager FileManager to use.
*/
canExecute: function(event, fileManager) {
var entries = CommandUtil.getCommandEntries(event.target);
// If entries contain fake or root entry, hide delete option. event.canExecute = entries.length > 0 &&
if (this.containsFakeOrRootEntry_(entries, fileManager)) { !this.containsReadOnlyEntry_(entries, fileManager);
event.canExecute = false; event.command.setHidden(false);
event.command.setHidden(true); },
return;
}
event.canExecute = entries.length > 0 && /**
!this.containsReadOnlyEntry_(entries, fileManager); * @param {!Array<!Entry>} entries
event.command.setHidden(false); * @param {!FileManager} fileManager
}, * @return {boolean} True if entries contain fake or root entry.
/** */
* @param {!Array<!Entry>} entries containsFakeOrRootEntry_: function(entries, fileManager) {
* @param {!FileManager} fileManager return entries.some(function(entry) {
* @return {boolean} True if entries contain fake or root entry. if (util.isFakeEntry(entry))
*/ return true;
containsFakeOrRootEntry_: function(entries, fileManager) {
return entries.some(function(entry) {
if (util.isFakeEntry(entry))
return true;
var volumeInfo = fileManager.volumeManager.getVolumeInfo(entry); var volumeInfo = fileManager.volumeManager.getVolumeInfo(entry);
if (!volumeInfo) if (!volumeInfo)
return true; return true;
return volumeInfo.displayRoot === entry; return volumeInfo.displayRoot === entry;
}); });
}, },
/**
* @param {!Array<!Entry>} entries /**
* @param {!FileManager} fileManager * @param {!Array<!Entry>} entries
* @return {boolean} True if entries contain read only entry. * @param {!FileManager} fileManager
*/ * @return {boolean} True if entries contain read only entry.
containsReadOnlyEntry_: function(entries, fileManager) { */
return entries.some(function(entry) { containsReadOnlyEntry_: function(entries, fileManager) {
var locationInfo = fileManager.volumeManager.getLocationInfo(entry); return entries.some(function(entry) {
return locationInfo && locationInfo.isReadOnly; var locationInfo = fileManager.volumeManager.getLocationInfo(entry);
}); return locationInfo && locationInfo.isReadOnly;
} });
}); }
};
return new DeleteCommand();
})();
/** /**
* Pastes files from clipboard. * Pastes files from clipboard.
...@@ -1394,8 +1408,6 @@ CommandHandler.COMMANDS_['configure'] = (function() { ...@@ -1394,8 +1408,6 @@ CommandHandler.COMMANDS_['configure'] = (function() {
}; };
ConfigureCommand.prototype = { ConfigureCommand.prototype = {
__proto__: Command.prototype,
/** /**
* @param {EventTarget} element * @param {EventTarget} element
* @param {!FileManager} fileManager * @param {!FileManager} fileManager
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment