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

Unified Diff: chrome/browser/android/location_settings_impl.cc

Issue 2709883005: Stubs for triggering the LSD in Clank (Closed)
Patch Set: Rebase onto head Created 3 years, 10 months 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: chrome/browser/android/location_settings_impl.cc
diff --git a/chrome/browser/android/location_settings_impl.cc b/chrome/browser/android/location_settings_impl.cc
index 9da10a1b624368c3967635d2ece2772a9651cefb..32232db88efe08df72614a2a956166937e8fee72 100644
--- a/chrome/browser/android/location_settings_impl.cc
+++ b/chrome/browser/android/location_settings_impl.cc
@@ -10,6 +10,11 @@
using base::android::AttachCurrentThread;
+using LocationSettingsDialogOutcome =
+ LocationSettings::LocationSettingsDialogOutcome;
+using LocationSettingsDialogOutcomeCallback =
+ LocationSettings::LocationSettingsDialogOutcomeCallback;
+
LocationSettingsImpl::LocationSettingsImpl() {}
LocationSettingsImpl::~LocationSettingsImpl() {}
@@ -20,3 +25,37 @@ bool LocationSettingsImpl::CanSitesRequestLocationPermission(
return Java_LocationSettings_canSitesRequestLocationPermission(
env, web_contents->GetJavaWebContents());
}
+
+bool LocationSettingsImpl::CanPromptToEnableSystemLocationSetting() {
+ JNIEnv* env = AttachCurrentThread();
+ return Java_LocationSettings_canPromptToEnableSystemLocationSetting(env);
+}
+
+void LocationSettingsImpl::PromptToEnableSystemLocationSetting(
+ const LocationSettingsDialogContext prompt_context,
+ content::WebContents* web_contents,
+ LocationSettingsDialogOutcomeCallback callback) {
+ JNIEnv* env = AttachCurrentThread();
+ auto* callback_ptr = new LocationSettingsDialogOutcomeCallback(
Ted C 2017/03/02 00:21:56 maybe a comment around ownership and that Outcome
qfiard 2017/03/02 13:28:13 Done.
+ std::move(callback));
+ Java_LocationSettings_promptToEnableSystemLocationSetting(
+ env, prompt_context, web_contents->GetJavaWebContents(),
+ reinterpret_cast<jlong>(callback_ptr));
+}
+
+static void OnLocationSettingsDialogOutcome(
+ JNIEnv* env,
+ const base::android::JavaParamRef<jclass>& jcaller,
+ jlong callback_ptr,
+ int result) {
+ auto* callback =
+ reinterpret_cast<LocationSettingsDialogOutcomeCallback*>(
+ callback_ptr);
+ std::move(*callback).Run(static_cast<LocationSettingsDialogOutcome>(
+ result));
+ delete callback;
+}
+
+bool LocationSettingsImpl::Register(JNIEnv* env) {
+ return RegisterNativesImpl(env);
+}

Powered by Google App Engine
This is Rietveld 408576698