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

Unified Diff: content/browser/android/content_view_core_impl.h

Issue 65273002: Add a mechanism to pause and resume geolocation requests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add new test case, fix a bug :) Created 7 years 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: content/browser/android/content_view_core_impl.h
diff --git a/content/browser/android/content_view_core_impl.h b/content/browser/android/content_view_core_impl.h
index 138aeccca19d9e1171af697bad9a7b6399bf0ff9..4a67d13460fcb4d9986ed19e471ca69ff679324d 100644
--- a/content/browser/android/content_view_core_impl.h
+++ b/content/browser/android/content_view_core_impl.h
@@ -63,6 +63,7 @@ class ContentViewCoreImpl : public ContentViewCore,
virtual void RequestContentClipping(const gfx::Rect& clipping,
const gfx::Size& content_size) OVERRIDE;
virtual void PauseVideo() OVERRIDE;
+ virtual void PauseOrResumeGeolocation(bool should_pause) OVERRIDE;
// --------------------------------------------------------------------------
// Methods called from Java via JNI
@@ -229,6 +230,8 @@ class ContentViewCoreImpl : public ContentViewCore,
jint count);
void SendSingleTapUma(JNIEnv* env, jobject obj, jint type, jint count);
+ bool IsGeolocationActiveForTesting(JNIEnv* env, jobject obj);
+
// --------------------------------------------------------------------------
// Public methods that call to Java via JNI
// --------------------------------------------------------------------------
@@ -383,6 +386,8 @@ class ContentViewCoreImpl : public ContentViewCore,
// will be sent to Renderer once it is ready.
int device_orientation_;
+ bool geolocation_needs_pause_;
+
DISALLOW_COPY_AND_ASSIGN(ContentViewCoreImpl);
};

Powered by Google App Engine
This is Rietveld 408576698