Files.app: Make delete button on the toolbar standard button instead of CommandButton.

ui/file_manager/file_manager/foreground/js/toolbar_controller.js

 // Copyright 2015 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.
 
 /**
  * This class controls wires toolbar UI and selection model. When selection
  * status is changed, this class changes the view of toolbar. If cancel
  * selection button is pressed, this class clears the selection.
- * @param {!HTMLElement} cancelSelectionButton Button to cancel selection.
- * @param {!HTMLElement} cancelSelectionButtonWrapper Wapper for the button,
- *     which works as a spacer for the filesSelectionLabel.
- * @param {!HTMLElement} filesSelectedLabel Label to show how many files are
- *     selected.
+ * @param {!HTMLElement} toolbar Toolbar element which contains controls.
  * @param {!HTMLElement} navigationList Navigation list on the left pane. The
  *     position of silesSelectedLabel depends on the navitaion list's width.
  * @param {!FileSelectionHandler} selectionHandler
- * @param {!cr.ui.ListSelectionModel|!cr.ui.ListSingleSelectionModel}
- *     selectionModel
+ * @param {!DirectoryModel} directoryModel
  * @constructor
  * @struct
  */
-function ToolbarController(cancelSelectionButton,
-                           cancelSelectionButtonWrapper,
-                           filesSelectedLabel,
+function ToolbarController(toolbar,
                            navigationList,
                            selectionHandler,
-                           selectionModel) {
+                           directoryModel) {
   /** @private {!HTMLElement} */

[hirono, 2015/02/23 09:48:37]

nit: We would like to add @const for member variables that are not reassigned.

[fukino, 2015/02/23 11:18:50]

Done.

-  this.cancelSelectionButtonWrapper_ = cancelSelectionButtonWrapper;
+  this.toolbar_ = toolbar;
 
   /** @private {!HTMLElement} */
-  this.filesSelectedLabel_ = filesSelectedLabel;
+  this.cancelSelectionButton_ =
+      queryRequiredElement(this.toolbar_, '#cancel-selection-button');
+
+  /** @private {!HTMLElement} */
+  this.cancelSelectionButtonWrapper_ =
+      queryRequiredElement(this.toolbar_, '#cancel-selection-button-wrapper');
+
+  /** @private {!HTMLElement} */
+  this.filesSelectedLabel_ =
+      queryRequiredElement(this.toolbar_, '#files-selected-label');
+
+  /** @private {!HTMLElement} */
+  this.deleteButton_ = queryRequiredElement(this.toolbar_, '#delete-button');
+
+  /** @private {!cr.ui.Command} */
+  this.deleteCommand_ = assertInstanceof(
+      queryRequiredElement(assert(this.toolbar_.ownerDocument), '#delete'),
+      cr.ui.Command);
 
   /** @private {!HTMLElement} */
   this.navigationList_ = navigationList;
 
   /** @private {!FileSelectionHandler} */
   this.selectionHandler_ = selectionHandler;
 
-  /** @private {!cr.ui.ListSelectionModel|!cr.ui.ListSingleSelectionModel} */
-  this.selectionModel_ = selectionModel;
+  /** @private {!DirectoryModel} */
+  this.directoryModel_ = directoryModel;
 
   this.selectionHandler_.addEventListener(
       FileSelectionHandler.EventType.CHANGE,
       this.onSelectionChanged_.bind(this));
 
-  cancelSelectionButton.addEventListener(
+  this.cancelSelectionButton_.addEventListener(
       'click', this.onCancelSelectionButtonClicked_.bind(this));
 
+  this.deleteButton_.addEventListener(
+      'click', this.onDeleteButtonClicked_.bind(this));
+
   this.navigationList_.addEventListener(
       'relayout', this.onNavigationListRelayout_.bind(this));
 }
 
 /**
  * Handles selection's change event to update the UI.
  * @private
  */
 ToolbarController.prototype.onSelectionChanged_ = function() {
   var selection = this.selectionHandler_.selection;
 
   // Update the label "x files selected." on the header.
   if (selection.totalCount === 0) {
     this.filesSelectedLabel_.textContent = '';
   } else {
     var text;
     if (selection.directoryCount == 0)
       text = strf('MANY_FILES_SELECTED', selection.fileCount);
     else if (selection.fileCount == 0)
       text = strf('MANY_DIRECTORIES_SELECTED', selection.directoryCount);
     else
       text = strf('MANY_ENTRIES_SELECTED', selection.totalCount);
     this.filesSelectedLabel_.textContent = text;
   }
 
+  // Update visibility of the delete button.
+  this.deleteButton_.hidden =
+      selection.totalCount === 0 || this.directoryModel_.isReadOnly();
+
   // Set .selecting class to containing element to change the view accordingly.
   // TODO(fukino): This code changes the state of body, not the toolbar, to
   // update the checkmark visibility on grid view. This should be moved to a
   // controller which controls whole app window. Or, both toolbar and FileGrid
   // should listen to the FileSelectionHandler.
   this.filesSelectedLabel_.ownerDocument.body.classList.toggle(
       'selecting', selection.totalCount > 0);
 }
 
 /**
  * Handles click event for cancel button to change the selection state.
  * @private
  */
 ToolbarController.prototype.onCancelSelectionButtonClicked_ = function() {
-  this.selectionModel_.unselectAll();
+  this.directoryModel_.selectEntries([]);
 }
 
 /**
+ * Handles click event for delete button to execute the delete command.
+ * @private
+ */
+ToolbarController.prototype.onDeleteButtonClicked_ = function() {
+  this.deleteCommand_.execute();
+}
+
+/**
  * Handles the relayout event occured on the navigation list.
  * @private
  */
 ToolbarController.prototype.onNavigationListRelayout_ = function() {
   // Make the width of spacer same as the width of navigation list.
   var navWidth = parseFloat(
       window.getComputedStyle(this.navigationList_).width);
   this.cancelSelectionButtonWrapper_.style.width = navWidth + 'px';
 }