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

Unified Diff: device/geolocation/location_provider.h

Issue 2226143002: Gets rid of the LocationArbitrator interface, in preference for LocationProvider. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge branch 'master' into lai 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/location_provider.h
diff --git a/device/geolocation/location_provider.h b/device/geolocation/location_provider.h
index 475dbee52083f42a41c18dc9672fa40c3f05b0dd..7f9d6f162c0bae16ae0518cc3c64b5d4e84113d1 100644
--- a/device/geolocation/location_provider.h
+++ b/device/geolocation/location_provider.h
@@ -14,7 +14,7 @@ namespace device {
class LocationProvider;
// The interface for providing location information.
-class LocationProvider {
+class DEVICE_GEOLOCATION_EXPORT LocationProvider {
public:
virtual ~LocationProvider() {}
@@ -40,11 +40,7 @@ class LocationProvider {
virtual void StopProvider() = 0;
// Gets the current best position estimate.
- virtual void GetPosition(Geoposition* position) = 0;
-
- // Provides a hint to the provider that new location data is needed as soon
- // as possible.
- virtual void RequestRefresh() = 0;
+ virtual const Geoposition& GetPosition() = 0;
// Called everytime permission is granted to a page for using geolocation.
// This may either be through explicit user action (e.g. responding to the
« no previous file with comments | « device/geolocation/location_arbitrator_impl_unittest.cc ('k') | device/geolocation/location_provider_android.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698