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

Unified Diff: components/sync_driver/BUILD.gn

Issue 1304163005: Sync more GN unit tests with GYP. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes Created 5 years, 4 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/suggestions/BUILD.gn ('k') | components/ui/zoom/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync_driver/BUILD.gn
diff --git a/components/sync_driver/BUILD.gn b/components/sync_driver/BUILD.gn
index c2fcfa990fea9919663d8583ad5eb0c713be5bbf..ec0312273a5b9dcd02dddecf6729366262b73af0 100644
--- a/components/sync_driver/BUILD.gn
+++ b/components/sync_driver/BUILD.gn
@@ -4,7 +4,7 @@
import("//build/config/features.gni")
-static_library("sync_driver") {
+source_set("sync_driver") {
sources = [
"backend_data_type_configurer.cc",
"backend_data_type_configurer.h",
@@ -98,6 +98,7 @@ static_library("sync_driver") {
"//components/history/core/browser",
"//components/invalidation/public",
"//components/os_crypt",
+ "//components/sessions",
"//components/signin/core/browser",
"//net",
"//sync",
@@ -109,41 +110,14 @@ static_library("sync_driver") {
"sync_policy_handler.cc",
"sync_policy_handler.h",
]
+ deps += [
+ "//components/policy",
+ "//components/policy:policy_component",
+ ]
}
}
-source_set("unit_tests") {
- testonly = true
- sources = [
- "data_type_manager_impl_unittest.cc",
- "device_info_data_type_controller_unittest.cc",
- "device_info_sync_service_unittest.cc",
- "generic_change_processor_unittest.cc",
- "model_association_manager_unittest.cc",
- "non_blocking_data_type_controller_unittest.cc",
- "non_ui_data_type_controller_unittest.cc",
- "shared_change_processor_unittest.cc",
- "sync_policy_handler_unittest.cc",
- "sync_prefs_unittest.cc",
- "sync_stopped_reporter_unittest.cc",
- "system_encryptor_unittest.cc",
- "tab_node_pool_unittest.cc",
- "ui_data_type_controller_unittest.cc",
- ]
-
- deps = [
- ":sync_driver",
- ":test_support",
- "//base",
- "//components/os_crypt",
- "//components/pref_registry",
- "//sync",
- "//testing/gmock",
- "//testing/gtest",
- ]
-}
-
-static_library("test_support") {
+source_set("test_support") {
testonly = true
sources = [
"change_processor_mock.cc",
@@ -166,13 +140,40 @@ static_library("test_support") {
"non_ui_data_type_controller_mock.h",
]
- deps = [
+ public_deps = [
":sync_driver",
+ "//sync",
+ ]
+
+ deps = [
"//base",
"//components/sessions",
- "//sync",
"//sync:test_support_sync_internal_api",
"//testing/gmock",
"//testing/gtest",
]
}
+
+source_set("unit_tests") {
+ testonly = true
+ sources = [
+ "data_type_manager_impl_unittest.cc",
+ "device_info_data_type_controller_unittest.cc",
+ "device_info_sync_service_unittest.cc",
+ "generic_change_processor_unittest.cc",
+ "model_association_manager_unittest.cc",
+ "non_blocking_data_type_controller_unittest.cc",
+ "non_ui_data_type_controller_unittest.cc",
+ "shared_change_processor_unittest.cc",
+ "sync_policy_handler_unittest.cc",
+ "sync_prefs_unittest.cc",
+ "system_encryptor_unittest.cc",
+ "tab_node_pool_unittest.cc",
+ "ui_data_type_controller_unittest.cc",
+ ]
+ deps = [
+ ":test_support",
+ "//testing/gmock",
+ "//testing/gtest",
+ ]
+}
« no previous file with comments | « components/suggestions/BUILD.gn ('k') | components/ui/zoom/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698