Only enable breakpad and keystone for official branded builds so the rolling google chrome build doesn't spend the time doing the work for these things.

BUG=none
TEST=Official builds still auto update and have breakpad.
Review URL: http://codereview.chromium.org/155364

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20401 0039d316-1c4b-4281-b951-d872f2087c98
Esse commit está contido em:
thomasvl@chromium.org
2009-07-10 19:35:03 +00:00
commit de0e0f1bb2
+32 -22
Ver Arquivo
@@ -2601,7 +2601,6 @@
},
}],
['mac_breakpad==1', {
# Only include breakpad in official builds.
'variables': {
# A real .dSYM is needed for dump_syms to operate on.
'mac_real_dsym': 1,
@@ -3975,27 +3974,38 @@
},
],
'conditions': [
# We set feature variables so the different parts that need to check for
# the mac build use of breakpad/keystone, check that flag instead of coding
# it based on branding.
# We need the Mac app name on disk, so we stick this into a variable so
# the different places that need it can use the common variable.
# NOTE: chrome/app/theme/chromium/BRANDING and
# chrome/app/theme/google_chrome/BRANDING have the short names, etc.;
# but extracting from there still means xcodeproject are out of date until
# the next project regeneration.
['OS=="mac" and branding=="Chrome"', {
'variables': {
'mac_breakpad%': 1,
'mac_keystone%': 1,
'mac_product_name%': 'Google Chrome',
}
}, {
'variables': {
'mac_breakpad%': 0,
'mac_keystone%': 0,
'mac_product_name%': 'Chromium',
}
['OS=="mac"', {
'conditions': [
# We need the Mac app name on disk, so we stick this into a variable so
# the different places that need it can use the common variable.
# NOTE: chrome/app/theme/chromium/BRANDING and
# chrome/app/theme/google_chrome/BRANDING have the short names, etc.;
# but extracting from there still means xcodeproject are out of date until
# the next project regeneration.
['branding=="Chrome"', {
'variables': {
'mac_product_name%': 'Google Chrome',
}
}, {
'variables': {
'mac_product_name%': 'Chromium',
}
}],
# We set feature variables so the different parts of the gyp file use
# these vars in conditions instead of repeating the check of branding
# and buildtype.
['branding=="Chrome" and buildtype=="Official"', {
'variables': {
'mac_breakpad%': 1,
'mac_keystone%': 1,
}
}, {
'variables': {
'mac_breakpad%': 0,
'mac_keystone%': 0,
}
}],
],
}],
['OS=="linux"', {
'conditions': [