Packaged Files app: open files from Downloads tab

chrome/browser/resources/file_manager/js/file_tasks.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.
 
 /**
  * This object encapsulates everything related to tasks execution.
  * @param {FileManager} fileManager FileManager instance.
  * @param {object} opt_params File manager load parameters.
  */
 function FileTasks(fileManager, opt_params) {
@@ skipping 179 matching lines @@
 
 /**
  * Executes a single task.
  * @param {string} taskId Task identifier.
  * @param {Array.<string>=} opt_urls Urls to execute on instead of |this.urls_|.
  * @private
  */
 FileTasks.prototype.execute_ = function(taskId, opt_urls) {
   var urls = opt_urls || this.urls_;
   this.checkAvailability_(function() {
-    chrome.fileBrowserPrivate.executeTask(taskId, urls);

[dgozman, 2012/11/08 13:49:28]

We used this before for metrics and default task handling.
So, you are sacrificing the metrics now.

[Vladislav Kaznacheev, 2012/11/08 14:00:18]

So this is why we did that! FWIW I could not find where in the C++ we record the
event, so I am not really changing anything.
On 2012/11/08 13:49:28, dgozman wrote:

-
     var task_parts = taskId.split('|');
     if (task_parts[0] == util.platform.getAppId() && task_parts[1] == 'file') {
       // For internal tasks we do not listen to the event to avoid
       // handling the same task instance from multiple tabs.
       // So, we manually execute the task.
       this.executeInternalTask_(task_parts[2], urls);
+    } else {
+      chrome.fileBrowserPrivate.executeTask(taskId, urls);
     }
   }.bind(this));
 };
 
 /**
  * Checks whether the remote files are available right now.
  * @param {function} callback The callback.
  * @private
  */
 FileTasks.prototype.checkAvailability_ = function(callback) {
@@ skipping 350 matching lines @@
       message,
       items, defaultIdx,
       onSuccess);
 };
 
 FileTasks.decorate('display');
 FileTasks.decorate('updateMenuItem');
 FileTasks.decorate('execute');
 FileTasks.decorate('executeDefault');