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

Side by Side Diff: build/config/BUILD.gn

Issue 2637853002: [iOS] Disable NSAssert and GTMDevAssert on release builds. (Closed)
Patch Set: Add comment explaining why asserts are disabled Created 3 years, 11 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/allocator.gni") 5 import("//build/config/allocator.gni")
6 import("//build/config/chrome_build.gni") 6 import("//build/config/chrome_build.gni")
7 import("//build/config/chromecast_build.gni") 7 import("//build/config/chromecast_build.gni")
8 import("//build/config/crypto.gni") 8 import("//build/config/crypto.gni")
9 import("//build/config/dcheck_always_on.gni") 9 import("//build/config/dcheck_always_on.gni")
10 import("//build/config/features.gni") 10 import("//build/config/features.gni")
(...skipping 203 matching lines...) Expand 10 before | Expand all | Expand 10 after
214 "WTF_USE_DYNAMIC_ANNOTATIONS=1", 214 "WTF_USE_DYNAMIC_ANNOTATIONS=1",
215 ] 215 ]
216 } else { 216 } else {
217 defines += [ "NVALGRIND" ] 217 defines += [ "NVALGRIND" ]
218 if (!is_nacl) { 218 if (!is_nacl) {
219 # NaCl always enables dynamic annotations. Currently this value is set to 219 # NaCl always enables dynamic annotations. Currently this value is set to
220 # 1 for all .nexes. 220 # 1 for all .nexes.
221 defines += [ "DYNAMIC_ANNOTATIONS_ENABLED=0" ] 221 defines += [ "DYNAMIC_ANNOTATIONS_ENABLED=0" ]
222 } 222 }
223 } 223 }
224
225 if (is_ios) {
226 # Disable NSAssert and GTMDevAssert (from Google Toolbox for Mac). This
227 # follows XCode's default behavior for Release builds.
228 defines += [ "NS_BLOCK_ASSERTIONS=1" ]
229 }
224 } 230 }
225 231
226 # Default libraries ------------------------------------------------------------ 232 # Default libraries ------------------------------------------------------------
227 233
228 # This config defines the default libraries applied to all targets. 234 # This config defines the default libraries applied to all targets.
229 config("default_libs") { 235 config("default_libs") {
230 if (is_win) { 236 if (is_win) {
231 # TODO(brettw) this list of defaults should probably be smaller, and 237 # TODO(brettw) this list of defaults should probably be smaller, and
232 # instead the targets that use the less common ones (e.g. wininet or 238 # instead the targets that use the less common ones (e.g. wininet or
233 # winspool) should include those explicitly. 239 # winspool) should include those explicitly.
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
393 # 399 #
394 # TODO(sof): determine VS2015 status and retire the setting from all 400 # TODO(sof): determine VS2015 status and retire the setting from all
395 # precompiled configurations. 401 # precompiled configurations.
396 cflags_c = [ "/wd4206" ] 402 cflags_c = [ "/wd4206" ]
397 } else if (is_mac) { 403 } else if (is_mac) {
398 precompiled_header = "build/precompile.h" 404 precompiled_header = "build/precompile.h"
399 precompiled_source = "//build/precompile.h" 405 precompiled_source = "//build/precompile.h"
400 } 406 }
401 } 407 }
402 } 408 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698