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

Side by Side Diff: content/content_renderer.gypi

Issue 393813002: content_renderer: Depend on libyuv even if WebRTC is not enabled. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | 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 (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 'dependencies': [ 6 'dependencies': [
7 'content_common_mojo_bindings', 7 'content_common_mojo_bindings',
8 '../base/base.gyp:base', 8 '../base/base.gyp:base',
9 '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_ann otations', 9 '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_ann otations',
10 '../cc/cc.gyp:cc', 10 '../cc/cc.gyp:cc',
(...skipping 835 matching lines...) Expand 10 before | Expand all | Expand 10 after
846 # Android does not build FFmpeg, which these depend on. 846 # Android does not build FFmpeg, which these depend on.
847 'renderer/pepper/video_decoder_shim.cc', 847 'renderer/pepper/video_decoder_shim.cc',
848 'renderer/pepper/video_decoder_shim.h', 848 'renderer/pepper/video_decoder_shim.h',
849 ], 849 ],
850 }], 850 }],
851 ], 851 ],
852 'dependencies': [ 852 'dependencies': [
853 '../ppapi/ppapi_internal.gyp:ppapi_host', 853 '../ppapi/ppapi_internal.gyp:ppapi_host',
854 '../ppapi/ppapi_internal.gyp:ppapi_proxy', 854 '../ppapi/ppapi_internal.gyp:ppapi_proxy',
855 '../ppapi/ppapi_internal.gyp:ppapi_shared', 855 '../ppapi/ppapi_internal.gyp:ppapi_shared',
856 '../third_party/libyuv/libyuv.gyp:libyuv',
Raphael Kubo da Costa (rakuco) 2014/07/15 12:46:26 I'm not sure depending on libyuv both here and if
bbudge 2014/07/16 01:20:12 I'm no gyp expert but it seems OK to me. But you'l
856 ], 857 ],
857 }, { # enable_plugins==0 858 }, { # enable_plugins==0
858 'sources!': [ 859 'sources!': [
859 # These are not in the plugins sources list since it also requires webrt c. 860 # These are not in the plugins sources list since it also requires webrt c.
860 'renderer/media/webrtc/video_destination_handler.cc', 861 'renderer/media/webrtc/video_destination_handler.cc',
861 'renderer/media/webrtc/video_destination_handler.h', 862 'renderer/media/webrtc/video_destination_handler.h',
862 ], 863 ],
863 }], 864 }],
864 ['enable_plugins==1 and enable_webrtc==1', { 865 ['enable_plugins==1 and enable_webrtc==1', {
865 'sources': [ 866 'sources': [
(...skipping 16 matching lines...) Expand all
882 }], 883 }],
883 ], 884 ],
884 'target_conditions': [ 885 'target_conditions': [
885 ['OS=="android"', { 886 ['OS=="android"', {
886 'sources/': [ 887 'sources/': [
887 ['include', '^renderer/render_view_linux\\.cc$'], 888 ['include', '^renderer/render_view_linux\\.cc$'],
888 ], 889 ],
889 }], 890 }],
890 ], 891 ],
891 } 892 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698