Stubs for triggering the LSD in Clank

chrome/browser/android/location_settings_impl.cc

 // Copyright 2015 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 "chrome/browser/android/location_settings_impl.h"
 
 #include "base/android/jni_android.h"
 #include "content/public/browser/web_contents.h"
 #include "jni/LocationSettings_jni.h"
 
 using base::android::AttachCurrentThread;
 
+using LocationSettingsDialogPromptOutcome =
+    LocationSettings::LocationSettingsDialogPromptOutcome;
+using LocationSettingsDialogPromptOutcomeCallback =
+    LocationSettings::LocationSettingsDialogPromptOutcomeCallback;
+
 LocationSettingsImpl::LocationSettingsImpl() {}
 
 LocationSettingsImpl::~LocationSettingsImpl() {}
 
 bool LocationSettingsImpl::CanSitesRequestLocationPermission(
     content::WebContents* web_contents) {
   JNIEnv* env = AttachCurrentThread();
   return Java_LocationSettings_canSitesRequestLocationPermission(
       env, web_contents->GetJavaWebContents());
 }
+
+bool LocationSettingsImpl::CanPromptToEnableSystemLocationSetting() {
+  JNIEnv* env = AttachCurrentThread();
+  return Java_LocationSettings_canPromptToEnableSystemLocationSetting(env);
+}
+
+void LocationSettingsImpl::PromptToEnableSystemLocationSetting(
+    const LocationSettingsDialogPromptContext prompt_context,
+    content::WebContents* web_contents,
+    LocationSettingsDialogPromptOutcomeCallback callback) {
+  JNIEnv* env = AttachCurrentThread();
+  auto* callback_ptr = new LocationSettingsDialogPromptOutcomeCallback(
+      std::move(callback));
+  Java_LocationSettings_promptToEnableSystemLocationSetting(
+      env, prompt_context, web_contents->GetJavaWebContents(),
+      reinterpret_cast<jlong>(callback_ptr));
+}
+
+static void OnLocationSettingsDialogPromptOutcomeReceived(
+    JNIEnv* env,
+    const base::android::JavaParamRef<jclass>& jcaller,
+    jlong callback_ptr,
+    int result) {
+  auto* callback =
+      reinterpret_cast<LocationSettingsDialogPromptOutcomeCallback*>(
+          callback_ptr);
+  std::move(*callback).Run(static_cast<LocationSettingsDialogPromptOutcome>(

[benwells, 2017/03/01 03:51:27]

I don't really understand the call to std::move here. 

Can you try doing something like 
https://cs.chromium.org/chromium/src/chrome/browser/android/shortcut_helper.c...

and

https://cs.chromium.org/chromium/src/chrome/browser/android/shortcut_helper.c...

[qfiard, 2017/03/01 08:48:18]

I followed the instructions in
https://cs.chromium.org/chromium/src/docs/callback.md, IIUC OnceCallbacks are
now the recommended way of passing callbacks that are meant to be called only
once. OnceCallbacks must be called on a rvalue to be able to consume the
callback.

But I am happy to revert to regular callbacks if you are not familiar with these
and would rather keep using base::Callbacks.

[benwells, 2017/03/01 10:47:45]

Ah ... well there you go, thanks for educating me!

+      result));
+  delete callback;
+}
+
+bool LocationSettingsImpl::Register(JNIEnv* env) {
+  return RegisterNativesImpl(env);
+}