Add a mechanism to pause and resume geolocation requests.

content/browser/android/content_view_core_impl.cc

Index: content/browser/android/content_view_core_impl.cc
diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc
index ba493a94b2d70b3d58a9f80b1d7aaf7a4dbf8787..43af9d955be991caf4b06057c1c8f2250d31ed56 100644
--- a/content/browser/android/content_view_core_impl.cc
+++ b/content/browser/android/content_view_core_impl.cc
@@ -348,6 +348,22 @@ void ContentViewCoreImpl::PauseVideo() {
     host->Send(new ViewMsg_PauseVideo(host->GetRoutingID()));
 }
 
+void ContentViewCoreImpl::PauseGeolocation() {
+  RenderViewHostImpl* rvh =
+      static_cast<RenderViewHostImpl*>(web_contents_->GetRenderViewHost());
+  if (rvh) {
+    rvh->PauseGeolocation();
+  }

[joth, 2013/11/10 17:59:02]

I think we could skip the RenderViewHostImpl altogether and go straight to
RenderProcessHost->geolocation_dispatcher here.

I think this would be simpler if you made this a single method taking a bool
right through the stack.

void ContentViewCoreImpl::SetGeolocationPaused(bool paused) {
  RenderViewHost* rvh  = web_contents_->GetRenderViewHost();
  if (rvh) {
    BrowserThread::PostTask(FROM_HERE, IO,
        &GeolocationDispatcherHost::PauseOrResume,
        static_cast<RenderProcessHostImpl*>(
            web_contents_->GetRenderProcessHost())->geolocation_dispatcher(),
        rvh->GetRoutingID(), 
        paused);
  }
}

[benm (inactive), 2013/11/11 14:50:41]

Done.

+}
+
+void ContentViewCoreImpl::ResumeGeolocation() {
+  RenderViewHostImpl* rvh =
+      static_cast<RenderViewHostImpl*>(web_contents_->GetRenderViewHost());
+  if (rvh) {
+    rvh->ResumeGeolocation();
+  }
+}
+
 void ContentViewCoreImpl::OnTabCrashed() {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = java_ref_.get(env);