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

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

Issue 10899021: Add CDM video decoder. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ffmpeg decoder ifdefs and more gyp changes. 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 'targets': [ 6 'targets': [
7 { 7 {
8 'target_name': 'webkit_media', 8 'target_name': 'webkit_media',
9 'type': 'static_library', 9 'type': 'static_library',
10 'variables': { 'enable_wexit_time_destructors': 1, }, 10 'variables': { 'enable_wexit_time_destructors': 1, },
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 }, { # OS != "android"' 95 }, { # OS != "android"'
96 'sources/': [ 96 'sources/': [
97 ['exclude', '^android/'], 97 ['exclude', '^android/'],
98 ], 98 ],
99 }], 99 }],
100 ], 100 ],
101 }, 101 },
102 { 102 {
103 'target_name': 'clearkeycdm', 103 'target_name': 'clearkeycdm',
104 'type': 'shared_library', 104 'type': 'shared_library',
105 'conditions': [
106 ['OS == "android" or OS == "ios"', {
ddorwin 2012/10/23 00:32:30 indent 2 more
Tom Finegan 2012/10/23 00:54:41 Done.
107 # Android and iOS don't use ffmpeg.
108 'use_ffmpeg%': 0,
ddorwin 2012/10/23 00:32:30 I think we should just do the OS check here then s
Tom Finegan 2012/10/23 00:54:41 I might have been doing something wrong, but to us
ddorwin 2012/10/23 01:02:55 Looks like the code I linked to, so LG.
109 'defines': ['CLEAR_KEY_CDM_USE_FAKE_VIDEO_DECODER'],
ddorwin 2012/10/23 00:32:30 Remove. We should just return error in InitializeD
Tom Finegan 2012/10/23 00:54:41 Done.
110 }, { # 'OS != "android" and OS != "ios"'
111 'use_ffmpeg%': 1,
112 'defines': ['CLEAR_KEY_CDM_USE_FFMPEG_DECODER'],
113 'dependencies': [
114 '<(DEPTH)/third_party/ffmpeg/ffmpeg.gyp:ffmpeg',
115 ],
116 'sources': [
117 'crypto/ppapi/ffmpeg_cdm_video_decoder.cc',
118 'crypto/ppapi/ffmpeg_cdm_video_decoder.h',
119 ],
120 }],
121 ],
105 'defines': ['CDM_IMPLEMENTATION'], 122 'defines': ['CDM_IMPLEMENTATION'],
106 'dependencies': [ 123 'dependencies': [
107 '<(DEPTH)/base/base.gyp:base', 124 '<(DEPTH)/base/base.gyp:base',
108 '<(DEPTH)/media/media.gyp:media' 125 '<(DEPTH)/media/media.gyp:media',
109 ], 126 ],
110 'sources': [ 127 'sources': [
111 'crypto/ppapi/clear_key_cdm.cc', 128 'crypto/ppapi/clear_key_cdm.cc',
112 'crypto/ppapi/clear_key_cdm.h', 129 'crypto/ppapi/clear_key_cdm.h',
113 ], 130 ],
114 }, 131 },
115 { 132 {
116 'target_name': 'clearkeycdmplugin', 133 'target_name': 'clearkeycdmplugin',
117 'type': 'none', 134 'type': 'none',
118 'dependencies': [ 135 'dependencies': [
119 '<(DEPTH)/ppapi/ppapi.gyp:ppapi_cpp', 136 '<(DEPTH)/ppapi/ppapi.gyp:ppapi_cpp',
120 'clearkeycdm', 137 'clearkeycdm',
121 ], 138 ],
122 'sources': [ 139 'sources': [
123 'crypto/ppapi/cdm_wrapper.cc', 140 'crypto/ppapi/cdm_wrapper.cc',
124 'crypto/ppapi/content_decryption_module.h', 141 'crypto/ppapi/content_decryption_module.h',
125 'crypto/ppapi/linked_ptr.h', 142 'crypto/ppapi/linked_ptr.h',
126 ], 143 ],
127 'conditions': [ 144 'conditions': [
128 ['os_posix==1 and OS!="mac"', { 145 # TODO(tomfinegan): Remove this hack that disables this target on
146 # android and ios, and include these targets only in desktop builds.
147 ['os_posix == 1 and OS != "mac" and OS != "ios" and OS != "android"', {
ddorwin 2012/10/23 00:32:30 Do you still need this? Shouldn't it all be fixed
Tom Finegan 2012/10/23 00:54:41 Done. Forgot to remove.
129 'cflags': ['-fvisibility=hidden'], 148 'cflags': ['-fvisibility=hidden'],
130 'type': 'loadable_module', 149 'type': 'loadable_module',
131 # -gstabs, used in the official builds, causes an ICE. Simply remove 150 # -gstabs, used in the official builds, causes an ICE. Simply remove
132 # it. 151 # it.
133 'cflags!': ['-gstabs'], 152 'cflags!': ['-gstabs'],
134 # Allow the plugin wrapper to find the CDM in the same directory. 153 # Allow the plugin wrapper to find the CDM in the same directory.
135 'ldflags': ['-Wl,-rpath=\$$ORIGIN'] 154 'ldflags': ['-Wl,-rpath=\$$ORIGIN']
136 }], 155 }],
137 ['OS=="win"', { 156 ['OS=="win"', {
138 'type': 'shared_library', 157 'type': 'shared_library',
139 }], 158 }],
140 ['OS=="mac"', { 159 ['OS=="mac"', {
141 'type': 'loadable_module', 160 'type': 'loadable_module',
142 'mac_bundle': 1, 161 'mac_bundle': 1,
143 'product_extension': 'plugin', 162 'product_extension': 'plugin',
144 'xcode_settings': { 163 'xcode_settings': {
145 'OTHER_LDFLAGS': [ 164 'OTHER_LDFLAGS': [
146 # Not to strip important symbols by -Wl,-dead_strip. 165 # Not to strip important symbols by -Wl,-dead_strip.
147 '-Wl,-exported_symbol,_PPP_GetInterface', 166 '-Wl,-exported_symbol,_PPP_GetInterface',
148 '-Wl,-exported_symbol,_PPP_InitializeModule', 167 '-Wl,-exported_symbol,_PPP_InitializeModule',
149 '-Wl,-exported_symbol,_PPP_ShutdownModule' 168 '-Wl,-exported_symbol,_PPP_ShutdownModule'
150 ]}, 169 ]},
151 }], 170 }],
152 ], 171 ],
153 } 172 }
154 ], 173 ],
155 } 174 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698