Delete images in Photo Editor

chrome/browser/resources/file_manager/js/photo/gallery.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.
 
 document.addEventListener('DOMContentLoaded', function() {
   if (document.location.hash)  // File path passed after the #.
     Gallery.openStandalone(decodeURI(document.location.hash.substr(1)));
 });
 
 /**
@@ skipping 352 matching lines @@
     this.enterMode_(opt_callback);
   }.bind(this));
 };
 
 /**
  * Delete event handler.
  * @private
  */
 Gallery.prototype.onDelete_ = function() {
   // Clone the sorted selected indexes array.
-  var toRemove = this.selectionModel_.selectedIndexes.slice();
-  this.selectionModel_.unselectAll();
+  var indexesToRemove = this.selectionModel_.selectedIndexes.slice();
+  if (!indexesToRemove.length)
+    return;
 
-  // Remove items starting from the highest index.
-  while (toRemove.length)
-    this.dataModel_.splice(toRemove.pop(), 1);
+  /* TODO(dgozman): implement Undo delete. */
 
-  // TODO: delete actual files.
+  var itemsToRemove = this.getSelectedItems();
+  var plural = itemsToRemove.length > 1;
+  var param = plural ? itemsToRemove.length : itemsToRemove[0].getFileName();
+
+  cr.ui.dialogs.BaseDialog.OK_LABEL = this.displayStringFunction_('OK_LABEL');
+  cr.ui.dialogs.BaseDialog.CANCEL_LABEL =
+      this.displayStringFunction_('CANCEL_LABEL');
+  var confirm = new cr.ui.dialogs.ConfirmDialog(this.container_);
+  confirm.show(this.displayStringFunction_(
+      plural ? 'CONFIRM_DELETE_SOME' : 'CONFIRM_DELETE_ONE', param),
+      function() {
+        this.selectionModel_.unselectAll();
+
+        // Remove items from the data model, starting from the highest index.
+        while (indexesToRemove.length)
+          this.dataModel_.splice(indexesToRemove.pop(), 1);
+
+        // Delete actual files.
+        var deleteNext = function() {

[dgozman, 2012/09/07 10:02:20]

You can define this function outside callback to reduce the nested complexity.

[Vladislav Kaznacheev, 2012/09/07 12:30:16]

Done.

+          var item = itemsToRemove.pop();
+          var url = item.getUrl();
+          webkitResolveLocalFileSystemURL(url,
+              function(entry) {
+                entry.remove(deleteNext,
+                    util.flog('Error deleting ' + url, deleteNext));
+              },
+              util.flog('Error resolving ' + url, deleteNext));
+        }.bind(this);

[dgozman, 2012/09/07 10:02:20]

No need to bind here.

[Vladislav Kaznacheev, 2012/09/07 12:30:16]

Done.

+        deleteNext();
+      }.bind(this));
 };
 
 /**
  * @return {Array.<Gallery.Item>} Current selection.
  */
 Gallery.prototype.getSelectedItems = function() {
   return this.selectionModel_.selectedIndexes.map(
       this.dataModel_.item.bind(this.dataModel_));
 };
 
@@ skipping 247 matching lines @@
         this.toggleShare_();  // Hide the menu.
         this.executeWhenReady(api.executeTask.bind(api, taskId, urls));
       }.bind(this, task.taskId));
     }
 
     var empty = this.shareMenu_.querySelector('.item') == null;
     ImageUtil.setAttribute(this.shareButton_, 'disabled', empty);
     this.shareMenu_.hidden = wasHidden || empty;
   }.bind(this));
 };