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

Unified Diff: device/geolocation/BUILD.gn

Issue 2161223003: Adds GeolocationFeature for Blimp Geolocation project. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@engine_feature_prep
Patch Set: More merging. Created 4 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
Index: device/geolocation/BUILD.gn
diff --git a/device/geolocation/BUILD.gn b/device/geolocation/BUILD.gn
index 0e4c4357bb0de899fb081f6616831d95832a70c4..58bc49cceb5cd2d36ccfc2d0f74e09663c029ae9 100644
--- a/device/geolocation/BUILD.gn
+++ b/device/geolocation/BUILD.gn
@@ -146,16 +146,32 @@ if (is_android) {
}
}
-source_set("unittests") {
+source_set("test_support") {
testonly = true
sources = [
"fake_access_token_store.cc",
"fake_access_token_store.h",
- "geolocation_provider_impl_unittest.cc",
- "location_arbitrator_impl_unittest.cc",
+ "fake_location_provider.cc",
+ "fake_location_provider.h",
"mock_location_provider.cc",
"mock_location_provider.h",
+ ]
+ public_deps = [
+ ":device_geolocation",
+ ]
+ deps = [
+ "//testing/gmock",
+ "//testing/gtest",
+ ]
+}
+
+source_set("unittests") {
+ testonly = true
+
+ sources = [
+ "geolocation_provider_impl_unittest.cc",
+ "location_arbitrator_impl_unittest.cc",
"network_location_provider_unittest.cc",
"wifi_data_provider_chromeos_unittest.cc",
"wifi_data_provider_common_unittest.cc",
@@ -166,6 +182,7 @@ source_set("unittests") {
":device_geolocation",
]
deps = [
+ ":test_support",
"//base",
"//base/third_party/dynamic_annotations",
"//net:test_support",
« no previous file with comments | « blimp/engine/feature/geolocation/mock_blimp_location_provider_delegate.h ('k') | device/geolocation/fake_location_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698