Add a mechanism to pause and resume geolocation requests.

content/public/browser/web_contents.h

Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index 2035404b81b8a9f80922d334914f808d06635e60..ffa3ad11b43616ef0cb0fae1fae75123944c76ab 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -465,6 +465,10 @@ class WebContents : public PageNavigator,
                             uint32_t max_bitmap_size,
                             const ImageDownloadCallback& callback) = 0;
 
+  // Pause and resume geolocation callbacks firing.
+  virtual void PauseGeolocation() = 0;
+  virtual void ResumeGeolocation() = 0;

[joth, 2013/11/08 20:41:14]

do we need this on WebContents? Why not just add to the public RenderViewHost
base class?

tactically, as it's only needed on Android, it maybe simpler to add this to
ContentViewCore .java / .cc, as content_view_core_impl.cc can see other content
internal APIs and so could poke straight into
GeolocationDispatcherHost::PauseOrResume() etc. No need for cross-plat public
API?

[benm (inactive), 2013/11/08 21:10:28]

i thought about this at first, but it seemed more a WebContents-y API than RVH.
But I really don't feel strongly either way.
SGTM. no need for CVC.java though, is there? Can go straight to native from
AwContents native?

+
  private:
   // This interface should only be implemented inside content.
   friend class WebContentsImpl;