Video Player: Use external entries instead of isolated entries

ui/file_manager/video_player/js/background.js

 // Copyright 2014 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';
 
 // Stores the app windows OLNY for test purpose.
 // We SHOULD NOT use it as it is except for test, since the files which have
 // the same name will be overridden each other.
 var appWindowsForTest = {};
 
-chrome.app.runtime.onLaunched.addListener(function(launchData) {
+var initializeQueue = new AsyncUtil.Queue();
+
+// Initializes the strings. This needs for the volume manager.
+initializeQueue.run(function(fulfill) {
+  chrome.fileBrowserPrivate.getStrings(function(stringData) {
+    loadTimeData.data = stringData;
+    fulfill();
+  }.wrap());
+}.wrap());
+
+// Initializes the volume manager. This needs for isolated entries.
+initializeQueue.run(function(fulfill) {
+  VolumeManager.getInstance(fulfill);
+}.wrap());
+
+// Registers a handler.
+initializeQueue.run(function(fulfill) {
+  chrome.app.runtime.onLaunched.addListener(onLaunched);

[hirono, 2014/07/23 02:48:58]

I'm not sure if it is OK to register the onLaunched handler asynchronously.
What happens when the background page is launched by the launched event?

[yoshiki, 2014/07/23 03:49:39]

I'm also not sure but I think it's better to care. Fixed.

+  fulfill();
+}.wrap());
+
+/**
+ * Called when an app is launched.
+ * @param {Object} launchData Launch data.
+ */
+function onLaunched(launchData) {
   if (!launchData || !launchData.items || launchData.items.length == 0)
     return;
 
-  var getFilePromises = launchData.items.map(function(item) {
-    var entry = item.entry;
-    return new Promise(function(fullfill, reject) {
-      entry.file(
-          function(file) {
-            fullfill({
+  var launchQueue = new AsyncUtil.Queue();
+  var videos = [];
+
+  launchQueue.run(function(fulfill) {
+    var isolatedEntries = launchData.items.map(function(item) {
+      return item.entry;
+    });
+
+    chrome.fileBrowserPrivate.resolveIsolatedEntries(isolatedEntries,
+        function(externalEntries) {
+          videos = externalEntries.map(function(entry) {
+            return {
               entry: entry,
-              file: file,
-              fileUrl: window.URL.createObjectURL(file)
-            });
-          },
-          function() {
-            fullfill({entry: entry, file: null, fileUrl: null});
+              title: entry.name,
+              url: entry.toURL(),
+            };
           });
-    });
+          fulfill();
+        }.wrap());
   });

[hirono, 2014/07/23 02:48:58]

Should wrap here?

[yoshiki, 2014/07/23 03:49:39]

Done.

 
-  Promise.all(getFilePromises).then(function(results) {
-    results = results.filter(function(result) { return result.file !== null; });
-    if (results.length > 0)
-      open(results);
-  }.wrap(),
-  function() {
-    // TODO(yoshiki): handle error in a smarter way.
-    open('', 'error');  // Empty URL shows the error message.
+  launchQueue.run(function(fulfill) {
+    if (videos.length > 0) {
+      open(videos);
+    } else {
+      // TODO(yoshiki): handle error in a smarter way.
+      open('', 'error');  // Empty URL shows the error message.
+    }
+    fulfill();
   }.wrap());
-}.wrap());
+}
 
 /**
  * Opens player window.
  * @param {Array.<Object>} videos List of videos to play.
- **/
+ */
 function open(videos) {
   chrome.app.window.create('video_player.html', {
     id: 'video',
     frame: 'none',
     singleton: false,
     minWidth: 480,
     minHeight: 270
   },
   function(createdWindow) {
     // Stores the window for test purpose.
     appWindowsForTest[videos[0].entry.name] = createdWindow;
 
     createdWindow.setIcon('images/200/icon.png');
     createdWindow.contentWindow.videos = videos;
     chrome.runtime.sendMessage({ready: true}, function() {});
   }.wrap());
 }