Implement a bluetooth picker dialog for Android

chrome/browser/ui/android/bluetooth_chooser_android.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/ui/android/bluetooth_chooser_android.h"
+
+#include "base/android/jni_android.h"
+#include "base/android/jni_string.h"
+#include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/ssl/connection_security.h"
+#include "chrome/browser/ui/android/window_android_helper.h"
+#include "content/public/browser/android/content_view_core.h"
+#include "jni/BluetoothChooserDialog_jni.h"
+
+using base::android::AttachCurrentThread;
+using base::android::ConvertUTF16ToJavaString;
+using base::android::ScopedJavaLocalRef;
+
+BluetoothChooserAndroid::BluetoothChooserAndroid(const GURL& origin,
+    content::WebContents* web_contents, const EventHandler& event_handler)
+        : event_handler_(event_handler) {
+  base::android::ScopedJavaLocalRef<jobject> context =
+      content::ContentViewCore::FromWebContents(web_contents)->GetJavaObject();
+
+  // Create (and show) the BluetoothChooser dialog.
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jstring> origin_string = ConvertUTF16ToJavaString(
+      env, base::UTF8ToUTF16(origin.spec()));
+  java_dialog_.Reset(Java_BluetoothChooserDialog_create(
+      env, context.obj(), origin_string.obj(),
+      connection_security::GetSecurityLevelForWebContents(web_contents),
+      reinterpret_cast<intptr_t>(this)));
+}
+
+BluetoothChooserAndroid::~BluetoothChooserAndroid() {

[Jeffrey Yasskin, 2015/08/28 18:16:33]

I think this destructor needs to call into Java to make sure it drops its
reference to this object.

The destructor can run before receiving a SELECTED or CANCELED event during
shutdown (in ~BluetoothDispatcherHost), and future maintenance might introduce
other cases, and I'd rather not risk a use-after-free.

[Finnur, 2015/08/31 15:25:22]

Done. Look ok?

+}
+
+void BluetoothChooserAndroid::SetAdapterPresence(AdapterPresence presence) {
+  if (presence != AdapterPresence::POWERED_ON) {
+    Java_BluetoothChooserDialog_notifyAdapterTurnedOff(
+        AttachCurrentThread(), java_dialog_.obj());
+  }
+}
+
+void BluetoothChooserAndroid::ShowDiscoveryState(DiscoveryState state) {
+  NOTIMPLEMENTED();  // Currently we don't show a 'still searching' state.
+}
+
+void BluetoothChooserAndroid::AddDevice(const std::string& device_id,
+               const base::string16& device_name) {
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jstring> java_device_id = ConvertUTF16ToJavaString(

[Jeffrey Yasskin, 2015/08/28 18:16:33]

Use ConvertUTF8ToJavaString() for this one.

[Finnur, 2015/08/31 15:25:22]

Done.

+      env, base::UTF8ToUTF16(device_id));
+  ScopedJavaLocalRef<jstring> java_device_name = ConvertUTF16ToJavaString(
+      env, device_name);
+  Java_BluetoothChooserDialog_addDevice(
+      env, java_dialog_.obj(), java_device_id.obj(), java_device_name.obj());
+}
+
+void BluetoothChooserAndroid::RemoveDevice(const std::string& device_id) {
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jstring> java_device_id = ConvertUTF16ToJavaString(
+      env, base::UTF8ToUTF16(device_id));
+  Java_BluetoothChooserDialog_removeDevice(
+      env, java_dialog_.obj(), java_device_id.obj());
+}
+
+void BluetoothChooserAndroid::ReportDeviceSelected(
+    JNIEnv* env, jobject obj, jstring device_id) {
+  event_handler_.Run(
+      Event::SELECTED, base::android::ConvertJavaStringToUTF8(env, device_id));

[Jeffrey Yasskin, 2015/08/28 18:16:33]

It looks like we're missing a way to report that the user has canceled the
dialog?

[Finnur, 2015/08/31 15:25:22]

And a way to report that the help link was pressed, etc. 

It is coming (next CL).

+}
+
+void BluetoothChooserAndroid::ReportRestartSearch(JNIEnv* env, jobject obj) {
+  event_handler_.Run(Event::RESTART, "");
+}
+
+// static
+bool BluetoothChooserAndroid::Register(JNIEnv* env) {
+  return RegisterNativesImpl(env);
+}