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

Unified Diff: device/geolocation/geolocation_provider_impl.cc

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: Removes LocationArbitrator.h 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/geolocation_provider_impl.cc
diff --git a/device/geolocation/geolocation_provider_impl.cc b/device/geolocation/geolocation_provider_impl.cc
index 1c025d0614157b201d1264feef9f2f23af0b9148..04d42388f9edf0e2eeb844cc7248e2fb8dae6892 100644
--- a/device/geolocation/geolocation_provider_impl.cc
+++ b/device/geolocation/geolocation_provider_impl.cc
@@ -140,13 +140,13 @@ void GeolocationProviderImpl::OnClientsChanged() {
void GeolocationProviderImpl::StopProviders() {
DCHECK(OnGeolocationThread());
DCHECK(arbitrator_);
- arbitrator_->StopProviders();
+ arbitrator_->StopProvider();
}
void GeolocationProviderImpl::StartProviders(bool enable_high_accuracy) {
DCHECK(OnGeolocationThread());
DCHECK(arbitrator_);
- arbitrator_->StartProviders(enable_high_accuracy);
+ arbitrator_->StartProvider(enable_high_accuracy);
}
void GeolocationProviderImpl::InformProvidersPermissionGranted() {
@@ -183,7 +183,7 @@ void GeolocationProviderImpl::CleanUp() {
arbitrator_.reset();
}
-std::unique_ptr<LocationArbitrator>
+std::unique_ptr<LocationProvider>
GeolocationProviderImpl::CreateArbitrator() {
LocationArbitratorImpl::LocationUpdateCallback callback = base::Bind(
&GeolocationProviderImpl::OnLocationUpdate, base::Unretained(this));

Powered by Google App Engine
This is Rietveld 408576698