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

Unified Diff: components/breakpad.gypi

Issue 327853002: Move the CrashKeysWin class to a pair of files as first step. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge refactor and testing back in. Created 6 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | components/breakpad/app/breakpad_win.cc » ('j') | components/breakpad/app/crash_keys_win.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/breakpad.gypi
diff --git a/components/breakpad.gypi b/components/breakpad.gypi
index 9dcf912411cd285612ee9bda0d8795088e85f39d..98bfe785a83ed9d12c0673a06711a1a11c866d0e 100644
--- a/components/breakpad.gypi
+++ b/components/breakpad.gypi
@@ -3,42 +3,43 @@
# found in the LICENSE file.
{
- 'target_defaults': {
- 'variables': {
- 'breakpad_component_target': 0,
- },
- 'target_conditions': [
- ['breakpad_component_target==1', {
- 'defines': ['BREAKPAD_IMPLEMENTATION'],
- 'sources': [
- 'breakpad/app/breakpad_client.cc',
- 'breakpad/app/breakpad_client.h',
- 'breakpad/app/breakpad_linux.cc',
- 'breakpad/app/breakpad_linux.h',
- 'breakpad/app/breakpad_linux_impl.h',
- 'breakpad/app/breakpad_mac.h',
- 'breakpad/app/breakpad_mac.mm',
- 'breakpad/app/breakpad_win.cc',
- 'breakpad/app/breakpad_win.h',
- 'breakpad/app/hard_error_handler_win.cc',
- 'breakpad/app/hard_error_handler_win.h',
- ],
- }],
- ],
- },
'targets': [
{
+ 'target_name': 'breakpad_component_lib',
+ 'type': 'static_library',
+ 'sources': [
+ 'breakpad/app/breakpad_client.cc',
+ 'breakpad/app/breakpad_client.h',
+ 'breakpad/app/crash_keys_win.cc',
+ 'breakpad/app/crash_keys_win.h',
+ ],
+ 'include_dirs': [
+ '..',
+ '../breakpad/src',
+ ],
+ },
+ {
# Note: if you depend on this target, you need to either link in
# content.gyp:content_common, or add
# content/public/common/content_switches.cc to your sources.
'target_name': 'breakpad_component',
'type': 'static_library',
- 'variables': {
- 'breakpad_component_target': 1,
- },
+ 'sources': [
+ 'breakpad/app/breakpad_linux.cc',
+ 'breakpad/app/breakpad_linux.h',
+ 'breakpad/app/breakpad_linux_impl.h',
+ 'breakpad/app/breakpad_mac.h',
+ 'breakpad/app/breakpad_mac.mm',
+ 'breakpad/app/breakpad_win.cc',
+ 'breakpad/app/breakpad_win.h',
+ 'breakpad/app/hard_error_handler_win.cc',
+ 'breakpad/app/hard_error_handler_win.h',
+ ],
'dependencies': [
+ 'breakpad_component_lib',
'../base/base.gyp:base',
],
+ 'defines': ['BREAKPAD_IMPLEMENTATION'],
'conditions': [
['OS=="mac"', {
'dependencies': [
@@ -71,6 +72,18 @@
}],
],
},
+ {
+ 'target_name': 'breakpad_test_support',
+ 'type': 'none',
+ 'dependencies': [
+ 'breakpad_component_lib',
+ ],
+ 'direct_dependent_settings': {
+ 'include_dirs' : [
+ '../breakpad/src',
+ ],
+ }
+ },
],
'conditions': [
['OS=="win"', {
@@ -98,11 +111,25 @@
# content/public/common/content_switches.cc to your sources.
'target_name': 'breakpad_win64',
'type': 'static_library',
- 'variables': {
- 'breakpad_component_target': 1,
- },
+ 'sources': [
+ 'breakpad/app/breakpad_client.cc',
+ 'breakpad/app/breakpad_client.h',
+ 'breakpad/app/breakpad_linux.cc',
+ 'breakpad/app/breakpad_linux.h',
+ 'breakpad/app/breakpad_linux_impl.h',
+ 'breakpad/app/breakpad_mac.h',
+ 'breakpad/app/breakpad_mac.mm',
+ 'breakpad/app/breakpad_win.cc',
+ 'breakpad/app/breakpad_win.h',
+ # TODO(siggi): test the x64 version too.
+ 'breakpad/app/crash_keys_win.cc',
+ 'breakpad/app/crash_keys_win.h',
+ 'breakpad/app/hard_error_handler_win.cc',
+ 'breakpad/app/hard_error_handler_win.h',
+ ],
'defines': [
'COMPILE_CONTENT_STATICALLY',
+ 'BREAKPAD_IMPLEMENTATION',
],
'dependencies': [
'../base/base.gyp:base_win64',
« no previous file with comments | « no previous file | components/breakpad/app/breakpad_win.cc » ('j') | components/breakpad/app/crash_keys_win.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698