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

Unified Diff: chrome/browser/geolocation/geolocation_dispatcher_host.cc

Issue 5938002: Remove legacy non-client-based geolocation code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove non-client-based geolocation code. Created 9 years, 12 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: chrome/browser/geolocation/geolocation_dispatcher_host.cc
diff --git a/chrome/browser/geolocation/geolocation_dispatcher_host.cc b/chrome/browser/geolocation/geolocation_dispatcher_host.cc
index 4f4020b333d7181ed027c66a8f5f27119a64e3ff..3b824c2636313b49b54974446f07350452e801a8 100644
--- a/chrome/browser/geolocation/geolocation_dispatcher_host.cc
+++ b/chrome/browser/geolocation/geolocation_dispatcher_host.cc
@@ -168,12 +168,10 @@ void GeolocationDispatcherHostImpl::OnStartUpdating(
int bridge_id,
const GURL& requesting_frame,
bool enable_high_accuracy) {
-#if defined(ENABLE_CLIENT_BASED_GEOLOCATION)
// StartUpdating() can be invoked as a result of high-accuracy mode
// being enabled / disabled. No need to register the dispatcher again.
if (!geolocation_renderer_ids_.count(render_view_id))
OnRegisterDispatcher(render_view_id);
-#endif
// WebKit sends the startupdating request before checking permissions, to
// optimize the no-location-available case and reduce latency in the success
// case (location lookup happens in parallel with the permission request).
@@ -197,9 +195,7 @@ void GeolocationDispatcherHostImpl::OnStopUpdating(int render_view_id,
RefreshGeolocationObserverOptions();
geolocation_permission_context_->StopUpdatingRequested(
render_process_id_, render_view_id, bridge_id);
-#if defined(ENABLE_CLIENT_BASED_GEOLOCATION)
OnUnregisterDispatcher(render_view_id);
-#endif
}
void GeolocationDispatcherHostImpl::OnSuspend(int render_view_id,

Powered by Google App Engine
This is Rietveld 408576698