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

Unified Diff: device/geolocation/location_provider_android.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_android.h
diff --git a/device/geolocation/location_provider_android.h b/device/geolocation/location_provider_android.h
index bae4bb1b20b73241416a566dd35da59fa6d9737f..7b5734cb35be4c2b087eeeb379f0148c7e8f6a41 100644
--- a/device/geolocation/location_provider_android.h
+++ b/device/geolocation/location_provider_android.h
@@ -25,11 +25,14 @@ class LocationProviderAndroid : public LocationProviderBase {
// LocationProvider.
bool StartProvider(bool high_accuracy) override;
void StopProvider() override;
- void GetPosition(Geoposition* position) override;
- void RequestRefresh() override;
+ const Geoposition& GetPosition() override;
void OnPermissionGranted() override;
private:
+ // Provides a hint to the provider that new location data is needed as soon
+ // as possible.
+ void RequestRefresh();
Wez 2016/08/19 01:33:12 Not sure what you intended here but: (1) This is p
CJ 2016/08/22 17:41:31 Done.
+
Geoposition last_position_;
};

Powered by Google App Engine
This is Rietveld 408576698