Index: build/common.gypi |
diff --git a/build/common.gypi b/build/common.gypi |
index 8aae28ea07f14161655aaf2185aa6b338db55e61..5d165c27d8c0f70dc2cea25dce738df8cd3a2f1d 100644 |
--- a/build/common.gypi |
+++ b/build/common.gypi |
@@ -17,17 +17,23 @@ |
'variables': { |
'variables': { |
'variables': { |
- # Whether we're building a ChromeOS build. |
- 'chromeos%': 0, |
+ 'variables': { |
+ # Whether we're building a ChromeOS build. |
+ 'chromeos%': 0, |
- # Whether we are using Views Toolkit |
- 'toolkit_views%': 0, |
+ # Whether or not we are using the Aura windowing framework. |
+ 'use_aura%': 0, |
- # Whether or not we are using the Aura windowing framework. |
- 'use_aura%': 0, |
+ # Whether or not we are building the Ash shell. |
+ 'use_ash%': 0, |
+ }, |
+ # Copy conditionally-set variables out one scope. |
+ 'chromeos%': '<(chromeos)', |
+ 'use_aura%': '<(use_aura)', |
+ 'use_ash%': '<(use_ash)', |
- # Whether or not we are building the Ash shell. |
- 'use_ash%': 0, |
+ # Whether we are using Views Toolkit |
+ 'toolkit_views%': 0, |
# Use OpenSSL instead of NSS. Under development: see http://crbug.com/62803 |
'use_openssl%': 0, |
@@ -57,6 +63,25 @@ |
# based on 'buildtype' (i.e. we don't care about saving symbols for |
# non-Official # builds). |
'buildtype%': 'Dev', |
+ |
+ 'conditions': [ |
+ # Chromeos implies ash. |
Mark Mentovai
2012/08/06 13:13:28
Isn’t ChromeOS spelled with more capital letters t
newt (away)
2012/08/06 18:23:55
Done.
|
+ ['chromeos==1', { |
+ 'use_ash%': 1, |
+ 'use_aura%': 1, |
+ }], |
+ |
+ # For now, Windows *AND* Linux builds that |use_aura| should also |
+ # imply using ash. This rule should be removed for the future when |
+ # both Linux and Windows are using the aura windows without the ash |
+ # interface. |
+ ['use_aura==1 and OS=="win"', { |
+ 'use_ash%': 1, |
+ }], |
+ ['use_ash==1', { |
+ 'use_aura%': 1, |
+ }], |
+ ], |
}, |
# Copy conditionally-set variables out one scope. |
'chromeos%': '<(chromeos)', |
@@ -87,23 +112,6 @@ |
'<!(uname -m | sed -e "s/i.86/ia32/;s/x86_64/x64/;s/amd64/x64/;s/arm.*/arm/;s/i86pc/ia32/")', |
}], |
- # Chromeos implies ash. |
- ['chromeos==1', { |
- 'use_ash%': 1, |
- 'use_aura%': 1, |
- }], |
- |
- # For now, Windows *AND* Linux builds that |use_aura| should also |
- # imply using ash. This rule should be removed for the future when |
- # both Linux and Windows are using the aura windows without the ash |
- # interface. |
- ['use_aura==1 and OS=="win"', { |
- 'use_ash%': 1, |
- }], |
- ['use_ash==1', { |
- 'use_aura%': 1, |
- }], |
- |
# Set default value of toolkit_views based on OS. |
['OS=="win" or chromeos==1 or use_aura==1', { |
'toolkit_views%': 1, |
@@ -111,6 +119,13 @@ |
'toolkit_views%': 0, |
}], |
+ # Set toolkit_uses_gtk for the Chromium browser on Linux. |
+ ['(OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="solaris") and use_aura==0', { |
+ 'toolkit_uses_gtk%': 1, |
+ }, { |
+ 'toolkit_uses_gtk%': 0, |
+ }], |
+ |
# Enable HiDPI on Mac OS. |
['OS=="mac"', { |
'enable_hidpi%': 1, |
@@ -127,6 +142,7 @@ |
'chromeos%': '<(chromeos)', |
'host_arch%': '<(host_arch)', |
'toolkit_views%': '<(toolkit_views)', |
+ 'toolkit_uses_gtk%': '<(toolkit_uses_gtk)', |
'use_aura%': '<(use_aura)', |
'use_ash%': '<(use_ash)', |
'use_openssl%': '<(use_openssl)', |
@@ -412,13 +428,6 @@ |
'use_x11%': 1, |
}], |
- # Set toolkit_uses_gtk for the Chromium browser on Linux. |
- ['(OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="solaris") and use_aura==0', { |
- 'toolkit_uses_gtk%': 1, |
- }, { |
- 'toolkit_uses_gtk%': 0, |
- }], |
- |
# We always use skia text rendering in Aura on Windows, since GDI |
# doesn't agree with our BackingStore. |
# TODO(beng): remove once skia text rendering is on by default. |
@@ -436,7 +445,7 @@ |
'use_gnome_keyring%': 1, |
}], |
- ['toolkit_views==0 or OS=="mac" or OS=="ios"', { |
+ ['toolkit_uses_gtk==1 or OS=="mac" or OS=="ios"', { |
# GTK+, Mac and iOS want Title Case strings |
'use_titlecase_in_grd_files%': 1, |
}], |