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

Side by Side Diff: ui/file_manager/video_player/js/compiled_resources.gyp

Issue 2292873003: Divide volume_manager.js into files for each classes and extract interfaces from them. (Closed)
Patch Set: Address comments. Created 4 years, 3 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 | « ui/file_manager/gallery/js/compiled_resources.gyp ('k') | 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 'targets': [ 5 'targets': [
6 { 6 {
7 'target_name': 'background', 7 'target_name': 'background',
8 'variables': { 8 'variables': {
9 'depends': [ 9 'depends': [
10 '../../../../third_party/jstemplate/compiled_resources.gyp:jstemplate' , 10 '../../../../third_party/jstemplate/compiled_resources.gyp:jstemplate' ,
11 '../../../webui/resources/js/cr.js', 11 '../../../webui/resources/js/cr.js',
12 '../../../webui/resources/js/cr/event_target.js', 12 '../../../webui/resources/js/cr/event_target.js',
13 '../../../webui/resources/js/cr/ui.js', 13 '../../../webui/resources/js/cr/ui.js',
14 '../../../webui/resources/js/cr/ui/array_data_model.js', 14 '../../../webui/resources/js/cr/ui/array_data_model.js',
15 '../../../webui/resources/js/promise_resolver.js', 15 '../../../webui/resources/js/promise_resolver.js',
16 '../../../webui/resources/js/load_time_data.js', 16 '../../../webui/resources/js/load_time_data.js',
17 '../../../webui/resources/js/util.js', 17 '../../../webui/resources/js/util.js',
18 '../../file_manager/common/js/util.js', 18 '../../file_manager/common/js/util.js',
19 '../../file_manager/common/js/async_util.js', 19 '../../file_manager/common/js/async_util.js',
20 '../../file_manager/common/js/file_type.js', 20 '../../file_manager/common/js/file_type.js',
21 '../../file_manager/common/js/metrics_base.js', 21 '../../file_manager/common/js/metrics_base.js',
22 '../../file_manager/common/js/metrics_events.js', 22 '../../file_manager/common/js/metrics_events.js',
23 '../../file_manager/common/js/metrics.js', 23 '../../file_manager/common/js/metrics.js',
24 '../../file_manager/common/js/volume_manager_common.js', 24 '../../file_manager/common/js/volume_manager_common.js',
25 '../../file_manager/background/js/app_window_wrapper.js', 25 '../../file_manager/background/js/app_window_wrapper.js',
26 '../../file_manager/background/js/background_base.js', 26 '../../file_manager/background/js/background_base.js',
27 '../../file_manager/background/js/volume_manager.js', 27 '../../file_manager/background/js/entry_location_impl.js',
28 '../../file_manager/background/js/volume_info_impl.js',
29 '../../file_manager/background/js/volume_info_list_impl.js',
30 '../../file_manager/background/js/volume_manager_factory.js',
31 '../../file_manager/background/js/volume_manager_impl.js',
32 '../../file_manager/background/js/volume_manager_util.js',
28 '../../file_manager/background/js/test_util_base.js', 33 '../../file_manager/background/js/test_util_base.js',
29 'error_util.js', 34 'error_util.js',
30 'test_util.js', 35 'test_util.js',
31 ], 36 ],
32 'externs': [ 37 'externs': [
33 '<(EXTERNS_DIR)/chrome_extensions.js', 38 '<(EXTERNS_DIR)/chrome_extensions.js',
34 '<(EXTERNS_DIR)/chrome_send.js', 39 '<(EXTERNS_DIR)/chrome_send.js',
35 '<(EXTERNS_DIR)/file_manager_private.js', 40 '<(EXTERNS_DIR)/file_manager_private.js',
36 '<(EXTERNS_DIR)/metrics_private.js', 41 '<(EXTERNS_DIR)/metrics_private.js',
37 '../../../../third_party/analytics/externs.js', 42 '../../../../third_party/analytics/externs.js',
38 '../../externs/chrome_test.js', 43 '../../externs/chrome_test.js',
44 '../../externs/entry_location.js',
39 '../../externs/es6_workaround.js', 45 '../../externs/es6_workaround.js',
40 '../../externs/platform.js', 46 '../../externs/platform.js',
41 '../../externs/volume_info.js', 47 '../../externs/volume_info.js',
48 '../../externs/volume_info_list.js',
49 '../../externs/volume_manager.js',
42 ], 50 ],
43 }, 51 },
44 'includes': [ 52 'includes': [
45 '../../compile_js.gypi', 53 '../../compile_js.gypi',
46 ], 54 ],
47 }, 55 },
48 { 56 {
49 'target_name': 'video_player', 57 'target_name': 'video_player',
50 'variables': { 58 'variables': {
51 'depends': [ 59 'depends': [
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 '../../file_manager/common/js/file_type.js', 93 '../../file_manager/common/js/file_type.js',
86 '../../file_manager/common/js/lru_cache.js', 94 '../../file_manager/common/js/lru_cache.js',
87 '../../file_manager/common/js/metrics_base.js', 95 '../../file_manager/common/js/metrics_base.js',
88 '../../file_manager/common/js/metrics_events.js', 96 '../../file_manager/common/js/metrics_events.js',
89 '../../file_manager/common/js/metrics.js', 97 '../../file_manager/common/js/metrics.js',
90 '../../file_manager/common/js/util.js', 98 '../../file_manager/common/js/util.js',
91 '../../file_manager/common/js/volume_manager_common.js', 99 '../../file_manager/common/js/volume_manager_common.js',
92 '../../file_manager/foreground/js/mouse_inactivity_watcher.js', 100 '../../file_manager/foreground/js/mouse_inactivity_watcher.js',
93 '../../file_manager/foreground/js/volume_manager_wrapper.js', 101 '../../file_manager/foreground/js/volume_manager_wrapper.js',
94 # Referenced in volume_manager_wrapper.js. 102 # Referenced in volume_manager_wrapper.js.
95 '../../file_manager/background/js/volume_manager.js',
96 '../../image_loader/image_loader_client.js', 103 '../../image_loader/image_loader_client.js',
97 'cast/cast_extension_discoverer.js', 104 'cast/cast_extension_discoverer.js',
98 'cast/cast_video_element.js', 105 'cast/cast_video_element.js',
99 'cast/media_manager.js', 106 'cast/media_manager.js',
100 'cast/caster.js', 107 'cast/caster.js',
101 'media_controls.js', 108 'media_controls.js',
102 ], 109 ],
103 'externs': [ 110 'externs': [
104 '<(EXTERNS_DIR)/chrome_extensions.js', 111 '<(EXTERNS_DIR)/chrome_extensions.js',
105 '<(EXTERNS_DIR)/chrome_send.js', 112 '<(EXTERNS_DIR)/chrome_send.js',
106 '<(EXTERNS_DIR)/file_manager_private.js', 113 '<(EXTERNS_DIR)/file_manager_private.js',
107 '<(EXTERNS_DIR)/media_player_private.js', 114 '<(EXTERNS_DIR)/media_player_private.js',
108 '<(EXTERNS_DIR)/metrics_private.js', 115 '<(EXTERNS_DIR)/metrics_private.js',
109 '../../../../third_party/analytics/externs.js', 116 '../../../../third_party/analytics/externs.js',
117 '../../externs/background_window_common.js',
110 '../../externs/chrome_test.js', 118 '../../externs/chrome_test.js',
111 '../../externs/chrome_cast.js', 119 '../../externs/chrome_cast.js',
112 '../../externs/chrome_webstore_widget_private.js', 120 '../../externs/chrome_webstore_widget_private.js',
121 '../../externs/entry_location.js',
113 '../../externs/es6_workaround.js', 122 '../../externs/es6_workaround.js',
114 '../../externs/platform.js', 123 '../../externs/platform.js',
115 '../../externs/video_player_foreground.js', 124 '../../externs/video_player_foreground.js',
116 '../../externs/volume_info.js', 125 '../../externs/volume_info.js',
126 '../../externs/volume_info_list.js',
127 '../../externs/volume_manager.js',
117 ], 128 ],
118 }, 129 },
119 'includes': [ 130 'includes': [
120 '../../compile_js.gypi', 131 '../../compile_js.gypi',
121 ], 132 ],
122 } 133 }
123 ], 134 ],
124 } 135 }
125 136
OLDNEW
« no previous file with comments | « ui/file_manager/gallery/js/compiled_resources.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698