[Cleanup] Files.app: Adds missing JSdoc annotations in file_manager/*.js.

chrome/browser/resources/file_manager/js/file_copy_manager.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.
 
 if (chrome.extension) {
   var getContentWindows = function() {
     return chrome.extension.getViews();
   };
 }
 
@@ skipping 31 matching lines @@
   return fileCopyManagerInstance;
 };
 
 /**
  * A record of a queued copy operation.
  *
  * Multiple copy operations may be queued at any given time.  Additional
  * Tasks may be added while the queue is being serviced.  Though a
  * cancel operation cancels everything in the queue.
  *
+ * @constructor

[mtomasz, 2013/02/14 04:38:56]

I think in most places we put @constructor after @param. Please check.

[yoshiki, 2013/02/14 06:17:36]

You're right. The closure-compiler doc says so.
https://developers.google.com/closure/compiler/docs/js-for-compiler

  * @param {DirectoryEntry} sourceDirEntry Source directory.
  * @param {DirectoryEntry} targetDirEntry Target directory.
  */
 FileCopyManager.Task = function(sourceDirEntry, targetDirEntry) {
   this.sourceDirEntry = sourceDirEntry;
   this.targetDirEntry = targetDirEntry;
   this.originalEntries = null;
 
   this.pendingDirectories = [];
   this.pendingFiles = [];
@@ skipping 217 matching lines @@
   rv.totalBytes = rv.pendingBytes + rv.completedBytes;
 
   rv.percentage = rv.completedBytes / rv.totalBytes;
   if (rv.pendingItems === 1)
     rv.filename = pendingFile.name;
 
   return rv;
 };
 
 /**
  * Send an event to all the FileManager windows.

[mtomasz, 2013/02/14 04:38:56]

Empty line here?

[yoshiki, 2013/02/14 06:17:36]

Done.

  * @private
  * @param {string} eventName Event name.
  * @param {Object} eventArgs An object with arbitrary event parameters.
  */
 FileCopyManager.prototype.sendEvent_ = function(eventName, eventArgs) {
   if (this.cancelRequested_)
     return;  // Swallow events until cancellation complete.
 
   eventArgs.status = this.getStatus();
 
@@ skipping 109 matching lines @@
  */
 FileCopyManager.prototype.maybeCancel_ = function() {
   if (!this.cancelRequested_)
     return false;
 
   this.doCancel_();
   return true;
 };
 
 /**
  * Convert string in clipboard to entries and kick off pasting.

[mtomasz, 2013/02/14 04:38:56]

Empty line? Here and in other methods?

[yoshiki, 2013/02/14 06:17:36]

Done.

  * @param {Object} clipboard Clipboard contents.
  * @param {string} targetPath Target path.
  * @param {boolean} targetOnDrive If target is on Drive.
  */
 FileCopyManager.prototype.paste = function(clipboard, targetPath,
                                            targetOnDrive) {
   var self = this;
   var results = {
     sourceDirEntry: null,
     entries: [],
@@ skipping 824 matching lines @@
  */
 FileCopyManager.prototype.sendDeleteEvent_ = function(task, reason) {
   this.sendEvent_('delete', {
     reason: reason,
     id: task.id,
     urls: task.entries.map(function(e) {
       return util.makeFilesystemUrl(e.fullPath);
     })
   });
 };