Add a mechanism to pause and resume geolocation requests.

content/browser/android/content_view_core_impl.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/android/content_view_core_impl.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/android/scoped_java_ref.h"
@@ skipping 330 matching lines @@
   GetWebContents()->WasHidden();
   PauseVideo();
 }
 
 void ContentViewCoreImpl::PauseVideo() {
   RenderViewHost* host = web_contents_->GetRenderViewHost();
   if (host)
     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);
   if (obj.is_null())
     return;
   Java_ContentViewCore_resetVSyncNotification(env, obj.obj());
 
   // Note that we might reach this place multiple times while the
   // ContentViewCore remains crashed. E.g. if two tabs share the render process
   // and the process crashes, this will be called for each tab. If the user
@@ skipping 1288 matching lines @@
       reinterpret_cast<ui::ViewAndroid*>(view_android),
       reinterpret_cast<ui::WindowAndroid*>(window_android));
   return reinterpret_cast<jint>(view);
 }
 
 bool RegisterContentViewCore(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content