Nitpicking fixes for Files.app code.

BUG=none

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269555 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/chromeos/file_manager/volume_manager.cc

@@ -236,6 +236,7 @@ std::string GetMountPointNameForMediaStorage(
 VolumeInfo::VolumeInfo()
     : file_system_id(0),
       type(VOLUME_TYPE_GOOGLE_DRIVE),
+      device_type(chromeos::DEVICE_TYPE_UNKNOWN),
       mount_condition(chromeos::disks::MOUNT_CONDITION_NONE),
       is_parent(false),
       is_read_only(false) {}

chrome/browser/chromeos/file_manager/volume_manager.h

@@ -146,7 +146,7 @@ class VolumeManager : public KeyedService,
   bool FindVolumeInfoById(const std::string& volume_id,
                           VolumeInfo* result) const;
 
-  // For testing purpose, registers a native local file system poniting to
+  // For testing purpose, registers a native local file system pointing to
   // |path| with DOWNLOADS type, and adds its volume info.
   bool RegisterDownloadsDirectoryForTesting(const base::FilePath& path);
 

ui/file_manager/file_manager/audio_player/elements/audio_player.js

@@ -200,7 +200,7 @@ Polymer('audio-player', {
    * list element.
    */
   onReplayCurrentTrack: function() {
-    // Changes the current time back to the beggining, regardless of the current
+    // Changes the current time back to the beginning, regardless of the current
     // status (playing or paused).
     this.audioElement.currentTime = 0;
     this.audioController.time = 0;
@@ -318,7 +318,7 @@ Polymer('audio-player', {
   },
 
   /**
-   * Invoked the 'keydown' event is fired.
+   * Invoked when the 'keydown' event is fired.
    * @param {Event} event The event object.
    */
   onKeyDown_: function(event) {

ui/file_manager/file_manager/audio_player/elements/volume_controller.js

@@ -43,7 +43,7 @@
     /**
      * Invoked when the model changed.
      * @param {AudioPlayerModel} oldValue Old Value.
-     * @param {AudioPlayerModel} newValue Nld Value.
+     * @param {AudioPlayerModel} newValue New Value.
      */
     modelChanged: function(oldValue, newValue) {
       this.onVolumeChanged((oldValue || {}).volume, (newValue || {}).volume);
@@ -76,7 +76,7 @@
     width: 32,
 
     /**
-     * Invoked the 'volume' value in the model is changed.
+     * Invoked when the 'volume' value in the model is changed.
      * @param {number} oldValue Old value.
      * @param {number} newValue New value.
      */
@@ -86,7 +86,7 @@
     },
 
     /**
-     * Invoked the 'rawValue' property is changed.
+     * Invoked when the 'rawValue' property is changed.
      * @param {number} oldValue Old value.
      * @param {number} newValue New value.
      */
@@ -96,7 +96,7 @@
     },
 
     /**
-     * Invoked the 'keydown' event is fired.
+     * Invoked when the 'keydown' event is fired.
      * @param {Event} event The event object.
      */
     onKeyDown_: function(event) {

ui/file_manager/file_manager/background/js/file_operation_manager.js

@@ -434,8 +434,8 @@ FileOperationManager.Task = function(
   // If directory already exists, we try to make a copy named 'dir (X)',
   // where X is a number. When we do this, all subsequent copies from
   // inside the subtree should be mapped to the new directory name.
-  // For example, if 'dir' was copied as 'dir (1)', then 'dir\file.txt' should
-  // become 'dir (1)\file.txt'.
+  // For example, if 'dir' was copied as 'dir (1)', then 'dir/file.txt' should
+  // become 'dir (1)/file.txt'.
   this.renamedDirectories_ = [];
 };
 

ui/file_manager/file_manager/foreground/js/drive_banners.js

@@ -346,7 +346,7 @@ FileListBannerController.prototype.checkSpaceAndMaybeShowWelcomeBanner_ =
     // minimize the invocation.
     group.add(function(onCompleted) {
       // Current directory must be set, since this code is called after
-      // scaning is completed. However, the volumeInfo may be gone.
+      // scanning is completed. However, the volumeInfo may be gone.
       chrome.fileBrowserPrivate.getSizeStats(
           driveVolume.volumeId,
           function(result) {

ui/file_manager/file_manager/foreground/js/image_editor/image_editor.js

@@ -788,7 +788,7 @@ ImageEditor.MouseControl.prototype.setDoubleTapCallback = function(callback) {
 };
 
 /**
- * Touch chancel handler.
+ * Touch cancel handler.
  */
 ImageEditor.MouseControl.prototype.onTouchCancel = function() {
   this.dragHandler_ = null;
@@ -1119,7 +1119,7 @@ ImageEditor.Prompt.prototype.setTimer = function(callback, timeout) {
  *
  * @param {string} text The prompt text.
  * @param {number} timeout Timeout in ms.
- * @param {Object} formatArgs varArgs for the formatting fuction.
+ * @param {Object} formatArgs varArgs for the formatting function.
  */
 ImageEditor.Prompt.prototype.show = function(text, timeout, formatArgs) {
   this.showAt.apply(this,

ui/file_manager/file_manager/foreground/js/media/audio_player.js

@@ -468,7 +468,7 @@ AudioPlayer.prototype.syncHeight_ = function() {
       targetHeight = AudioPlayer.CONTROLS_HEIGHT + expandedListHeight;
     }
   } else {
-    // Not expaned.
+    // Not expanded.
     targetHeight = AudioPlayer.CONTROLS_HEIGHT + AudioPlayer.TRACK_HEIGHT;
   }
 

ui/file_manager/file_manager/foreground/js/ui/breadcrumbs_controller.js

@@ -99,7 +99,7 @@ BreadcrumbsController.prototype.show = function(entry) {
   queue.run(resolveParent.bind(this, entry, null));
 
   queue.run(function(callback) {
-    // If an error was occured, just skip.
+    // If an error occurred, just skip.
     if (error) {
       callback();
       return;

ui/file_manager/file_manager/foreground/js/ui/conflict_dialog.js

@@ -91,7 +91,7 @@ ConflictDialog.prototype = {
  *
  * @param {string} fileName Filename that is conflicted.
  * @param {function(ConflictDialog.Result, boolean)} callback Complete
- *     callbak. See also ConflictDialog#callback_.
+ *     callback. See also ConflictDialog#callback_.
  * @return {boolean} True if the dialog can show successfully. False if the
  *     dialog failed to show due to an existing dialog.
  */

ui/file_manager/file_manager/foreground/js/ui/navigation_list.js

@@ -310,7 +310,7 @@ NavigationList.prototype.activateModelItem_ = function(modelItem) {
         url,
         onEntryResolved,
         function() {
-          // Error, the entry can't be re-resolved. It may happen for shotrcuts
+          // Error, the entry can't be re-resolved. It may happen for shortcuts
           // which targets got removed after resolving the Entry during
           // initialization.
           this.dataModel.onItemNotFoundError(modelItem);

ui/file_manager/file_manager/foreground/js/ui/preview_panel.js

@@ -159,7 +159,7 @@ PreviewPanel.prototype = {
   set visibilityType(visibilityType) {
     this.visibilityType_ = visibilityType;
     this.updateVisibility_();
-    // Also update the preview area contents, because the update is surpressed
+    // Also update the preview area contents, because the update is suppressed
     // while the visibility is hiding or hidden.
     this.updatePreviewArea_();
   },
@@ -186,7 +186,7 @@ PreviewPanel.prototype.initialize = function() {
   this.element_.addEventListener('webkitTransitionEnd',
                                  this.onTransitionEnd_.bind(this));
   this.updateVisibility_();
-  // Also update the preview area contents, because the update is surpressed
+  // Also update the preview area contents, because the update is suppressed
   // while the visibility is hiding or hidden.
   this.updatePreviewArea_();
 };
@@ -251,7 +251,7 @@ PreviewPanel.prototype.updateVisibility_ = function() {
  * @private
  */
 PreviewPanel.prototype.updatePreviewArea_ = function(breadCrumbsVisible) {
-  // If the previw panel is hiding, does not update the current view.
+  // If the preview panel is hiding, does not update the current view.
   if (!this.visible)
     return;
   var selection = this.selection_;

ui/file_manager/file_manager/foreground/js/ui/progress_center_panel.js

@@ -49,7 +49,7 @@ function ProgressCenterItemElement(document) {
  */
 ProgressCenterItemElement.safelySetAnimation_ = function(callback) {
   var requestId = requestAnimationFrame(function() {
-    // The transitoin start properties currently set are rendered at this frame.
+    // The transition start properties currently set are rendered at this frame.
     // And the transition end properties set by the callback is rendered at the
     // next frame.
     requestId = requestAnimationFrame(callback);
@@ -67,9 +67,9 @@ ProgressCenterItemElement.safelySetAnimation_ = function(callback) {
 ProgressCenterItemElement.PROGRESS_ANIMATION_END_EVENT = 'progressAnimationEnd';
 
 /**
- * Decoreates the given element as a progress item.
- * @param {HTMLElement} element Item to be decoreated.
- * @return {ProgressCenterItemElement} Decoreated item.
+ * Decorates the given element as a progress item.
+ * @param {HTMLElement} element Item to be decorated.
+ * @return {ProgressCenterItemElement} Decorated item.
  */
 ProgressCenterItemElement.decorate = function(element) {
   element.__proto__ = ProgressCenterItemElement.prototype;

ui/file_manager/file_manager/foreground/js/ui/search_box.js

@@ -105,7 +105,7 @@ SearchBox.AutocompleteList.prototype.onMouseOver_ = function(event) {
 };
 
 /**
- * ListItem element for autocomple.
+ * ListItem element for autocomplete.
  *
  * @param {HTMLDocument} document Document.
  * @param {Object} item An object representing a suggestion.
@@ -193,7 +193,7 @@ SearchBox.prototype.onIconClick_ = function() {
 SearchBox.prototype.onDragEnter_ = function(event) {
   // For normal elements, they does not accept drag drop by default, and accept
   // it by using event.preventDefault. But input elements accept drag drop
-  // by default. So disalbe the input element here to prohibit drag drop.
+  // by default. So disable the input element here to prohibit drag drop.
   if (event.dataTransfer.types.indexOf('text/plain') === -1)
     this.inputElement.style.pointerEvents = 'none';
 };

ui/file_manager/gallery/js/gallery.js

@@ -67,14 +67,14 @@ function Gallery(volumeManager) {
 Gallery.prototype.__proto__ = cr.EventTarget.prototype;
 
 /**
- * Tools fade-out timeout im milliseconds.
+ * Tools fade-out timeout in milliseconds.
  * @const
  * @type {number}
  */
 Gallery.FADE_TIMEOUT = 3000;
 
 /**
- * First time tools fade-out timeout im milliseconds.
+ * First time tools fade-out timeout in milliseconds.
  * @const
  * @type {number}
  */

ui/file_manager/gallery/js/mosaic_mode.js

@@ -466,7 +466,7 @@ Mosaic.prototype.onSplice_ = function(event) {
   if (event.removed.length) {
     for (var t = 0; t !== event.removed.length; t++) {
       // If the layout for the tile has not done yet, the parent is null.
-      // And the layout will not be done after onSplice_ becuase it is removed
+      // And the layout will not be done after onSplice_ because it is removed
       // from this.tiles_.
       if (this.tiles_[index + t].parentNode)
         this.removeChild(this.tiles_[index + t]);

ui/file_manager/image_loader/cache.js

@@ -284,7 +284,7 @@ Cache.prototype.loadImage = function(key, timestamp, onSuccess, onFailure) {
 
     // Check if both entries are available or both unavailable.
     if (!!metadataEntry != !!dataEntry) {
-      console.warn('Incosistent cache database.');
+      console.warn('Inconsistent cache database.');
       onFailure();
       return;
     }

ui/file_manager/image_loader/image_loader.js

@@ -36,7 +36,7 @@ function ImageLoader() {
     });
     initPromises.push(new Promise(this.cache_.initialize.bind(this.cache_)));
 
-    // After all initializatino promises are done, start the worker.
+    // After all initialization promises are done, start the worker.
     Promise.all(initPromises).then(this.worker_.start.bind(this.worker_));
 
     // Listen for mount events, and grant permissions to volumes being mounted.

ui/file_manager/video_player/js/error_util.js

@@ -22,7 +22,7 @@ window.onerror = function() { window.JSErrorCount++; };
  *  - Bind this object
  *
  * @param {Object} thisObject Object to be used as this.
- * @return {function} Wapped function.
+ * @return {function} Wrapped function.
  */
 Function.prototype.wrap = function(thisObject) {
   var func = this;