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

Unified Diff: content/browser/BUILD.gn

Issue 2192683003: Revert of Reland: Geolocation: move from content/browser to device/ (patchset #2 id:20001 of https:… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2810
Patch Set: Created 4 years, 5 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 | « content/app/android/library_loader_hooks.cc ('k') | content/browser/DEPS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/BUILD.gn
diff --git a/content/browser/BUILD.gn b/content/browser/BUILD.gn
index 6f43a3fb6afab01b0915bb2d9d92e75b833839b1..35b2f198f77181648b747a85d1b78592258bec0e 100644
--- a/content/browser/BUILD.gn
+++ b/content/browser/BUILD.gn
@@ -63,7 +63,6 @@ source_set("browser") {
"//device/battery",
"//device/bluetooth",
"//device/gamepad",
- "//device/geolocation:device_geolocation",
"//device/power_save_blocker",
"//device/vibration",
"//gin",
@@ -212,7 +211,10 @@ source_set("browser") {
}
if (is_win) {
- sources -= [ "device_sensors/data_fetcher_shared_memory_default.cc" ]
+ sources -= [
+ "device_sensors/data_fetcher_shared_memory_default.cc",
+ "geolocation/empty_wifi_data_provider.cc",
+ ]
defines += [
# This prevents the inclusion of atlhost.h which paired
# with the windows 8 sdk it does the wrong thing.
@@ -291,6 +293,18 @@ source_set("browser") {
deps += [ "//ui/gfx/x" ]
}
+ # Dealing with *wifi_data_provider_*.cc is also a bit complicated given
+ # android, chromeos, linux and use_dbus.
+ if (is_android) {
+ sources -= [ "geolocation/wifi_data_provider_common.cc" ]
+ }
+ if (is_chromeos || (is_linux && !use_dbus)) {
+ sources -= [ "geolocation/wifi_data_provider_linux.cc" ]
+ }
+ if (is_linux && use_dbus) {
+ sources -= [ "geolocation/empty_wifi_data_provider.cc" ]
+ }
+
if (use_pango) {
configs += [ "//build/config/linux/pangocairo" ]
}
@@ -302,6 +316,10 @@ source_set("browser") {
sources -= [
"browser_ipc_logging.cc",
"device_sensors/data_fetcher_shared_memory_default.cc",
+ "geolocation/network_location_provider.cc",
+ "geolocation/network_location_provider.h",
+ "geolocation/network_location_request.cc",
+ "geolocation/network_location_request.h",
"media/session/media_session_delegate_default.cc",
"power_usage_monitor_impl.cc",
"power_usage_monitor_impl.h",
@@ -341,7 +359,11 @@ source_set("browser") {
}
if (is_mac) {
- sources -= [ "device_sensors/data_fetcher_shared_memory_default.cc" ]
+ sources -= [
+ "device_sensors/data_fetcher_shared_memory_default.cc",
+ "geolocation/empty_wifi_data_provider.cc",
+ "geolocation/empty_wifi_data_provider.h",
+ ]
deps += [
"//media",
"//sandbox/mac:sandbox",
« no previous file with comments | « content/app/android/library_loader_hooks.cc ('k') | content/browser/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698