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

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

Issue 1400183002: VideoPlayer: Use system header. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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
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 /** 15 /**
16 * Configuration of the video player panel. 16 * Configuration of the video player panel.
17 * @type {Object} 17 * @type {Object}
18 */ 18 */
19 var windowCreateOptions = { 19 var windowCreateOptions = {
20 frame: 'none',
yawano 2015/10/14 09:26:54 Could you confirm that header color of mock is sam
fukino 2015/10/15 03:56:40 Yes, it's different. The design spec was rgb(250,2
21 minWidth: 480, 20 minWidth: 480,
22 minHeight: 270 21 minHeight: 270
23 }; 22 };
24 23
25 /** 24 /**
26 * Backgound object. This is necessary for AppWindowWrapper. 25 * Backgound object. This is necessary for AppWindowWrapper.
27 * @type {BackgroundBase} 26 * @type {BackgroundBase}
28 */ 27 */
29 var background = new BackgroundBase(); 28 var background = new BackgroundBase();
30 29
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 appWindow.focus(); 83 appWindow.focus();
85 84
86 return windowId; 85 return windowId;
87 }.wrap()).catch(function(error) { 86 }.wrap()).catch(function(error) {
88 console.error('Launch failed' + error.stack || error); 87 console.error('Launch failed' + error.stack || error);
89 return Promise.reject(error); 88 return Promise.reject(error);
90 }.wrap()); 89 }.wrap());
91 } 90 }
92 91
93 background.setLaunchHandler(openVideoPlayerWindow); 92 background.setLaunchHandler(openVideoPlayerWindow);
OLDNEW
« no previous file with comments | « ui/file_manager/video_player/css/header.css ('k') | ui/file_manager/video_player/js/video_player.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698