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

Unified Diff: chrome/browser/resources/file_manager/video_player.html

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/video_player.html
diff --git a/chrome/browser/resources/file_manager/video_player.html b/chrome/browser/resources/file_manager/video_player.html
deleted file mode 100644
index cb1ac0a907afc39cae21922e5606d679029e9759..0000000000000000000000000000000000000000
--- a/chrome/browser/resources/file_manager/video_player.html
+++ /dev/null
@@ -1,57 +0,0 @@
-<!--
- -- Copyright (c) 2012 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.
- -->
-<html>
-<head>
- <!-- We have to set some default title, or chrome will use the page name.
- -- As soon as the i18n'd strings are loaded we replace it with the correct
- -- string. Until then, use an invisible non-whitespace character.
- -->
- <title>&#xFEFF;</title>
- <link rel="icon" type="image/png" href="chrome://theme/IDR_FILE_MANAGER_IMG_FILETYPE_VIDEO">
- <link rel="stylesheet" type="text/css" href="foreground/css/media_controls.css">
- <link rel="stylesheet" type="text/css" href="foreground/css/video_player.css">
hirono 2014/03/25 07:37:26 Can we remove the CSS?
yoshiki 2014/03/25 07:45:46 Thanks, I missed it. Removed.
-
- <!-- Don't load video_player_scripts.js when flattening is disabled -->
- <if expr="False"><!-- </if>
- <script src="foreground/js/media/video_player_scripts.js"></script>
- <if expr="False"> --></if>
-
- <if expr="False">
- <!-- This section is used when the file manager is loaded with
- 'filemgr-ext-path' command-line flag. -->
- <!-- Keep the list in sync with video_player_scripts.js. -->
- <script src="chrome://resources/js/cr.js"></script>
- <script src="chrome://resources/js/cr/event_target.js"></script>
- <script src="chrome://resources/js/cr/ui/array_data_model.js"></script>
- <script src="chrome://resources/js/load_time_data.js"></script>
-
- <script src="common/js/async_util.js"></script>
- <script src="common/js/util.js"></script>
- <script src="common/js/path_util.js"></script>
-
- <script src="foreground/js/file_type.js"></script>
- <script src="foreground/js/volume_manager_wrapper.js"></script>
- <script src="foreground/js/metadata/metadata_cache.js"></script>
-
- <script src="foreground/js/media/media_controls.js"></script>
- <script src="foreground/js/media/util.js"></script>
- <script src="foreground/js/media/video_player.js"></script>
- <script src="foreground/js/media/player_testapi.js"></script>
- </if>
-</head>
-<body>
- <div id="video-player" tools>
- <div id="video-container">
- </div>
- <div id="controls-wrapper">
- <div id="controls" class="tool"></div>
- </div>
- <div id="error-wrapper">
- <div id="error"></div>
- </div>
- </div>
-</body>
-</html>

Powered by Google App Engine
This is Rietveld 408576698