Add notification before getMultiProfileShareEntries_.

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

 // Copyright 2013 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';
 
 /**
  * Utilities for FileOperationManager.
  */
 var fileOperationUtil = {};
@@ skipping 1166 matching lines @@
 };
 
 /**
  * Kick off pasting.
  *
  * @param {Array.<Entry>} sourceEntries Entries of the source files.
  * @param {DirectoryEntry} targetEntry The destination entry of the target
  *     directory.
  * @param {boolean} isMove True if the operation is "move", otherwise (i.e.
  *     if the operation is "copy") false.
+ * @param {string} optTaskId If the corresponding item has already created
+ *     at another places, we need to specify the ID of the item. If the
+ *     item is not created, FileOperationManager generates new ID.
  */
 FileOperationManager.prototype.paste = function(
-    sourceEntries, targetEntry, isMove) {
+    sourceEntries, targetEntry, isMove, optTaskId) {

[hirono, 2014/09/17 03:33:20]

nit: The type name should be string= since it is optional.
nit: Start the optional variable name with opt_.
https://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml?showon...

[iseki, 2014/09/17 04:08:47]

Done.

   // Do nothing if sourceEntries is empty.
   if (sourceEntries.length === 0)
     return;
 
   var filteredEntries = [];
   var resolveGroup = new AsyncUtil.Queue();
 
   if (isMove) {
     for (var index = 0; index < sourceEntries.length; index++) {
       resolveGroup.run(function(sourceEntry, callback) {
@@ skipping 13 matching lines @@
   } else {
     // Always copy all of the files.
     filteredEntries = sourceEntries;
   }
 
   resolveGroup.run(function(callback) {
     // Do nothing, if we have no entries to be pasted.
     if (filteredEntries.length === 0)
       return;
 
-    this.queueCopy_(targetEntry, filteredEntries, isMove);
+    this.queueCopy_(targetEntry, filteredEntries, isMove, optTaskId);
   }.bind(this));
 };
 
 /**
  * Initiate a file copy. When copying files, null can be specified as source
  * directory.
  *
  * @param {DirectoryEntry} targetDirEntry Target directory.
  * @param {Array.<Entry>} entries Entries to copy.
  * @param {boolean} isMove In case of move.
+ * @param {string} optTaskId If the corresponding item has already created
+ *     at another places, we need to specify the ID of the item. If the
+ *     item is not created, FileOperationManager generates new ID.
  * @private
  */
 FileOperationManager.prototype.queueCopy_ = function(
-    targetDirEntry, entries, isMove) {
+    targetDirEntry, entries, isMove, optTaskId) {
   var task;
   if (isMove) {
     // When moving between different volumes, moving is implemented as a copy
     // and delete. This is because moving between volumes is slow, and moveTo()
     // is not cancellable nor provides progress feedback.
     if (util.isSameFileSystem(entries[0].filesystem,
                               targetDirEntry.filesystem)) {
       task = new FileOperationManager.MoveTask(entries, targetDirEntry);
     } else {
       task = new FileOperationManager.CopyTask(entries, targetDirEntry, true);
     }
   } else {
     task = new FileOperationManager.CopyTask(entries, targetDirEntry, false);
   }
 
-  task.taskId = this.generateTaskId_();
+  task.taskId = optTaskId || this.generateTaskId_();
   this.eventRouter_.sendProgressEvent('BEGIN', task.getStatus(), task.taskId);
   task.initialize(function() {
     this.copyTasks_.push(task);
     if (this.copyTasks_.length === 1)
       this.serviceAllTasks_();
   }.bind(this));
 };
 
 /**
  * Service all pending tasks, as well as any that might appear during the
@@ skipping 188 matching lines @@
 
 /**
  * Generates new task ID.
  *
  * @return {string} New task ID.
  * @private
  */
 FileOperationManager.prototype.generateTaskId_ = function() {
   return 'file-operation-' + this.taskIdCounter_++;
 };