Pass JavaRef to Java methods in device/bluetooth.

device/bluetooth/bluetooth_remote_gatt_service_android.cc

Index: device/bluetooth/bluetooth_remote_gatt_service_android.cc
diff --git a/device/bluetooth/bluetooth_remote_gatt_service_android.cc b/device/bluetooth/bluetooth_remote_gatt_service_android.cc
index e26326de51b55a3694f0928b24e3d17eb11e3e9f..71e33d9e406f19aaca6dbe175c4e0828f05f0eda 100644
--- a/device/bluetooth/bluetooth_remote_gatt_service_android.cc
+++ b/device/bluetooth/bluetooth_remote_gatt_service_android.cc
@@ -15,6 +15,7 @@
 
 using base::android::AttachCurrentThread;
 using base::android::JavaParamRef;
+using base::android::JavaRef;
 
 namespace device {
 
@@ -23,9 +24,11 @@ std::unique_ptr<BluetoothRemoteGattServiceAndroid>
 BluetoothRemoteGattServiceAndroid::Create(
     BluetoothAdapterAndroid* adapter,
     BluetoothDeviceAndroid* device,
-    jobject /* BluetoothGattServiceWrapper */ bluetooth_gatt_service_wrapper,
+    const JavaRef<
+        jobject>& /* BluetoothGattServiceWrapper */ bluetooth_gatt_service_wrapper,

[Torne, 2016/09/07 16:50:50]

This line and others like it in the change end up blowing the 80 char limit,
because clang-format isn't willing to split the comment apart from the type and
variable name. Are these comments actually useful? The parameters all seem to be
named the exact same thing as the type anyway - removing the comments would make
this code much easier to format sensibly..

[scheib, 2016/09/07 18:00:30]

I believe clang format will be happier if the comments change to // single lines
- it will respect the line breaks.

The comments were added early on at some reviewers requests to keep track of the
data types -- though I agree that the names match the data types, it's still
hard to know what java object the thing is supposed to be. jobject is rather
like passing around a lot of void*s, and to C++ programmers it feels odd.

     const std::string& instance_id,
-    jobject /* ChromeBluetoothDevice */ chrome_bluetooth_device) {
+    const JavaRef<
+        jobject>& /* ChromeBluetoothDevice */ chrome_bluetooth_device) {
   std::unique_ptr<BluetoothRemoteGattServiceAndroid> service(
       new BluetoothRemoteGattServiceAndroid(adapter, device, instance_id));