bluetooth: Android adapter can be created with and without Bluetooth permission.

device/bluetooth/bluetooth_adapter_android.h

 // 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.
 
 #ifndef DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_ANDROID_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_ANDROID_H_
 
 #include "base/android/jni_android.h"
 #include "base/memory/weak_ptr.h"
 #include "device/bluetooth/bluetooth_adapter.h"
 
 namespace base {
 class SequencedTaskRunner;
 }  // namespace base
 
 namespace device {
 
 // The BluetoothAdapterAndroid class implements BluetoothAdapter for the
 // Android platform.
 class DEVICE_BLUETOOTH_EXPORT BluetoothAdapterAndroid final

[Ted C, 2015/05/07 22:23:57]

FWIW, you can call the java side BluetoothAdapterAndroid to keep it in sync with
the native side of things and then you won't have the name collision problem you
have.

With that said, it is a bit weird since it's not actually the android bluetooth
adapter but instead chrome's.  Also, the shed should be light blue.

[scheib, 2015/05/07 23:59:17]

Thanks. I've pondered a bit. For now I'm going to keep the java class name
shorter so the C++ calls to it are simpler, as there are more of those so far vs
the number of fully qualified android.bluetooth.BluetoothAdapter I have to use
in java classes (which also get 100 columns, so a bit more space there). 

     : public BluetoothAdapter {
  public:
   // Create a BluetoothAdapterAndroid instance.
   static base::WeakPtr<BluetoothAdapterAndroid> CreateAdapter();
 
+  // Create a BluetoothAdapterAndroid instance without Bluetooth permission.
+  static base::WeakPtr<BluetoothAdapterAndroid>
+  CreateAdapterWithoutPermissionForTesting();
+
   // Register C++ methods exposed to Java using JNI.
   static bool RegisterJNI(JNIEnv* env);
 
   // True if this app has android permissions necessary for Bluetooth.
   bool HasBluetoothPermission() const;
 
   // BluetoothAdapter:
   std::string GetAddress() const override;
   std::string GetName() const override;
   void SetName(const std::string& name,
@@ skipping 55 matching lines @@
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<BluetoothAdapterAndroid> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BluetoothAdapterAndroid);
 };
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_ANDROID_H_