commit c43f68e52caa927e71b97fe602268a31390ea706
author Gustavo Garcia <gustavogb@gmail.com> Thu Jun 29 08:25:16 2017
committer Commit Bot <commit-bot@chromium.org> Tue Jul 04 13:50:15 2017
tree 54b74a0414902168bac556e753d4126970b086c8
parent cc8856c9c29509d5c814495f61f489651a4c9f91

Fix do not unregister bluetooth receiver if it was not registered

Bug: webrtc:7890
Change-Id: Ib46b4a4407fa030500930ed03a093b26c71f8963
Reviewed-on: https://chromium-review.googlesource.com/550617
Commit-Queue: Henrik Andreasson <henrika@webrtc.org>
Reviewed-by: Henrik Andreasson <henrika@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#18892}

AUTHORS

diff --git a/AUTHORS b/AUTHORS
index cb537cf..3a17351 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -19,6 +19,7 @@
 Frederik Riedel, Frogg GmbH <frederik.riedel@frogg.io>
 Giji Gangadharan <giji.g@samsung.com>
 Graham Yoakum <gyoakum@skobalt.com>
+Gustavo Garcia <gustavogb@gmail.com>
 Hugues Ekra <hekra01@gmail.com>
 Jake Hilton <jakehilton@gmail.com>
 James H. Brown <jbrown@burgoyne.com>

webrtc/examples/androidapp/src/org/appspot/apprtc/AppRTCBluetoothManager.java

diff --git a/webrtc/examples/androidapp/src/org/appspot/apprtc/AppRTCBluetoothManager.java b/webrtc/examples/androidapp/src/org/appspot/apprtc/AppRTCBluetoothManager.java
index 10b1a5c..99979a3 100644
--- a/webrtc/examples/androidapp/src/org/appspot/apprtc/AppRTCBluetoothManager.java
+++ b/webrtc/examples/androidapp/src/org/appspot/apprtc/AppRTCBluetoothManager.java
@@ -276,23 +276,25 @@
   /** Stops and closes all components related to Bluetooth audio. */
   public void stop() {
     ThreadUtils.checkIsOnMainThread();
-    unregisterReceiver(bluetoothHeadsetReceiver);
     Log.d(TAG, "stop: BT state=" + bluetoothState);
-    if (bluetoothAdapter != null) {
-      // Stop BT SCO connection with remote device if needed.
-      stopScoAudio();
-      // Close down remaining BT resources.
-      if (bluetoothState != State.UNINITIALIZED) {
-        cancelTimer();
-        if (bluetoothHeadset != null) {
-          bluetoothAdapter.closeProfileProxy(BluetoothProfile.HEADSET, bluetoothHeadset);
-          bluetoothHeadset = null;
-        }
-        bluetoothAdapter = null;
-        bluetoothDevice = null;
-        bluetoothState = State.UNINITIALIZED;
-      }
+    if (bluetoothAdapter == null) {
+      return;
     }
+    // Stop BT SCO connection with remote device if needed.
+    stopScoAudio();
+    // Close down remaining BT resources.
+    if (bluetoothState == State.UNINITIALIZED) {
+      return;
+    }
+    unregisterReceiver(bluetoothHeadsetReceiver);
+    cancelTimer();
+    if (bluetoothHeadset != null) {
+      bluetoothAdapter.closeProfileProxy(BluetoothProfile.HEADSET, bluetoothHeadset);
+      bluetoothHeadset = null;
+    }
+    bluetoothAdapter = null;
+    bluetoothDevice = null;
+    bluetoothState = State.UNINITIALIZED;
     Log.d(TAG, "stop done: BT state=" + bluetoothState);
   }