Remove ContentViewCore::PauseOrResumeGeolocation()

android_webview/native/aw_contents.cc

Index: android_webview/native/aw_contents.cc
diff --git a/android_webview/native/aw_contents.cc b/android_webview/native/aw_contents.cc
index 335886a2bfa4fb091a07535e41ac3dfdf59725fd..2838dda4fc4ea31bcb3af301362bd0519e103ee7 100644
--- a/android_webview/native/aw_contents.cc
+++ b/android_webview/native/aw_contents.cc
@@ -850,11 +850,6 @@ void AwContents::SetIsPaused(JNIEnv* env,
                              bool paused) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   browser_view_renderer_.SetIsPaused(paused);

[timvolodine, 2016/10/12 15:12:44]

could you add a comment saying that geolocation is paused via the page
visibility mechanism? not sure, maybe a better place for this is on the java
side..

[boliu, 2016/10/12 15:35:38]

I prefer java side too :)

[Jinsuk Kim, 2016/10/14 07:04:02]

Added a comment in AwContents.updateContentViewCoreVisibility(). See if it was
put in a good spot.

-  ContentViewCore* cvc =
-      ContentViewCore::FromWebContents(web_contents_.get());
-  if (cvc) {
-    cvc->PauseOrResumeGeolocation(paused);
-  }
 }
 
 void AwContents::OnAttachedToWindow(JNIEnv* env,