Video Player: Add a test to open a single video file

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 = {};
@@ skipping 49 matching lines @@
     } else {
       // TODO(yoshiki): handle error in a smarter way.
       open('', 'error');  // Empty URL shows the error message.
     }
     fulfill();
   }.wrap());
 }
 
 /**
  * Opens player window.
  * @param {Array.<Object>} videos List of videos to play.

[hirono, 2014/08/20 04:14:06]

nit: @return

[yoshiki, 2014/08/20 04:26:21]

Done.

  */
 function open(videos) {
-  chrome.app.window.create('video_player.html', {
-    id: 'video',
-    frame: 'none',
-    singleton: false,
-    minWidth: 480,
-    minHeight: 270
-  },
-  function(createdWindow) {
+  return new Promise(function(fulfill, reject) {
+    chrome.app.window.create('video_player.html', {
+      id: 'video',
+      frame: 'none',
+      singleton: false,
+      minWidth: 480,
+      minHeight: 270
+    },
+    fulfill);
+  }).then(function(createdWindow) {
     // Stores the window for test purpose.
     appWindowsForTest[videos[0].entry.name] = createdWindow;
 
     createdWindow.setIcon('images/icon/video-player-64.png');
     createdWindow.contentWindow.videos = videos;
     chrome.runtime.sendMessage({ready: true}, function() {});
-  }.wrap());
+  }).catch(function(error) {
+    console.error('Launch failed', error.stack || error);

[hirono, 2014/08/20 04:14:06]

How about doing: return Promise.reject(error) ?
It make the returned promise rejected, and thus the test is failed by the error.

[yoshiki, 2014/08/20 04:26:21]

Done.

+  });
 }
+
+// If is is run in the browser test, wait for the test resources are installed
+// as a component extension, and then load the test resources.
+if (chrome.test) {
+  window.testExtensionId = 'ljoplibgfehghmibaoaepfagnmbbfiga';
+  chrome.runtime.onMessageExternal.addListener(function(message) {
+    if (message.name !== 'testResourceLoaded')
+      return;
+    var script = document.createElement('script');
+    script.src =
+        'chrome-extension://' + window.testExtensionId +
+        '/common/test_loader.js';
+    document.documentElement.appendChild(script);
+  });
+}