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

Side by Side Diff: webkit/media/webkit_media.gypi

Issue 11316045: Add a libvpx video decoder to ClearKeyCdm and move the fake video decoder to its own class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments. Created 8 years, 1 month 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 | Annotate | Revision Log
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 'variables': { 6 'variables': {
7 'conditions': [ 7 'conditions': [
8 ['OS == "android" or OS == "ios"', { 8 ['OS == "android" or OS == "ios"', {
9 # Android and iOS don't use ffmpeg. 9 # Android and iOS don't use ffmpeg.
10 'use_ffmpeg%': 0, 10 'use_ffmpeg%': 0,
11 }, { # 'OS != "android" and OS != "ios"' 11 }, { # 'OS != "android" and OS != "ios"'
12 'use_ffmpeg%': 1, 12 'use_ffmpeg%': 1,
13 }], 13 }],
14 ], 14 ],
15 # Set |use_fake_video_decoder| to 1 to ignore input frames in |clearkeycdm|,
16 # and produce video frames filled with a solid color instead.
17 'use_fake_video_decoder%': 0,
18 # Set |use_libvpx| to 1 to use libvpx for VP8 decoding in |clearkeycdm|.
19 'use_libvpx%': 0,
15 }, 20 },
16 'targets': [ 21 'targets': [
17 { 22 {
18 'target_name': 'webkit_media', 23 'target_name': 'webkit_media',
19 'type': 'static_library', 24 'type': 'static_library',
20 'variables': { 'enable_wexit_time_destructors': 1, }, 25 'variables': { 'enable_wexit_time_destructors': 1, },
21 'include_dirs': [ 26 'include_dirs': [
22 '<(SHARED_INTERMEDIATE_DIR)', # Needed by key_systems.cc. 27 '<(SHARED_INTERMEDIATE_DIR)', # Needed by key_systems.cc.
23 ], 28 ],
24 'dependencies': [ 29 'dependencies': [
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 'sources/': [ 111 'sources/': [
107 ['exclude', '^android/'], 112 ['exclude', '^android/'],
108 ], 113 ],
109 }], 114 }],
110 ], 115 ],
111 }, 116 },
112 { 117 {
113 'target_name': 'clearkeycdm', 118 'target_name': 'clearkeycdm',
114 'type': 'none', 119 'type': 'none',
115 'conditions': [ 120 'conditions': [
116 ['use_ffmpeg == 1' , { 121 ['use_fake_video_decoder == 1' , {
122 'defines': ['CLEAR_KEY_CDM_USE_FAKE_VIDEO_DECODER'],
123 'sources': [
124 'crypto/ppapi/fake_cdm_video_decoder.cc',
125 'crypto/ppapi/fake_cdm_video_decoder.h',
126 ],
127 }],
128 ['use_ffmpeg == 1 and use_fake_video_decoder == 0' , {
ddorwin 2012/12/04 04:22:37 'and use_fake_video_decoder == 0' only applies to
Tom Finegan 2012/12/04 20:59:41 Yeah... this section is a mess now. I addressed yo
117 'defines': ['CLEAR_KEY_CDM_USE_FFMPEG_DECODER'], 129 'defines': ['CLEAR_KEY_CDM_USE_FFMPEG_DECODER'],
118 'dependencies': [ 130 'dependencies': [
119 '<(DEPTH)/third_party/ffmpeg/ffmpeg.gyp:ffmpeg', 131 '<(DEPTH)/third_party/ffmpeg/ffmpeg.gyp:ffmpeg',
120 ], 132 ],
121 'sources': [ 133 'sources': [
122 'crypto/ppapi/ffmpeg_cdm_audio_decoder.cc', 134 'crypto/ppapi/ffmpeg_cdm_audio_decoder.cc',
123 'crypto/ppapi/ffmpeg_cdm_audio_decoder.h', 135 'crypto/ppapi/ffmpeg_cdm_audio_decoder.h',
124 'crypto/ppapi/ffmpeg_cdm_video_decoder.cc', 136 'crypto/ppapi/ffmpeg_cdm_video_decoder.cc',
125 'crypto/ppapi/ffmpeg_cdm_video_decoder.h', 137 'crypto/ppapi/ffmpeg_cdm_video_decoder.h',
126 ], 138 ],
127 }], 139 }],
140 ['use_libvpx == 1 and use_fake_video_decoder == 0' , {
141 'defines': ['CLEAR_KEY_CDM_USE_LIBVPX_DECODER'],
142 'dependencies': [
143 '<(DEPTH)/third_party/libvpx/libvpx.gyp:libvpx',
144 ],
145 'sources': [
146 'crypto/ppapi/libvpx_cdm_video_decoder.cc',
147 'crypto/ppapi/libvpx_cdm_video_decoder.h',
148 ],
149 }],
128 ['os_posix == 1 and OS != "mac"', { 150 ['os_posix == 1 and OS != "mac"', {
129 'type': 'loadable_module', # Must be in PRODUCT_DIR for ASAN bots. 151 'type': 'loadable_module', # Must be in PRODUCT_DIR for ASAN bots.
130 }, { # 'os_posix != 1 or OS == "mac"' 152 }, { # 'os_posix != 1 or OS == "mac"'
131 'type': 'shared_library', 153 'type': 'shared_library',
132 }], 154 }],
133 ], 155 ],
134 'defines': ['CDM_IMPLEMENTATION'], 156 'defines': ['CDM_IMPLEMENTATION'],
135 'dependencies': [ 157 'dependencies': [
136 '<(DEPTH)/base/base.gyp:base', 158 '<(DEPTH)/base/base.gyp:base',
137 '<(DEPTH)/media/media.gyp:media', 159 '<(DEPTH)/media/media.gyp:media',
138 ], 160 ],
139 'sources': [ 161 'sources': [
162 'crypto/ppapi/cdm_video_decoder.cc',
163 'crypto/ppapi/cdm_video_decoder.h',
140 'crypto/ppapi/clear_key_cdm.cc', 164 'crypto/ppapi/clear_key_cdm.cc',
141 'crypto/ppapi/clear_key_cdm.h', 165 'crypto/ppapi/clear_key_cdm.h',
142 ], 166 ],
143 }, 167 },
144 { 168 {
145 'target_name': 'clearkeycdmplugin', 169 'target_name': 'clearkeycdmplugin',
146 'type': 'none', 170 'type': 'none',
147 'dependencies': [ 171 'dependencies': [
148 '<(DEPTH)/ppapi/ppapi.gyp:ppapi_cpp', 172 '<(DEPTH)/ppapi/ppapi.gyp:ppapi_cpp',
149 'clearkeycdm', 173 'clearkeycdm',
(...skipping 26 matching lines...) Expand all
176 # Not to strip important symbols by -Wl,-dead_strip. 200 # Not to strip important symbols by -Wl,-dead_strip.
177 '-Wl,-exported_symbol,_PPP_GetInterface', 201 '-Wl,-exported_symbol,_PPP_GetInterface',
178 '-Wl,-exported_symbol,_PPP_InitializeModule', 202 '-Wl,-exported_symbol,_PPP_InitializeModule',
179 '-Wl,-exported_symbol,_PPP_ShutdownModule' 203 '-Wl,-exported_symbol,_PPP_ShutdownModule'
180 ]}, 204 ]},
181 }], 205 }],
182 ], 206 ],
183 } 207 }
184 ], 208 ],
185 } 209 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698