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

Side by Side Diff: ui/file_manager/video_player/js/background.js

Issue 893533004: [Video Player] Remove the default id of the video player (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2272
Patch Set: Created 5 years, 10 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * Icon of the video player. 6 * Icon of the video player.
7 * TODO(yoshiki): Consider providing an exact size icon, instead of relying 7 * TODO(yoshiki): Consider providing an exact size icon, instead of relying
8 * on downsampling by ash. 8 * on downsampling by ash.
9 * 9 *
10 * @type {string} 10 * @type {string}
11 * @const 11 * @const
12 */ 12 */
13 var ICON_IMAGE = 'images/icon/video-player-64.png'; 13 var ICON_IMAGE = 'images/icon/video-player-64.png';
14 14
15 var appWindowsForTest = {}; 15 var appWindowsForTest = {};
16 16
17 /** 17 /**
18 * Configuration of the video player panel. 18 * Configuration of the video player panel.
19 * @type {Object} 19 * @type {Object}
20 */ 20 */
21 var windowCreateOptions = { 21 var windowCreateOptions = {
22 id: 'video',
23 frame: 'none', 22 frame: 'none',
24 minWidth: 480, 23 minWidth: 480,
25 minHeight: 270 24 minHeight: 270
26 }; 25 };
27 26
28 /** 27 /**
29 * Backgound object. This is necessary for AppWindowWrapper. 28 * Backgound object. This is necessary for AppWindowWrapper.
30 * @type {BackgroundBase} 29 * @type {BackgroundBase}
31 */ 30 */
32 var background = new BackgroundBase(); 31 var background = new BackgroundBase();
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 chrome.runtime.onMessageExternal.addListener(function(message) { 150 chrome.runtime.onMessageExternal.addListener(function(message) {
152 if (message.name !== 'testResourceLoaded') 151 if (message.name !== 'testResourceLoaded')
153 return; 152 return;
154 var script = document.createElement('script'); 153 var script = document.createElement('script');
155 script.src = 154 script.src =
156 'chrome-extension://' + window.testExtensionId + 155 'chrome-extension://' + window.testExtensionId +
157 '/common/test_loader.js'; 156 '/common/test_loader.js';
158 document.documentElement.appendChild(script); 157 document.documentElement.appendChild(script);
159 }.wrap()); 158 }.wrap());
160 } 159 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698