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

Unified Diff: third_party/ffmpeg/ffmpeg.gyp

Issue 1741010: gyp: unify toolkit_views usage (Closed)
Patch Set: Created 10 years, 8 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
« chrome/test/interactive_ui/interactive_ui_tests.gypi ('K') | « ipc/ipc.gyp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/ffmpeg/ffmpeg.gyp
diff --git a/third_party/ffmpeg/ffmpeg.gyp b/third_party/ffmpeg/ffmpeg.gyp
index 0d661a30a5b578c538ab49f05520bff9231315e0..7f428904d881bdeeb46fac2c8e46f3bf399f22ec 100644
--- a/third_party/ffmpeg/ffmpeg.gyp
+++ b/third_party/ffmpeg/ffmpeg.gyp
@@ -39,9 +39,9 @@
# Allow overridding the selection of which FFmpeg binaries to copy via an
# environment variable. Affects the ffmpeg_binaries target.
'conditions': [
- ['chromeos!=0 or toolkit_views!=0', {
+ ['chromeos==1', {
Daniel Erat 2010/04/27 01:12:25 Am I misreading this, or does it change the logic?
Evan Martin 2010/04/27 01:16:20 You are right. But I think it was wrong before.
'ffmpeg_branding%': '<(branding)OS',
- },{ # else chromeos==0, assume Chrome/Chromium.
+ }, { # otherwise, assume Chrome/Chromium.
'ffmpeg_branding%': '<(branding)',
}],
['armv7==1 and arm_neon==1', {
« chrome/test/interactive_ui/interactive_ui_tests.gypi ('K') | « ipc/ipc.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698