Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(783)

Unified Diff: ui/file_manager/file_manager/foreground/js/file_tasks.js

Issue 571453002: Correct indentation, JSDoc, etc... to comply with closure linter. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/file_manager/file_manager/foreground/js/file_tasks.js
diff --git a/ui/file_manager/file_manager/foreground/js/file_tasks.js b/ui/file_manager/file_manager/foreground/js/file_tasks.js
index 6f65fb59bf8811db934e149cd6275f795762d455..4d459d86627e53184d7744fc83b076f80ea1cf48 100644
--- a/ui/file_manager/file_manager/foreground/js/file_tasks.js
+++ b/ui/file_manager/file_manager/foreground/js/file_tasks.js
@@ -84,6 +84,7 @@ FileTasks.createWebStoreLink = function(extension, mimeType) {
* Complete the initialization.
*
* @param {Array.<Entry>} entries List of file entries.
+ * @param {Array.<string>=} opt_mimeTypes Mime-type specified for each entries.
*/
FileTasks.prototype.init = function(entries, opt_mimeTypes) {
this.entries_ = entries;
@@ -481,15 +482,15 @@ FileTasks.prototype.checkAvailability_ = function(callback) {
fm.alert.showHtml(
loadTimeData.getString('OFFLINE_HEADER'),
props[0].hosted ?
- loadTimeData.getStringF(
- entries.length === 1 ?
- 'HOSTED_OFFLINE_MESSAGE' :
- 'HOSTED_OFFLINE_MESSAGE_PLURAL') :
- loadTimeData.getStringF(
- entries.length === 1 ?
- 'OFFLINE_MESSAGE' :
- 'OFFLINE_MESSAGE_PLURAL',
- loadTimeData.getString('OFFLINE_COLUMN_LABEL')));
+ loadTimeData.getStringF(
+ entries.length === 1 ?
+ 'HOSTED_OFFLINE_MESSAGE' :
+ 'HOSTED_OFFLINE_MESSAGE_PLURAL') :
+ loadTimeData.getStringF(
+ entries.length === 1 ?
+ 'OFFLINE_MESSAGE' :
+ 'OFFLINE_MESSAGE_PLURAL',
+ loadTimeData.getString('OFFLINE_COLUMN_LABEL')));
});
return;
}
@@ -546,12 +547,11 @@ FileTasks.prototype.executeInternalTask_ = function(id, entries) {
// crbug.com/345527.
var urls = util.entriesToURLs(entries);
var position = urls.indexOf(selectedEntry.toURL());
- chrome.fileManagerPrivate.getProfiles(function(profiles,
- currentId,
- displayedId) {
- fm.backgroundPage.launchAudioPlayer({items: urls, position: position},
- displayedId);
- });
+ chrome.fileManagerPrivate.getProfiles(
+ function(profiles, currentId, displayedId) {
+ fm.backgroundPage.launchAudioPlayer(
+ {items: urls, position: position}, displayedId);
+ });
return;
}
@@ -646,7 +646,7 @@ FileTasks.prototype.display_ = function(combobutton) {
combobutton.addSeparator();
var changeDefaultMenuItem = combobutton.addDropDownItem({
- label: loadTimeData.getString('CHANGE_DEFAULT_MENU_ITEM')
+ label: loadTimeData.getString('CHANGE_DEFAULT_MENU_ITEM')
});
changeDefaultMenuItem.classList.add('change-default');
}

Powered by Google App Engine
This is Rietveld 408576698