Remove outdated devices from Android device chooser

device/bluetooth/android/java/src/org/chromium/device/bluetooth/ChromeBluetoothAdapter.java

 // 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.
 
 package org.chromium.device.bluetooth;
 
 import android.Manifest;
 import android.annotation.TargetApi;
 import android.bluetooth.BluetoothAdapter;
 import android.bluetooth.le.ScanSettings;
@@ skipping 18 matching lines @@
  * Lifetime is controlled by device::BluetoothAdapterAndroid.
  */
 @JNINamespace("device")
 @TargetApi(Build.VERSION_CODES.LOLLIPOP)
 final class ChromeBluetoothAdapter extends BroadcastReceiver {
     private static final String TAG = "Bluetooth";
 
     private long mNativeBluetoothAdapterAndroid;
     // mAdapter is final to ensure registerReceiver is followed by unregisterReceiver.
     private final Wrappers.BluetoothAdapterWrapper mAdapter;
-    private int mNumDiscoverySessions;
     private ScanCallback mScanCallback;
 
     // ---------------------------------------------------------------------------------------------
     // Construction and handler for C++ object destruction.
 
     /**
      * Constructs a ChromeBluetoothAdapter.
      * @param nativeBluetoothAdapterAndroid Is the associated C++
      *                                      BluetoothAdapterAndroid pointer value.
      * @param adapterWrapper Wraps the default android.bluetooth.BluetoothAdapter,
@@ skipping 84 matching lines @@
         return isPresent()
                 && mAdapter.getScanMode() == BluetoothAdapter.SCAN_MODE_CONNECTABLE_DISCOVERABLE;
     }
 
     // Implements BluetoothAdapterAndroid::IsDiscovering.
     @CalledByNative
     private boolean isDiscovering() {
         return isPresent() && (mAdapter.isDiscovering() || mScanCallback != null);
     }
 
-    // Implements BluetoothAdapterAndroid::AddDiscoverySession.
-    @CalledByNative
-    private boolean addDiscoverySession() {
-        if (!isPowered()) {
-            Log.d(TAG, "addDiscoverySession: Fails: !isPowered");
-            return false;
-        }
-
-        mNumDiscoverySessions++;
-        Log.d(TAG, "addDiscoverySession: Now %d sessions.", mNumDiscoverySessions);
-        if (mNumDiscoverySessions > 1) {
-            return true;
-        }
-
-        if (!startScan()) {
-            mNumDiscoverySessions--;
-            return false;
-        }
-        return true;
-    }
-
-    // Implements BluetoothAdapterAndroid::RemoveDiscoverySession.
-    @CalledByNative
-    private boolean removeDiscoverySession() {
-        if (mNumDiscoverySessions == 0) {
-            assert false;
-            Log.w(TAG, "removeDiscoverySession: No scan in progress.");
-            return false;
-        }
-
-        --mNumDiscoverySessions;
-
-        if (mNumDiscoverySessions == 0) {
-            Log.d(TAG, "removeDiscoverySession: Now 0 sessions. Stopping scan.");
-            return stopScan();
-        }
-
-        Log.d(TAG, "removeDiscoverySession: Now %d sessions.", mNumDiscoverySessions);
-        return true;
-    }
-
-    // ---------------------------------------------------------------------------------------------
-    // Implementation details:
-
-    /**
-     * @return true if Chromium has permission to scan for Bluetooth devices.
-     */
-    private boolean canScan() {
-        Wrappers.ContextWrapper context = mAdapter.getContext();
-        return context.checkPermission(Manifest.permission.ACCESS_COARSE_LOCATION)
-                || context.checkPermission(Manifest.permission.ACCESS_FINE_LOCATION);
-    }
-
-    private void registerBroadcastReceiver() {
-        if (mAdapter != null) {
-            mAdapter.getContext().registerReceiver(
-                    this, new IntentFilter(BluetoothAdapter.ACTION_STATE_CHANGED));
-        }
-    }
-
-    private void unregisterBroadcastReceiver() {
-        if (mAdapter != null) {
-            mAdapter.getContext().unregisterReceiver(this);
-        }
-    }
-
     /**
      * Starts a Low Energy scan.
      * @return True on success.
      */
+    @CalledByNative
     private boolean startScan() {
         Wrappers.BluetoothLeScannerWrapper scanner = mAdapter.getBluetoothLeScanner();
 
         if (scanner == null) {
             return false;
         }
 
         if (!canScan()) {
             return false;
         }
@@ skipping 16 matching lines @@
             mScanCallback = null;
             return false;
         }
         return true;
     }
 
     /**
      * Stops the Low Energy scan.
      * @return True if a scan was in progress.
      */
+    @CalledByNative
     private boolean stopScan() {
         if (mScanCallback == null) {
             return false;
         }
 
         try {
             Wrappers.BluetoothLeScannerWrapper scanner = mAdapter.getBluetoothLeScanner();
             if (scanner != null) {
                 scanner.stopScan(mScanCallback);
             }
         } catch (IllegalArgumentException e) {
             Log.e(TAG, "Cannot stop scan: " + e);
         } catch (IllegalStateException e) {
             Log.e(TAG, "Adapter is off. Cannot stop scan: " + e);
         }
         mScanCallback = null;
         return true;
     }
 
+    // ---------------------------------------------------------------------------------------------
+    // Implementation details:
+
+    /**
+     * @return true if Chromium has permission to scan for Bluetooth devices.
+     */
+    private boolean canScan() {
+        Wrappers.ContextWrapper context = mAdapter.getContext();
+        return context.checkPermission(Manifest.permission.ACCESS_COARSE_LOCATION)
+                || context.checkPermission(Manifest.permission.ACCESS_FINE_LOCATION);
+    }
+
+    private void registerBroadcastReceiver() {
+        if (mAdapter != null) {
+            mAdapter.getContext().registerReceiver(
+                    this, new IntentFilter(BluetoothAdapter.ACTION_STATE_CHANGED));
+        }
+    }
+
+    private void unregisterBroadcastReceiver() {
+        if (mAdapter != null) {
+            mAdapter.getContext().unregisterReceiver(this);
+        }
+    }
+
     /**
      * Implements callbacks used during a Low Energy scan by notifying upon
      * devices discovered or detecting a scan failure.
      */
     private class ScanCallback extends Wrappers.ScanCallbackWrapper {
         @Override
         public void onBatchScanResult(List<Wrappers.ScanResultWrapper> results) {
             Log.v(TAG, "onBatchScanResults");
         }
 
         @Override
         public void onScanResult(int callbackType, Wrappers.ScanResultWrapper result) {
             Log.v(TAG, "onScanResult %d %s %s", callbackType, result.getDevice().getAddress(),
                     result.getDevice().getName());
 
             List<ParcelUuid> uuids = result.getScanRecord_getServiceUuids();
 
             nativeCreateOrUpdateDeviceOnScan(mNativeBluetoothAdapterAndroid,
                     result.getDevice().getAddress(), result.getDevice(), uuids);
         }
 
         @Override
         public void onScanFailed(int errorCode) {
             Log.w(TAG, "onScanFailed: %d", errorCode);
             nativeOnScanFailed(mNativeBluetoothAdapterAndroid);
-            mNumDiscoverySessions = 0;
         }
     }
 
     @Override
     public void onReceive(Context context, Intent intent) {
         String action = intent.getAction();
 
         if (isPresent() && BluetoothAdapter.ACTION_STATE_CHANGED.equals(action)) {
             int state = intent.getIntExtra(BluetoothAdapter.EXTRA_STATE, BluetoothAdapter.ERROR);
 
@@ skipping 39 matching lines @@
     // 'Object' type must be used for |bluetoothDeviceWrapper| because inner class
     // Wrappers.BluetoothDeviceWrapper reference is not handled by jni_generator.py JavaToJni.
     // http://crbug.com/505554
     private native void nativeCreateOrUpdateDeviceOnScan(long nativeBluetoothAdapterAndroid,
             String address, Object bluetoothDeviceWrapper, List<ParcelUuid> advertisedUuids);
 
     // Binds to BluetoothAdapterAndroid::nativeOnAdapterStateChanged
     private native void nativeOnAdapterStateChanged(
             long nativeBluetoothAdapterAndroid, boolean powered);
 }