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

Unified Diff: build/common.gypi

Issue 8824003: Breakpad: Compile Breakpad into Chromium by default on Mac (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Split out mac_breakpad_symbols too Created 9 years 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 | « no previous file | chrome/chrome.gyp » ('j') | ppapi/native_client/src/trusted/plugin/plugin.gyp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 6f2d9d7cb4a024201afc8c71cc5340c36bd0366c..aa18965df9f45a61f0ede08df037dcc14cb36c74 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -720,6 +720,9 @@
['OS=="mac"', {
# Enable clang on mac by default!
'clang%': 1,
+ # Compile in Breakpad support by default so that it can be
+ # tested, even if it is not enabled by default at runtime.
+ 'mac_breakpad%': 1,
'conditions': [
# mac_product_name is set to the name of the .app bundle as it should
# appear on disk. This duplicates data from
@@ -732,14 +735,17 @@
'mac_product_name%': 'Chromium',
}],
- # Feature variables for enabling Mac Breakpad and Keystone auto-update
- # support. Both features are on by default in official builds with
- # Chrome branding.
+ # Feature variables for enabling Keystone auto-update
+ # support and enabling uploading crash dumps in Mac
+ # Breakpad. Both features are on by default in official
+ # builds with Chrome branding.
['branding=="Chrome" and buildtype=="Official"', {
- 'mac_breakpad%': 1,
+ 'mac_breakpad_uploads%': 1,
+ 'mac_breakpad_symbols%': 1,
'mac_keystone%': 1,
}, { # else: branding!="Chrome" or buildtype!="Official"
- 'mac_breakpad%': 0,
+ 'mac_breakpad_uploads%': 0,
+ 'mac_breakpad_symbols%': 0,
'mac_keystone%': 0,
}],
],
« no previous file with comments | « no previous file | chrome/chrome.gyp » ('j') | ppapi/native_client/src/trusted/plugin/plugin.gyp » ('J')

Powered by Google App Engine
This is Rietveld 408576698