Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 Loading... | |
| 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 } |
| OLD | NEW |