Add tests for the video player in Files.app.

chrome/test/data/extensions/api_test/file_manager_browsertest/test_cases.js

 // Copyright (c) 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.
 
 /**
  * Expected files before tests are performed. Entries for Local tests.
  * @type {Array.<Array.<string>>}
  * @const
  */
 var EXPECTED_FILES_BEFORE_LOCAL = [
   ['hello.txt', '51 bytes', 'Plain text', 'Sep 4, 1998 12:34 PM'],
   ['world.ogv', '59 KB', 'OGG video', 'Jul 4, 2012 10:35 AM'],
   ['My Desktop Background.png', '272 bytes', 'PNG image',
       'Jan 18, 2038 1:02 AM'],
+  ['Beautiful Song.ogg', '14 KB', 'OGG audio', 'Nov 12, 2086 12:00 PM'],
   ['photos', '--', 'Folder', 'Jan 1, 1980 11:59 PM']
   // ['.warez', '--', 'Folder', 'Oct 26, 1985 1:39 PM']  # should be hidden
 ].sort();
 
 /**
  * Expected files before tests are performed. Entries for Drive tests.
  * @type {Array.<Array.<string>>}
  * @const
  */
 var EXPECTED_FILES_BEFORE_DRIVE = [
   ['hello.txt', '51 bytes', 'Plain text', 'Sep 4, 1998 12:34 PM'],
   ['world.ogv', '59 KB', 'OGG video', 'Jul 4, 2012 10:35 AM'],
   ['My Desktop Background.png', '272 bytes', 'PNG image',
       'Jan 18, 2038 1:02 AM'],
+  ['Beautiful Song.ogg', '14 KB', 'OGG audio', 'Nov 12, 2086 12:00 PM'],
   ['photos', '--', 'Folder', 'Jan 1, 1980 11:59 PM'],
   ['Test Document.gdoc','--','Google document','Apr 10, 2013 4:20 PM'],
   ['Test Shared Document.gdoc','--','Google document','Mar 20, 2013 10:40 PM']
 ].sort();
 
 /**
  * Expected files added during some tests.
  * @type {Array.<Array.<string>>}
  * @const
  */
@@ skipping 146 matching lines @@
     },
     function(result) {
       chrome.test.assertTrue(result);
       // Wait for the image in the gallery's screen image.
       callRemoteTestUtil('waitForElement',
                          appId,
                          ['.gallery .content canvas.image',
                           'iframe.overlay-pane'],
                          steps.shift());
     },
-    function(attributes) {
+    function(element) {
       // Verify the gallery's screen image.
-      chrome.test.assertEq('320', attributes['width']);
-      chrome.test.assertEq('240', attributes['height']);
+      chrome.test.assertEq('320', element.attributes.width);
+      chrome.test.assertEq('240', element.attributes.height);
       // Get the full-resolution image.
       callRemoteTestUtil('waitForElement',
                          appId,
                          ['.gallery .content canvas.fullres',
                           'iframe.overlay-pane'],
                          steps.shift());
     },
-    function(attributes) {
+    function(element) {
       // Verify the gallery's screen image.
-      chrome.test.assertEq('800', attributes['width']);
-      chrome.test.assertEq('600', attributes['height']);
+      chrome.test.assertEq('800', element.attributes.width);
+      chrome.test.assertEq('600', element.attributes.height);
       chrome.test.succeed();
-    },
+    }
   ];
   steps = steps.map(function(f) { return chrome.test.callbackPass(f); });
   steps.shift()();
+};
+
+/**
+ * Tests if the audio player shows up for the selected image and that the audio
+ * is loaded successfully.
+ *
+ * @param {string} path Directory path to be tested.
+ */
+testcase.intermediate.audioOpen = function(path) {
+  var appId;
+  var audioAppId;
+  var steps = [
+    function() {
+      setupAndWaitUntilReady(path, steps.shift());
+    },
+    function(inAppId) {
+      appId = inAppId;
+      // Select the song.
+      callRemoteTestUtil(
+          'selectFile', appId, ['Beautiful Song.ogg'], steps.shift());
+    },
+    function(result) {
+      chrome.test.assertTrue(result);
+      // Click on the label to enter the audio player.
+      callRemoteTestUtil(
+          'fakeMouseClick',
+          appId,
+          ['#file-list li.table-row[selected] .filename-label span'],
+          steps.shift());
+    },
+    function(result) {
+      chrome.test.assertTrue(result);
+      // Wait for the audio player.
+      callRemoteTestUtil('waitForWindow',
+                         null,
+                         ['mediaplayer.html'],
+                         steps.shift());
+    },
+    function(inAppId) {
+      audioAppId = inAppId;
+      // Wait for the audio tag and verify the source.
+      callRemoteTestUtil('waitForElement',
+                         audioAppId,
+                         ['audio[src]'],
+                         steps.shift());
+    },
+    function(element) {
+      chrome.test.assertEq(
+          'filesystem:chrome-extension://hhaomjibdihmijegdhdafkllkbggdgoj/' +
+              'external' + path + '/Beautiful%20Song.ogg',
+          element.attributes.src);
+      // Get the title tag.
+      callRemoteTestUtil('waitForElement',
+                         audioAppId,
+                         ['.data-title'],
+                         steps.shift());
+    },
+    function(element) {
+      chrome.test.assertEq('Beautiful Song', element.text);
+      // Get the artist tag.
+      callRemoteTestUtil('waitForElement',
+                         audioAppId,
+                         ['.data-artist'],
+                         steps.shift());
+    },
+    function(element) {
+      chrome.test.assertEq('Unknown Artist', element.text);
+      chrome.test.succeed();
+    }
+  ];
+  steps = steps.map(function(f) { return chrome.test.callbackPass(f); });
+  steps.shift()();
+};
+
+/**
+ * Tests if the video player shows up for the selected movie and that it is
+ * loaded successfully.
+ *
+ * @param {string} path Directory path to be tested.
+ */
+testcase.intermediate.videoOpen = function(path) {
+  var appId;
+  var videoAppId;
+  var steps = [
+    function() {
+      setupAndWaitUntilReady(path, steps.shift());
+    },
+    function(inAppId) {
+      appId = inAppId;
+      // Select the song.
+      callRemoteTestUtil(
+          'selectFile', appId, ['world.ogv'], steps.shift());
+    },
+    function(result) {
+      chrome.test.assertTrue(result);
+      // Click on the label to enter the video player.
+      callRemoteTestUtil(
+          'fakeMouseClick',
+          appId,
+          ['#file-list li.table-row[selected] .filename-label span'],
+          steps.shift());
+    },
+    function(result) {
+      chrome.test.assertTrue(result);
+      // Wait for the video player.
+      callRemoteTestUtil('waitForWindow',
+                         null,
+                         ['video_player.html'],
+                         steps.shift());
+    },
+    function(appId) {

[hirono, 2013/05/28 07:59:46]

Please use 'inAppId'.

[mtomasz, 2013/05/28 10:26:38]

Done.

+      videoAppId = appId;
+      // Wait for the video tag and verify the source.
+      callRemoteTestUtil('waitForElement',
+                         videoAppId,
+                         ['video[src]'],
+                         steps.shift());
+    },
+    function(element) {
+      chrome.test.assertEq(
+          'filesystem:chrome-extension://hhaomjibdihmijegdhdafkllkbggdgoj/' +
+              'external' + path + '/world.ogv',
+          element.attributes.src);
+      // Wait for the window's inner dimensions. Should be changed to the video
+      // size once the metadata is loaded.
+      callRemoteTestUtil('waitForWindowGeometry',
+                         videoAppId,
+                         [320, 192],
+                         chrome.test.succeed);

[hirono, 2013/05/28 07:59:46]

It seems that chrome.test.succeed just records success log.
Even if it is not called, the test will succeed.
We can use chrome.test.callbackPass to ensure the callback function is called.

[mtomasz, 2013/05/28 10:26:38]

I think chrome.test.succeed marks the test case as successful. However, the
default response is successful. Moreover, callbackPass implementation calls
chrome.test.succeed when there are no other callbacks pending, on closure end.

I think we should get rid of chrome.test.succeed in the future, since it seem to
be a no-op here. I may be wrong though. I have to double check it.

I've fixed here. Done.

+    }
+  ];
+  steps = steps.map(function(f) { return chrome.test.callbackPass(f); });
+  steps.shift()();
 };
 
 /**
  * Tests copying a file to the same directory and waits until the file lists
  * changes.
  *
  * @param {string} path Directory path to be tested.
  */
 testcase.intermediate.keyboardCopy = function(path, callback) {
   setupAndWaitUntilReady(path, function(appId) {
@@ skipping 28 matching lines @@
 };
 
 testcase.fileDisplayDownloads = function() {
   testcase.intermediate.fileDisplay('/Downloads');
 };
 
 testcase.galleryOpenDownloads = function() {
   testcase.intermediate.galleryOpen('/Downloads');
 };
 
+testcase.audioOpenDownloads = function() {
+  testcase.intermediate.audioOpen('/Downloads');
+};
+
+testcase.videoOpenDownloads = function() {
+  testcase.intermediate.videoOpen('/Downloads');
+};
+
 testcase.keyboardCopyDownloads = function() {
   testcase.intermediate.keyboardCopy('/Downloads');
 };
 
 testcase.keyboardDeleteDownloads = function() {
   testcase.intermediate.keyboardDelete('/Downloads');
 };
 
 testcase.fileDisplayDrive = function() {
   testcase.intermediate.fileDisplay('/drive/root');
 };
 
 testcase.galleryOpenDrive = function() {
   testcase.intermediate.galleryOpen('/drive/root');
 };
 
+testcase.audioOpenDrive = function() {
+  testcase.intermediate.audioOpen('/drive/root');
+};
+
+testcase.videoOpenDrive = function() {
+  testcase.intermediate.videoOpen('/drive/root');
+};
+
 testcase.keyboardCopyDrive = function() {
   testcase.intermediate.keyboardCopy('/drive/root');
 };
 
 testcase.keyboardDeleteDrive = function() {
   testcase.intermediate.keyboardDelete('/drive/root');
 };
 
 /**
  * Tests opening the "Recent" on the sidebar navigation by clicking the icon,
@@ skipping 281 matching lines @@
 /**
  * Tests copy from drive's offline to drive's root.
  */
 testcase.transferFromOfflineToDrive = function() {
   testcase.intermediate.copyBetweenVolumes('Test Document.gdoc',
                                            'drive_offline',
                                            EXPECTED_FILES_IN_OFFLINE,
                                            'drive',
                                            EXPECTED_FILES_BEFORE_DRIVE);
 };