bluetooth: android: Initial BluetoothDeviceAndroid implementation.

device/bluetooth/bluetooth_adapter_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 "device/bluetooth/bluetooth_adapter_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/sequenced_task_runner.h"
 #include "base/single_thread_task_runner.h"
 #include "base/thread_task_runner_handle.h"
 #include "device/bluetooth/android/wrappers.h"
 #include "device/bluetooth/bluetooth_advertisement.h"
+#include "device/bluetooth/bluetooth_device_android.h"
 #include "jni/ChromeBluetoothAdapter_jni.h"
 
 using base::android::AttachCurrentThread;
 using base::android::ConvertJavaStringToUTF8;
 
 namespace device {
 
 // static
 base::WeakPtr<BluetoothAdapter> BluetoothAdapter::CreateAdapter(
     const InitCallback& init_callback) {
@@ skipping 100 matching lines @@
     scoped_ptr<BluetoothAdvertisement::Data> advertisement_data,
     const CreateAdvertisementCallback& callback,
     const CreateAdvertisementErrorCallback& error_callback) {
   error_callback.Run(BluetoothAdvertisement::ERROR_UNSUPPORTED_PLATFORM);
 }
 
 void BluetoothAdapterAndroid::OnScanFailed(JNIEnv* env, jobject obj) {
   MarkDiscoverySessionsAsInactive();
 }
 
+void BluetoothAdapterAndroid::CreateOrUpdateDeviceOnScan(
+    JNIEnv* env,
+    jobject obj,

[Jeffrey Yasskin, 2015/07/07 00:11:14]

|obj| isn't used? Wildly guessing at why it's there, does it go away if you make
'nativeCreateOrUpdateDeviceOnScan' static? ...
http://docs.oracle.com/javase/1.5.0/docs/guide/jni/spec/design.html#wp715 says
you get a reference to the class if you make it static, which isn't really an
improvement. Oh well.

[scheib, 2015/07/08 00:49:10]

:( Yes, this seems unfortunate that JNI presumes this on all signatures instead
of having Java calling code provide object references explicitly when needed. 

+    const jstring& jaddress,
+    jobject bluetooth_device_wrapper,
+    jobject advertised_uuids) {
+  BluetoothDevice*& device = devices_[ConvertJavaStringToUTF8(env, jaddress)];
+  if (!device) {
+    device = BluetoothDeviceAndroid::Create(bluetooth_device_wrapper);
+    static_cast<BluetoothDeviceAndroid*>(device)
+        ->UpdateAdvertisedUUIDs(advertised_uuids);
+    FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
+                      DeviceAdded(this, device));
+  } else {
+    if (static_cast<BluetoothDeviceAndroid*>(device)
+            ->UpdateAdvertisedUUIDs(advertised_uuids)) {
+      FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
+                        DeviceChanged(this, device));
+    }
+  }
+}
+
 BluetoothAdapterAndroid::BluetoothAdapterAndroid() : weak_ptr_factory_(this) {
 }
 
 BluetoothAdapterAndroid::~BluetoothAdapterAndroid() {
   Java_ChromeBluetoothAdapter_onBluetoothAdapterAndroidDestruction(
       AttachCurrentThread(), j_adapter_.obj());
 }
 
 void BluetoothAdapterAndroid::AddDiscoverySession(
     BluetoothDiscoveryFilter* discovery_filter,
@@ skipping 27 matching lines @@
   // TODO(scheib): Support filters crbug.com/490401
   NOTIMPLEMENTED();
   error_callback.Run();
 }
 
 void BluetoothAdapterAndroid::RemovePairingDelegateInternal(
     device::BluetoothDevice::PairingDelegate* pairing_delegate) {
 }
 
 }  // namespace device