GeoLocation: add support for GmsCore location provider

Desktop / Chromium - mcasas [chromium.org] - 20 April 2017 23:00 EDT

This CL adds LocationProviderGmsCore.java, an implementation of LocationProviderFactory.LocationProvider interface using the Google Play Services (GMS) provided location.

The tests content/public/.../LocationProviderTest.java are removed and instead a new pumped up one is added in device/geolocation, doing what the previous one was doing and parameterized to try other location provider APIs (was: android only, now: mock, android and GMS core).

BUG=706014

Review-Url: https://codereview.chromium.org/2809813002 Cr-Commit-Position: refs/heads/master@{#466239}

638d9c4 GeoLocation: add support for GmsCore location provider
content/public/android/BUILD.gn | 1 -
.../content/browser/LocationProviderTest.java | 72 --------
.../src/org/chromium/content/browser/OWNERS | 3 -
device/BUILD.gn | 7 +
device/geolocation/BUILD.gn | 5 +
.../geolocation/LocationProviderAdapter.java | 15 +-
.../geolocation/LocationProviderAndroid.java | 44 ++---
.../geolocation/LocationProviderFactory.java | 15 +-
.../geolocation/LocationProviderGmsCore.java | 136 +++++++++++++++
.../device/geolocation/MockLocationProvider.java | 7 +-
.../device/geolocation/LocationProviderTest.java | 194 +++++++++++++++++++++
11 files changed, 389 insertions(+), 110 deletions(-)

Upstream: git.chromium.org


  • Share