Correct indentation, JSDoc, etc... to comply with closure linter.

ui/file_manager/file_manager/foreground/js/file_transfer_controller.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';
 
 /**
  * Global (placed in the window object) variable name to hold internal
  * file dragging information. Needed to show visual feedback while dragging
  * since DataTransfer object is in protected state. Reachable from other
@@ skipping 19 matching lines @@
                                 volumeManager,
                                 multiProfileShareDialog) {
   this.document_ = doc;
   this.fileOperationManager_ = fileOperationManager;
   this.metadataCache_ = metadataCache;
   this.directoryModel_ = directoryModel;
   this.volumeManager_ = volumeManager;
   this.multiProfileShareDialog_ = multiProfileShareDialog;
 
   this.directoryModel_.getFileList().addEventListener(
       'change', function(event) {

[yoshiki, 2014/09/16 00:49:08]

nit: new line for function?

[fukino, 2014/09/17 05:17:26]

Done.

-    if (this.directoryModel_.getFileListSelection().
-        getIndexSelected(event.index)) {
-      this.onSelectionChanged_();
-    }
-  }.bind(this));
+        if (this.directoryModel_.getFileListSelection().
+            getIndexSelected(event.index)) {
+          this.onSelectionChanged_();
+        }
+      }.bind(this));
   this.directoryModel_.getFileListSelection().addEventListener('change',
       this.onSelectionChanged_.bind(this));
 
   /**
    * Promise to be fulfilled with the thumbnail image of selected file in drag
    * operation. Used if only one element is selected.
    * @type {Promise}
    * @private
    */
   this.preloadedThumbnailImagePromise_ = null;
@@ skipping 203 matching lines @@
       return Promise.all(promises).then(concatArrays);
     };
 
     // Check all file entries and keeps only those need sharing operation.
     var processFileEntries = function(entries) {
       return new Promise(function(callback) {
         // TODO(mtomasz): Move conversion from entry to url to custom bindings.
         // crbug.com/345527.
         var urls = util.entriesToURLs(entries);
         chrome.fileManagerPrivate.getEntryProperties(urls, callback);
-      }).
-      then(function(metadatas) {
+      }).then(function(metadatas) {
         return entries.filter(function(entry, i) {
           var metadata = metadatas[i];
           return metadata && metadata.isHosted && !metadata.sharedWithMe;
         });
       });
     };
 
     // Check child entries.
     var processDirectoryEntry = function(entry) {
       return readEntries(entry.createReader());
@@ skipping 46 matching lines @@
     var effectAllowed = dataTransfer.effectAllowed !== 'uninitialized' ?
         dataTransfer.effectAllowed : dataTransfer.getData('fs/effectallowed');
     var toMove = util.isDropEffectAllowed(effectAllowed, 'move') &&
         (!util.isDropEffectAllowed(effectAllowed, 'copy') ||
          opt_effect === 'move');
     var destinationEntry =
         opt_destinationEntry || this.currentDirectoryContentEntry;
     var entries;
     var failureUrls;
 
-    util.URLsToEntries(sourceURLs).
-    then(function(result) {
+    util.URLsToEntries(sourceURLs).then(function(result) {
       entries = result.entries;
       failureUrls = result.failureUrls;
       // Check if cross share is needed or not.
       return this.getMultiProfileShareEntries_(entries);
-    }.bind(this)).
-    then(function(shareEntries) {
+    }.bind(this)).then(function(shareEntries) {
       if (shareEntries.length === 0)
         return;
       return this.multiProfileShareDialog_.show(shareEntries.length > 1).
           then(function(dialogResult) {
             if (dialogResult === 'cancel')
               return Promise.reject('ABORT');
             // Do cross share.
             // TODO(hirono): Make the loop cancellable.
             var requestDriveShare = function(index) {
               if (index >= shareEntries.length)
                 return;
               return new Promise(function(fulfill) {
                 chrome.fileManagerPrivate.requestDriveShare(
                     shareEntries[index].toURL(),
                     dialogResult,
                     function() {
                       // TODO(hirono): Check chrome.runtime.lastError here.
                       fulfill();
                     });
               }).then(requestDriveShare.bind(null, index + 1));
             };
             return requestDriveShare(0);
           });
-    }.bind(this)).
-    then(function() {
+    }.bind(this)).then(function() {
       // Start the pasting operation.
       this.fileOperationManager_.paste(
           entries, destinationEntry, toMove);
 
       // Publish events for failureUrls.
       for (var i = 0; i < failureUrls.length; i++) {
         var fileName =
             decodeURIComponent(failureUrls[i].replace(/^.+\//, ''));
         var event = new Event('source-not-found');
         event.fileName = fileName;
         event.progressType =
             toMove ? ProgressItemType.MOVE : ProgressItemType.COPY;
         this.dispatchEvent(event);
       }
-    }.bind(this)).
-    catch(function(error) {
+    }.bind(this)).catch(function(error) {
       if (error !== 'ABORT')
         console.error(error.stack ? error.stack : error);
     });
     return toMove ? 'move' : 'copy';
   },
 
   /**
    * Preloads an image thumbnail for the specified file entry.
    *
    * @this {FileTransferController}
    * @param {Entry} entry Entry to preload a thumbnail for.
    */
   preloadThumbnailImage_: function(entry) {
     var metadataPromise = new Promise(function(fulfill, reject) {
-      this.metadataCache_.getOne(entry,
-                                 'thumbnail|filesystem',
-                                 function(metadata) {
-        if (metadata)
-          fulfill(metadata);
-        else
-          reject('Failed to fetch metadata.');
-      });
+      this.metadataCache_.getOne(
+          entry, 'thumbnail|filesystem', function(metadata) {

[yoshiki, 2014/09/16 00:49:08]

nit: new line for function?

[fukino, 2014/09/17 05:17:26]

Done.

+            if (metadata)
+              fulfill(metadata);
+            else
+              reject('Failed to fetch metadata.');
+          });
     }.bind(this));
 
     var imagePromise = metadataPromise.then(function(metadata) {
       return new Promise(function(fulfill, reject) {
         var loader = new ThumbnailLoader(
             entry, ThumbnailLoader.LoaderType.Image, metadata);
         loader.loadDetachedImage(function(result) {
           if (result)
             fulfill(loader.getImage());
         });
@@ skipping 113 matching lines @@
     } else {
       event.preventDefault();
       return;
     }
 
     var dragThumbnail = this.renderThumbnail_();
     dt.setDragImage(dragThumbnail, 0, 0);
 
     window[DRAG_AND_DROP_GLOBAL_DATA] = {
       sourceRootURL: dt.getData('fs/sourceRootURL'),
-      missingFileContents: dt.getData('fs/missingFileContents'),
+      missingFileContents: dt.getData('fs/missingFileContents')
     };
   },
 
   /**
    * @this {FileTransferController}
    * @param {cr.ui.List} list Drop target list.
    * @param {Event} event A dragend event of DOM.
    */
   onDragEnd_: function(list, event) {
     // TODO(fukino): This is workaround for crbug.com/373125.
@@ skipping 510 matching lines @@
           !event.ctrlKey) {
         return 'move';
       }
       if (event.shiftKey) {
         return 'move';
       }
     }
     return 'copy';
   },
 };