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

Unified Diff: device/geolocation/mock_location_arbitrator.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/mock_location_arbitrator.cc
diff --git a/device/geolocation/mock_location_arbitrator.cc b/device/geolocation/mock_location_arbitrator.cc
index 0ad98c1122337ec4659e583035e047ce0821580a..cf951d3de3e7360763067791b1d5b5c6783bdd9a 100644
--- a/device/geolocation/mock_location_arbitrator.cc
+++ b/device/geolocation/mock_location_arbitrator.cc
@@ -12,11 +12,15 @@ namespace device {
MockLocationArbitrator::MockLocationArbitrator()
: permission_granted_(false), providers_started_(false) {}
-void MockLocationArbitrator::StartProviders(bool enable_high_accuracy) {
+void MockLocationArbitrator::SetUpdateCallback(
+ const LocationProviderUpdateCallback& callback) {}
+
+bool MockLocationArbitrator::StartProvider(bool enable_high_accuracy) {
providers_started_ = true;
+ return providers_started_;
}
-void MockLocationArbitrator::StopProviders() {
+void MockLocationArbitrator::StopProvider() {
providers_started_ = false;
}
@@ -24,6 +28,10 @@ void MockLocationArbitrator::OnPermissionGranted() {
permission_granted_ = true;
}
+void MockLocationArbitrator::RequestRefresh() {}
+
+void MockLocationArbitrator::GetPosition(Geoposition* position) {}
+
bool MockLocationArbitrator::HasPermissionBeenGranted() const {
return permission_granted_;
}

Powered by Google App Engine
This is Rietveld 408576698