Packaged Files app: open files from Downloads tab

chrome/browser/resources/file_manager/js/file_tasks.js

Index: chrome/browser/resources/file_manager/js/file_tasks.js
diff --git a/chrome/browser/resources/file_manager/js/file_tasks.js b/chrome/browser/resources/file_manager/js/file_tasks.js
index 6117ec56371cd9e282192bd7db70e63f9a076ec4..656f18ab93f566fde7d6c1a2ff3380dbd0b6c64d 100644
--- a/chrome/browser/resources/file_manager/js/file_tasks.js
+++ b/chrome/browser/resources/file_manager/js/file_tasks.js
@@ -197,14 +197,14 @@ FileTasks.prototype.executeDefault_ = function() {
 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));
 };