Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6683)

Unified Diff: chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js

Issue 210623003: [VideoPlayer] Remove the old video player from Files.app (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/resources/video_player/manifest.json ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js
diff --git a/chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js b/chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js
index 0da4d4063c3faad269385386525388852d8d64fa..969569c66a88627c1da5d455e1a9969a0ee73cd7 100644
--- a/chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js
+++ b/chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js
@@ -98,50 +98,6 @@ function audioOpen(path) {
}
/**
- * 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.
- */
-function videoOpen(path) {
- var appId;
- var videoAppId;
- StepsRunner.run([
- function() {
- setupAndWaitUntilReady(null, path, this.next);
- },
- function(inAppId) {
- appId = inAppId;
- // Select the song.
- callRemoteTestUtil(
- 'openFile', appId, ['world.ogv'], this.next);
- },
- function(result) {
- chrome.test.assertTrue(result);
- // Wait for the video player.
- waitForWindow('video_player.html').then(this.next);
- },
- function(inAppId) {
- videoAppId = inAppId;
- // Wait for the video tag and verify the source.
- waitForElement(videoAppId, 'video[src]').then(this.next);
- },
- 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.
- waitForWindowGeometry(videoAppId, 320, 192).then(this.next);
- },
- function(element) {
- checkIfNoErrorsOccured(this.next);
- }
- ]);
-}
-
-/**
* Tests if we can open and unmount a zip file.
* @param {string} path Directory path to be tested.
*/
@@ -216,10 +172,6 @@ testcase.audioOpenDownloads = function() {
audioOpen(RootPath.DOWNLOADS);
};
-testcase.videoOpenDownloads = function() {
- videoOpen(RootPath.DOWNLOADS);
-};
-
testcase.galleryOpenDrive = function() {
galleryOpen(RootPath.DRIVE);
};
@@ -228,10 +180,6 @@ testcase.audioOpenDrive = function() {
audioOpen(RootPath.DRIVE);
};
-testcase.videoOpenDrive = function() {
- videoOpen(RootPath.DRIVE);
-};
-
testcase.zipOpenDownloads = function() {
zipOpen(RootPath.DOWNLOADS);
};
« no previous file with comments | « chrome/browser/resources/video_player/manifest.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698