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

Unified Diff: chrome/chrome_tests.gypi

Issue 2802015: Massively simplify the NetworkChangeNotifier infrastructure:... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 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
Index: chrome/chrome_tests.gypi
===================================================================
--- chrome/chrome_tests.gypi (revision 50775)
+++ chrome/chrome_tests.gypi (working copy)
@@ -504,7 +504,6 @@
'chrome_resources',
'chrome_strings',
'common',
- 'common_net_test_support',
'debugger',
'profile_import',
'renderer',
@@ -948,7 +947,6 @@
'browser/sync/glue/theme_util_unittest.cc',
'browser/sync/glue/typed_url_model_associator_unittest.cc',
'browser/sync/glue/ui_model_worker_unittest.cc',
- 'browser/sync/net/network_change_notifier_io_thread_unittest.cc',
'browser/sync/profile_sync_factory_impl_unittest.cc',
'browser/sync/profile_sync_factory_mock.cc',
'browser/sync/profile_sync_factory_mock.h',
@@ -1014,7 +1012,6 @@
'common/json_pref_store_unittest.cc',
'common/json_value_serializer_unittest.cc',
'common/mru_cache_unittest.cc',
- # TODO(sanjeevr): Move the 2 below files to common_net_unit_tests
'common/net/gaia/gaia_authenticator_unittest.cc',
'common/net/url_fetcher_unittest.cc',
'common/net/test_url_fetcher_factory.cc',
@@ -1055,7 +1052,6 @@
'renderer/spellchecker/spellcheck_worditerator_unittest.cc',
'renderer/translate_helper_unittest.cc',
'service/cloud_print/cloud_print_helpers_unittest.cc',
- 'service/net/service_network_change_notifier_thread_unittest.cc',
'test/browser_with_test_window_test.cc',
'test/browser_with_test_window_test.h',
'test/file_test_utils.cc',
@@ -1773,58 +1769,6 @@
], # conditions
},
{
- 'target_name': 'common_net_test_support',
- 'type': '<(library)',
- 'sources': [
- 'common/net/fake_network_change_notifier_thread.cc',
- 'common/net/fake_network_change_notifier_thread.h',
- 'common/net/thread_blocker.cc',
- 'common/net/thread_blocker.h',
- ],
- 'dependencies': [
- 'common_net',
- '../base/base.gyp:base',
- '../net/net.gyp:net_base',
- ],
- },
- {
- # TODO(akalin): Add this to all.gyp.
- 'target_name': 'common_net_unit_tests',
- 'type': 'executable',
- 'sources': [
- # TODO(akalin): Write our own test suite and runner.
- '../base/test/run_all_unittests.cc',
- '../base/test/test_suite.h',
- 'common/net/fake_network_change_notifier_thread_unittest.cc',
- 'common/net/mock_network_change_observer.h',
- 'common/net/network_change_notifier_proxy_unittest.cc',
- 'common/net/network_change_observer_proxy_unittest.cc',
- 'common/net/thread_blocker_unittest.cc',
- ],
- 'include_dirs': [
- '..',
- ],
- 'dependencies': [
- 'common_net',
- 'common_net_test_support',
- '../build/temp_gyp/googleurl.gyp:googleurl',
- '../testing/gmock.gyp:gmock',
- '../testing/gtest.gyp:gtest',
- ],
- # TODO(akalin): Remove this once we have our own test suite and
- # runner.
- 'conditions': [
- ['OS == "linux" or OS == "freebsd" or OS == "openbsd" or OS == "solaris"', {
- 'dependencies': [
- # Needed to handle the #include chain:
- # base/test/test_suite.h
- # gtk/gtk.h
- '../build/linux/system.gyp:gtk',
- ],
- }],
- ],
- },
- {
'target_name': 'notifier_unit_tests',
'type': 'executable',
'sources': [
@@ -1840,7 +1784,6 @@
'..',
],
'dependencies': [
- 'common_net_test_support',
'notifier',
'../base/base.gyp:base',
'../testing/gmock.gyp:gmock',
@@ -1915,7 +1858,6 @@
'dependencies': [
'browser/sync/protocol/sync_proto.gyp:sync_proto_cpp',
'common',
- 'common_net_test_support',
'debugger',
'../skia/skia.gyp:skia',
'../testing/gmock.gyp:gmock',

Powered by Google App Engine
This is Rietveld 408576698