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

Unified Diff: chrome/browser/resources/file_manager/foreground/js/file_tasks.js

Issue 210623003: [VideoPlayer] Remove the old video player from Files.app (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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/foreground/js/file_tasks.js
diff --git a/chrome/browser/resources/file_manager/foreground/js/file_tasks.js b/chrome/browser/resources/file_manager/foreground/js/file_tasks.js
index ad8f6dd021bc0faf9ef4bb7478ffef5d2059603b..837631016f89764d06e65c5864d9b7d4e2bd6790 100644
--- a/chrome/browser/resources/file_manager/foreground/js/file_tasks.js
+++ b/chrome/browser/resources/file_manager/foreground/js/file_tasks.js
@@ -188,7 +188,6 @@ FileTasks.isInternalTask_ = function(taskId) {
return (appId === chrome.runtime.id &&
taskType === 'file' &&
(actionId === 'play' ||
- actionId === 'watch' ||
actionId === 'mount-archive' ||
actionId === 'gallery'));
};
@@ -235,9 +234,6 @@ FileTasks.prototype.processTasks_ = function(tasks) {
} else if (taskParts[2] === 'gallery') {
task.iconType = 'image';
task.title = loadTimeData.getString('ACTION_OPEN');
- } else if (taskParts[2] === 'watch') {
- task.iconType = 'video';
- task.title = loadTimeData.getString('ACTION_WATCH');
} else if (taskParts[2] === 'open-hosted-generic') {
if (this.entries_.length > 1)
task.iconType = 'generic';
@@ -551,17 +547,6 @@ FileTasks.prototype.executeInternalTask_ = function(id, entries) {
return;
}
- if (id === 'watch') {
- console.assert(entries.length === 1, 'Cannot open multiple videos');
- // TODO(mtomasz): Pass an entry instead.
- chrome.fileBrowserPrivate.getProfiles(function(profiles,
- currentId,
- displayedId) {
- fm.backgroundPage.launchVideoPlayer(entries[0].toURL(), displayedId);
- });
- return;
- }
-
if (id === 'mount-archive') {
this.mountArchivesInternal_(entries);
return;

Powered by Google App Engine
This is Rietveld 408576698