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

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: Swap around Gyp options; use mac_breakpad_compiled_in 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
« no previous file with comments | « chrome/chrome.gyp ('k') | chrome/chrome_exe.gypi » ('j') | 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) 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 'target_name': 'chrome_dll', 9 'target_name': 'chrome_dll',
10 'type': 'shared_library', 10 'type': 'shared_library',
(...skipping 340 matching lines...) Expand 10 before | Expand all | Expand 10 after
351 { 351 {
352 # Modify the Info.plist as needed. The script explains why 352 # Modify the Info.plist as needed. The script explains why
353 # this is needed. This is also done in the chrome target. 353 # this is needed. This is also done in the chrome target.
354 # The framework needs the Breakpad keys if this feature is 354 # The framework needs the Breakpad keys if this feature is
355 # enabled. It does not need the Keystone keys; these always 355 # enabled. It does not need the Keystone keys; these always
356 # come from the outer application bundle. The framework 356 # come from the outer application bundle. The framework
357 # doesn't currently use the Subversion keys for anything, 357 # doesn't currently use the Subversion keys for anything,
358 # but this seems like a really good place to store them. 358 # but this seems like a really good place to store them.
359 'postbuild_name': 'Tweak Info.plist', 359 'postbuild_name': 'Tweak Info.plist',
360 'action': ['<(tweak_info_plist_path)', 360 'action': ['<(tweak_info_plist_path)',
361 '-b<(mac_breakpad)', 361 '--breakpad=<(mac_breakpad_compiled_in)',
362 '--breakpad_uploads=<(mac_breakpad_uploads)',
362 '-k0', 363 '-k0',
363 '-s1', 364 '-s1',
364 '<(branding)', 365 '<(branding)',
365 '<(mac_bundle_id)'], 366 '<(mac_bundle_id)'],
366 }, 367 },
367 { 368 {
368 'postbuild_name': 'Symlink Libraries', 369 'postbuild_name': 'Symlink Libraries',
369 'action': [ 370 'action': [
370 'ln', 371 'ln',
371 '-fhs', 372 '-fhs',
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
445 '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Default Apps', 446 '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Default Apps',
446 'files': ['<@(default_apps_list)'], 447 'files': ['<@(default_apps_list)'],
447 }, 448 },
448 ], 449 ],
449 }], 450 }],
450 ['mac_breakpad==1', { 451 ['mac_breakpad==1', {
451 'variables': { 452 'variables': {
452 # A real .dSYM is needed for dump_syms to operate on. 453 # A real .dSYM is needed for dump_syms to operate on.
453 'mac_real_dsym': 1, 454 'mac_real_dsym': 1,
454 }, 455 },
456 }],
457 ['mac_breakpad_compiled_in==1', {
455 'sources': [ 458 'sources': [
456 'app/breakpad_mac.mm', 459 'app/breakpad_mac.mm',
457 'app/breakpad_mac.h', 460 'app/breakpad_mac.h',
458 ], 461 ],
459 'dependencies': [ 462 'dependencies': [
460 '../breakpad/breakpad.gyp:breakpad', 463 '../breakpad/breakpad.gyp:breakpad',
461 'app/policy/cloud_policy_codegen.gyp:policy', 464 'app/policy/cloud_policy_codegen.gyp:policy',
462 ], 465 ],
463 'copies': [ 466 'copies': [
464 { 467 {
465 'destination': '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Res ources', 468 'destination': '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Res ources',
466 'files': [ 469 'files': [
467 '<(PRODUCT_DIR)/crash_inspector', 470 '<(PRODUCT_DIR)/crash_inspector',
468 '<(PRODUCT_DIR)/crash_report_sender.app' 471 '<(PRODUCT_DIR)/crash_report_sender.app'
469 ], 472 ],
470 }, 473 },
471 ], 474 ],
472 }, { # else: mac_breakpad!=1 475 }, { # else: mac_breakpad_compiled_in!=1
473 # No Breakpad, put in the stubs. 476 # No Breakpad, put in the stubs.
474 'sources': [ 477 'sources': [
475 'app/breakpad_mac_stubs.mm', 478 'app/breakpad_mac_stubs.mm',
476 'app/breakpad_mac.h', 479 'app/breakpad_mac.h',
477 ], 480 ],
478 }], # mac_breakpad 481 }], # mac_breakpad_compiled_in
479 ['mac_keystone==1', { 482 ['mac_keystone==1', {
480 'mac_bundle_resources': [ 483 'mac_bundle_resources': [
481 'browser/mac/keystone_promote_preflight.sh', 484 'browser/mac/keystone_promote_preflight.sh',
482 'browser/mac/keystone_promote_postflight.sh', 485 'browser/mac/keystone_promote_postflight.sh',
483 ], 486 ],
484 'postbuilds': [ 487 'postbuilds': [
485 { 488 {
486 'postbuild_name': 'Copy KeystoneRegistration.framework', 489 'postbuild_name': 'Copy KeystoneRegistration.framework',
487 'action': [ 490 'action': [
488 'tools/build/mac/copy_framework_unversioned', 491 'tools/build/mac/copy_framework_unversioned',
(...skipping 18 matching lines...) Expand all
507 ], 510 ],
508 }], 511 }],
509 ], # conditions 512 ], # conditions
510 }], # OS=="mac" 513 }], # OS=="mac"
511 ], # conditions 514 ], # conditions
512 }, # target chrome_dll 515 }, # target chrome_dll
513 ], # targets 516 ], # targets
514 }], # OS=="mac" or OS=="win" 517 }], # OS=="mac" or OS=="win"
515 ], 518 ],
516 } 519 }
OLDNEW
« no previous file with comments | « chrome/chrome.gyp ('k') | chrome/chrome_exe.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698