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

Unified Diff: components/components_tests.gyp

Issue 963683003: Enable on iOS all components unittests for components that are built (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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 | « components/components.gyp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/components_tests.gyp
diff --git a/components/components_tests.gyp b/components/components_tests.gyp
index 9bb9e84c6bd1bb94fedfa948b77452b4d8c04cfb..4813e2c8c12d66dd27638c1ef8fe886311abb473 100644
--- a/components/components_tests.gyp
+++ b/components/components_tests.gyp
@@ -531,43 +531,87 @@
'webp_transcode/webp_decoder_unittest.mm',
],
'sources/': [
+ # First start by excluding all tests on iOS.
['exclude', '\\.cc$'],
['exclude', '\\.mm$'],
+
+ # The include the test runner.
blundell 2015/03/02 10:52:05 s/The//
sdefresne 2015/03/02 15:13:35 Done.
['include', '^test/run_all_unittests\\.cc$'],
+
+ # Include tests for all components that are built on iOS.
['include', '^auto_login_parser/'],
- ['include', '^autofill/core/'],
+ ['include', '^autofill/'],
['include', '^bookmarks/'],
+ ['include', '^captive_portal/'],
+ ['include', '^cloud_devices/'],
['include', '^component_updater/'],
- ['include', '^crash/'],
['include', '^content_settings/'],
+ ['include', '^crash/'],
+ ['include', '^cronet/'],
+ ['include', '^crx_file/'],
['include', '^data_reduction_proxy/'],
+ ['include', '^device_event_log/'],
['include', '^dom_distiller/'],
+ ['include', '^domain_reliability/'],
['include', '^enhanced_bookmarks/'],
- ['include', '^gcm_driver/'],
+ ['include', '^error_page/'],
+ ['include', '^favicon/'],
+ ['include', '^favicon_base/'],
+ ['include', '^gcm_driver'],
blundell 2015/03/02 10:52:05 why remove the "/" here and on some other ones?
sdefresne 2015/03/02 15:13:35 It was a mistake. Fixed, thank you.
['include', '^google/'],
+ ['include', '^handoff/'],
['include', '^history/'],
- ['include', '^invalidation/'],
+ ['include', '^infobars/'],
+ ['include', '^invalidation'],
['include', '^json_schema/'],
- ['include', '^keyed_service/core/'],
+ ['include', '^keyed_service/'],
['include', '^language_usage_metrics/'],
['include', '^leveldb_proto/'],
+ ['include', '^login/'],
['include', '^metrics/'],
+ ['include', '^navigation_metrics/'],
+ ['include', '^network_hints/'],
['include', '^network_time/'],
+ ['include', '^omnibox'],
+ ['include', '^onc/'],
+ ['include', '^open_from_clipboard'],
+ ['include', '^os_crypt/'],
+ ['include', '^ownership/'],
+ ['include', '^packed_ct_ev_whitelist/'],
['include', '^password_manager/'],
- ['include', '^precache/core/'],
+ ['include', '^policy/'],
+ ['include', '^precache/'],
+ ['include', '^pref_registry/'],
['include', '^query_parser/'],
+ ['include', '^rappor/'],
+ ['include', '^renderer_context_menu'],
['include', '^search/'],
- ['include', '^search_engines/'],
+ ['include', '^search_engines'],
['include', '^search_provider_logos/'],
- ['include', '^sessions/ios/'],
- ['include', '^sessions/serialized_navigation_entry_unittest\\.cc$'],
- ['exclude', '^signin/core/browser/mutable_profile_oauth2_token_service_unittest\\.cc$'],
+ ['include', '^sessions/'],
+ ['include', '^signin/'],
+ ['include', '^startup_metric_utils/'],
['include', '^suggestions/'],
- ['include', '^sync_driver/'],
+ ['include', '^sync_driver'],
['include', '^translate/'],
+ ['include', '^ui_zoom/'],
+ ['include', '^update_client/'],
['include', '^url_fixer/'],
+ ['include', '^url_matcher/'],
+ ['include', '^user_prefs/'],
['include', '^variations/'],
+ ['include', '^wallpaper/'],
+ ['include', '^web_resource/'],
+ ['include', '^webdata/'],
+ ['include', '^webdata_services/'],
['include', '^webp_transcode/'],
+
+ # Exclude individual tests that are known not to work on iOS.
+ ['exclude', '^signin/core/browser/mutable_profile_oauth2_token_service_unittest\\.cc$'],
+
+ # Exclude all tests that depends on //content (based on layered-
+ # component directory structure).
+ ['exclude', '^[^/]*/content/'],
],
'dependencies': [
'../ios/ios_tests.gyp:test_support_ios',
« no previous file with comments | « components/components.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698