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

Unified Diff: apps/apps.gypi

Issue 205653002: Add apps_browsertests target (currently running 0 tests). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: win gyp fix Created 6 years, 9 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 | « apps/DEPS ('k') | apps/shell/browser/shell_browser_main_parts.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: apps/apps.gypi
diff --git a/apps/apps.gypi b/apps/apps.gypi
index 65d1e0da76806712d42f7dbd80a942a5fe8dcc36..ddbec1b5d314728f3ab1a095c382b608bfe22cd3 100644
--- a/apps/apps.gypi
+++ b/apps/apps.gypi
@@ -153,8 +153,8 @@
],
},
{
- 'target_name': 'app_shell',
- 'type': 'executable',
+ 'target_name': 'app_shell_lib',
+ 'type': 'static_library',
'defines!': ['CONTENT_IMPLEMENTATION'],
'variables': {
'chromium_code': 1,
@@ -177,7 +177,6 @@
'sources': [
'shell/app/shell_main_delegate.cc',
'shell/app/shell_main_delegate.h',
- 'shell/app/shell_main.cc',
'shell/browser/shell_app_sorting.cc',
'shell/browser/shell_app_sorting.h',
'shell/browser/shell_app_window_delegate.cc',
@@ -207,6 +206,24 @@
'shell/renderer/shell_content_renderer_client.cc',
'shell/renderer/shell_content_renderer_client.h',
],
+ },
+ {
+ 'target_name': 'app_shell',
+ 'type': 'executable',
+ 'defines!': ['CONTENT_IMPLEMENTATION'],
+ 'variables': {
+ 'chromium_code': 1,
+ },
+ 'dependencies': [
+ 'app_shell_lib',
+ 'app_shell_pak',
+ ],
+ 'include_dirs': [
+ '..',
+ ],
+ 'sources': [
+ 'shell/app/shell_main.cc',
+ ],
'conditions': [
['OS=="win"', {
'msvs_settings': {
@@ -221,7 +238,34 @@
}],
],
},
+ {
+ 'target_name': 'apps_browsertests',
+ 'type': '<(gtest_target_type)',
+ 'variables': {
+ 'chromium_code': 1,
+ },
+ 'dependencies': [
+ 'app_shell_lib',
+ # TODO(yoz): find the right deps
+ '../base/base.gyp:test_support_base',
+ '../content/content.gyp:content_app_both',
+ '../content/content_shell_and_tests.gyp:content_browser_test_support',
+ '../content/content_shell_and_tests.gyp:test_support_content',
+ '../testing/gtest.gyp:gtest',
+ ],
+ 'defines': [
+ 'HAS_OUT_OF_PROC_TEST_RUNNER',
+ ],
+ 'sources': [
+ # TODO(yoz): Refactor once we have a second test target.
+ 'test/app_shell_test.h',
+ 'test/app_shell_test.cc',
+ 'test/apps_test_launcher_delegate.cc',
+ 'test/apps_test_launcher_delegate.h',
+ 'test/apps_tests_main.cc',
+ ],
+ },
], # targets
- }], # chromeos==1
+ }], # chromeos==1 or linux aura or win aura
], # conditions
}
« no previous file with comments | « apps/DEPS ('k') | apps/shell/browser/shell_browser_main_parts.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698