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

Unified Diff: content/content_browser.gypi

Issue 23444072: Use libyuv in Android VideoCaptureController (not for Android WebView) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed wjia@'s comments. Created 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/renderer_host/media/video_capture_controller.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/content_browser.gypi
diff --git a/content/content_browser.gypi b/content/content_browser.gypi
index f62174e0bc4220c3afc3e3b435768880b5e3c01e..87e704adcc494657cfd424386765b95b0972653f 100644
--- a/content/content_browser.gypi
+++ b/content/content_browser.gypi
@@ -1288,9 +1288,17 @@
['OS!="android" and OS!="ios"', {
'dependencies': [
'browser/tracing/tracing_resources.gyp:tracing_resources',
+ ],
+ }],
+ ['android_webview_build==0 and OS!="ios"', {
+ 'dependencies': [
'../third_party/libyuv/libyuv.gyp:libyuv',
],
}],
+ ['android_webview_build==0', {
+ }, { # android_webview_build!=0
jam 2013/09/27 17:55:43 why not just android_webview_build==1?
mcasas 2013/09/30 17:57:46 The only explicit setting of webview is =0 at [1]
wjia(left Chromium) 2013/09/30 18:14:25 I think jam might also mean using android_webiew_b
jam 2013/10/02 21:13:56 i don't understand what you mean. why would it be
+ 'defines': ['AVOID_LIBYUV_FOR_ANDROID_WEBVIEW'],
+ }],
['enable_webrtc==1', {
'dependencies': [
'../jingle/jingle.gyp:jingle_glue',
« no previous file with comments | « content/browser/renderer_host/media/video_capture_controller.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698