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

Side by Side Diff: chrome/chrome_dll.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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2011 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 'conditions': [ 5 'conditions': [
6 ['OS=="mac" or OS=="win"', { 6 ['OS=="mac" or OS=="win"', {
7 'targets': [ 7 'targets': [
8 { 8 {
9 'variables': { 9 'variables': {
10 'conditions' : [ 10 'conditions' : [
(...skipping 348 matching lines...) Expand 10 before | Expand all | Expand 10 after
359 { 359 {
360 # Modify the Info.plist as needed. The script explains why 360 # Modify the Info.plist as needed. The script explains why
361 # this is needed. This is also done in the chrome target. 361 # this is needed. This is also done in the chrome target.
362 # The framework needs the Breakpad keys if this feature is 362 # The framework needs the Breakpad keys if this feature is
363 # enabled. It does not need the Keystone keys; these always 363 # enabled. It does not need the Keystone keys; these always
364 # come from the outer application bundle. The framework 364 # come from the outer application bundle. The framework
365 # doesn't currently use the Subversion keys for anything, 365 # doesn't currently use the Subversion keys for anything,
366 # but this seems like a really good place to store them. 366 # but this seems like a really good place to store them.
367 'postbuild_name': 'Tweak Info.plist', 367 'postbuild_name': 'Tweak Info.plist',
368 'action': ['<(tweak_info_plist_path)', 368 'action': ['<(tweak_info_plist_path)',
369 '-b<(mac_breakpad)', 369 '--breakpad=<(mac_breakpad)',
370 '--breakpad_uploads=<(mac_breakpad_uploads)',
370 '-k0', 371 '-k0',
371 '-s1', 372 '-s1',
372 '<(branding)', 373 '<(branding)',
373 '<(mac_bundle_id)'], 374 '<(mac_bundle_id)'],
374 }, 375 },
375 { 376 {
376 'postbuild_name': 'Symlink Libraries', 377 'postbuild_name': 'Symlink Libraries',
377 'action': [ 378 'action': [
378 'ln', 379 'ln',
379 '-fhs', 380 '-fhs',
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
450 # This location is for the Mac build. Note that the 451 # This location is for the Mac build. Note that the
451 # copying of these files for Windows and Linux is handled 452 # copying of these files for Windows and Linux is handled
452 # in chrome.gyp, as Mac needs to be dropped inside the 453 # in chrome.gyp, as Mac needs to be dropped inside the
453 # framework. 454 # framework.
454 'destination': 455 'destination':
455 '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Default Apps', 456 '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Default Apps',
456 'files': ['<@(default_apps_list)'], 457 'files': ['<@(default_apps_list)'],
457 }, 458 },
458 ], 459 ],
459 }], 460 }],
460 ['mac_breakpad==1', { 461 ['mac_breakpad_symbols==1', {
461 'variables': { 462 'variables': {
462 # A real .dSYM is needed for dump_syms to operate on. 463 # A real .dSYM is needed for dump_syms to operate on.
463 'mac_real_dsym': 1, 464 'mac_real_dsym': 1,
464 }, 465 },
466 }],
467 ['mac_breakpad==1', {
465 'sources': [ 468 'sources': [
466 'app/breakpad_mac.mm', 469 'app/breakpad_mac.mm',
467 'app/breakpad_mac.h', 470 'app/breakpad_mac.h',
468 ], 471 ],
469 'dependencies': [ 472 'dependencies': [
470 '../breakpad/breakpad.gyp:breakpad', 473 '../breakpad/breakpad.gyp:breakpad',
471 'app/policy/cloud_policy_codegen.gyp:policy', 474 'app/policy/cloud_policy_codegen.gyp:policy',
472 ], 475 ],
473 'copies': [ 476 'copies': [
474 { 477 {
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
517 ], 520 ],
518 }], 521 }],
519 ], # conditions 522 ], # conditions
520 }], # OS=="mac" 523 }], # OS=="mac"
521 ], # conditions 524 ], # conditions
522 }, # target chrome_dll 525 }, # target chrome_dll
523 ], # targets 526 ], # targets
524 }], # OS=="mac" or OS=="win" 527 }], # OS=="mac" or OS=="win"
525 ], 528 ],
526 } 529 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698