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

Unified Diff: chrome/browser/resources/file_manager/background/js/background.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
Index: chrome/browser/resources/file_manager/background/js/background.js
diff --git a/chrome/browser/resources/file_manager/background/js/background.js b/chrome/browser/resources/file_manager/background/js/background.js
index 386aa2358d54ab7b7550cb036ccdf612d1ebeb69..604c2540183c7be77eef4f0fd9bf5f51444bad71 100644
--- a/chrome/browser/resources/file_manager/background/js/background.js
+++ b/chrome/browser/resources/file_manager/background/js/background.js
@@ -690,10 +690,6 @@ Background.prototype.onExecute_ = function(action, details) {
launchAudioPlayer({items: urls, position: 0});
break;
- case 'watch':
- launchVideoPlayer(urls[0]);
- break;
-
default:
var launchEnable = null;
var queue = new AsyncUtil.Queue();
@@ -803,21 +799,6 @@ function launchAudioPlayer(playlist, opt_displayedId) {
});
}
-var videoPlayer = new SingletonAppWindowWrapper('video_player.html',
- {hidden: true});
-
-/**
- * Launches the video player.
- * @param {string} url Video url.
- * @param {string=} opt_displayedId ProfileID of the desktop where the video
- * player should show.
- */
-function launchVideoPlayer(url, opt_displayedId) {
- videoPlayer.launch({url: url}, false, function(appWindow) {
- AppWindowWrapper.focusOnDesktop(videoPlayer.rawAppWindow, opt_displayedId);
- });
-}
-
/**
* Launches the app.
* @private
@@ -871,9 +852,6 @@ Background.prototype.onRestarted_ = function() {
});
callback();
});
-
- // Reopen video player.
- videoPlayer.reopen();
};
/**

Powered by Google App Engine
This is Rietveld 408576698