Upstream Android Geolocation

content/content_browser.gypi

Index: content/content_browser.gypi
diff --git a/content/content_browser.gypi b/content/content_browser.gypi
index 1ef5f79ee793a376f092c8a757ed0551cf0bc4d8..1af92be1460f72936330a5345a1f5a264d8992db 100644
--- a/content/content_browser.gypi
+++ b/content/content_browser.gypi
@@ -308,6 +308,8 @@
     'browser/gamepad/platform_data_fetcher_mac.mm',
     'browser/gamepad/platform_data_fetcher_win.cc',
     'browser/gamepad/platform_data_fetcher_win.h',
+    'browser/geolocation/android_location_api_adapter.cc',
+    'browser/geolocation/android_location_api_adapter.h',
     'browser/geolocation/arbitrator_dependency_factory.cc',
     'browser/geolocation/arbitrator_dependency_factory.h',
     'browser/geolocation/core_location_data_provider_mac.h',
@@ -331,6 +333,8 @@
     'browser/geolocation/location_arbitrator.h',
     'browser/geolocation/location_provider.cc',
     'browser/geolocation/location_provider.h',
+    'browser/geolocation/location_provider_android.cc',
+    'browser/geolocation/location_provider_android.h',
     'browser/geolocation/network_location_provider.cc',
     'browser/geolocation/network_location_provider.h',
     'browser/geolocation/network_location_request.cc',
@@ -801,10 +805,26 @@
         # A main loop would be handled in Java.
         'browser/browser_main_loop.cc',
         'browser/browser_main_loop.h',
+        'browser/geolocation/network_location_provider.cc',
+        'browser/geolocation/network_location_provider.h',
+        'browser/geolocation/network_location_request.cc',
+        'browser/geolocation/network_location_request.h',
+      ],
+      'dependencies': [
+        'content.gyp:content_jni_headers',
+        'content.gyp:content_java',
       ],
       'dependencies!': [
         '../third_party/flac/flac.gyp:libflac',
-      ]}],
+      ],
+    }, { # OS!="android"

[Yaron, 2012/04/24 17:10:23]

This is unnecessary as files with _android are omitted from other builds. See
build/filename_rules.gypi

[John Knottenbelt, 2012/04/25 15:25:37]

Right you are -- although filename_rules.gypi won't cover
browser/geolocation/android_location_api_adapter.cc because it's not in android/
and it doesn't end in _android. 

Do you think that moving them to android/ makes most sense?

On 2012/04/24 17:10:23, Yaron wrote:

[Yaron, 2012/04/25 17:26:09]

I don't think it's necessary. Either rename it location_api_adapter_android or
add "android_*" to android-only builds.

[John Knottenbelt, 2012/04/26 15:14:50]

Renamed to location_api_adapter_android

On 2012/04/25 17:26:09, Yaron wrote:

+      'sources!': [
+        'browser/geolocation/android_location_api_adapter.cc',
+        'browser/geolocation/android_location_api_adapter.h',
+        'browser/geolocation/location_provider_android.cc',
+        'browser/geolocation/location_provider_android.h',
+      ],
+    }],
     ['OS=="mac"', {
       'sources': [
         # Build necessary Mozilla sources