Bug: 247492602
Bug: 257929630
Test: stress test on/off
Change-Id: Ibbb382404875e61bf24d86e00294c6b470152126

The following
```
verify(adapterService).onProfileServiceStateChanged(profile)
assertThat(profile).isNull();
```
can fail because the profile is set to null after the call to
onProfileServiceStateChanged

By adding a verify on `removeProfile` I ensure the `profile.stop()` has
been called

Bug: 257183155
Test: atest BluetoothInstrumentationTests
Change-Id: I5b80b0f08435c911a41e63e9a40516ba8469c9eb

Bug: 257375445
Test: atest BluetoothInstrumentationTests
Change-Id: I8232879b8e0a383b1767e74ec727a35be21bf90d

Test: NA
Change-Id: Ifffacac7a34022b6c04ceaaf332a0f73fac3ea12

- records enable/disable AdvertisingSet Correctly
- handles advertising disabled events by duration timeout (status is 60)

Bug: 213610155
Tag: #stability
Test: manual test with test app and adb shell dumpsys bluetooth_manager
Change-Id: I1ac5b56fe85cc06201c1346f4266ad0ff9d4e7bc

And document why the fix is needed

Test: atest BluetoothInstrumentationTests -- --exclude-filter AdapterServiceTest#testObfuscateBluetoothAddress_PersistentBetweenAdapterServiceInitialization
Fix: 255088822
Change-Id: Ia8aa2aa35881e1f23c430761b81ce39577027006

Test: atest pts-bot
Change-Id: I086b170637ac4456e9816949d9932fc1898761ea

Bug: 237467631
Test: atest BatteryStateMachineTest
Change-Id: I1b72f5f439851e156de516f688b3884affd90bb9

- Fix answers on some mock instances.
- Updeate test internal values  before sending broadcast intents.

Bug: 256684931
Tag: #refactor
Test: atest BluetoothInstrumentationTests:ActiveDeviceManagerTest (1000 times)
Change-Id: I8285dd829934fddf58af399301c2322634dba2f9

Bug: 237467631
Test: atest SapServerTest
Change-Id: Ib97d626be9cf00da54223bb3b731271e9b2e8a59

RFCOMM/DEVA/
RFCOMM/DEVB/
RFCOMM/DEVA-DEVB/

There are two tests that are still skipped.

I couldn't find the IXIT value referenced in the test spec,
which would allow the tests to pass:

The role of DevA/DevB taken on by the IUT does not
matter to achieve a pass verdict for some test cases
for this Protocol as indicated in the test purpose and is
specified in the test case identifier for role agnostic
tests as detailed in Table 4.1. The role of the IUT/Lower
Tester for these tests is specified in the IXIT [7] in
order to enable the correct test environment conditions
to provoke the Lower Tester.

Since we don't support hooking up a physical serial_port,
TSPC_SPP_2_1 is set to false, which disables
RFCOMM/DEVA-DEVB/RFC/BV-14-C

Bug: 240483035
Test: atest pts-bot:RFCOMM/ -v
Change-Id: I7df1d38f1a5501a42030d6dfeff5560eb1124fe2

Bug: 255278328
Tag: #refactor
Test: atest BluetoothInstrumentationTests:ActiveDeviceManagerTest
Change-Id: I642420fb5f6ae941404484ffd9298e22dc897c1e

This CL adds tests for:
- RequestGetImage
- RequestGetImageProperties
- StackEvent

Bug: 237467631
Test: Ran each test with atest
Change-Id: I4d121e488167f1af97fa0bc73cdb6cb2b50d4899

doReturn().when(foo).bar() should replace all when(foo.bar()).thenReturn()

Bug: 256752741
Test: atest BluetoothInstrumentationTests
Change-Id: I89d444cdffaee80d45eb1ad58daa7ded2500c9e3

This crash happens when we receive scan result while shutting down BT
stack.

We should be obtaining mutex before calling any callbacks to prevent
such crashes.

Test: scan in noisy environment, stop BT stack multiple times
Bug: 246518104
Change-Id: I7fe7e0f7b9e69b24cf8b41a99dabfb00119080f3

Bug: 256395307
Test: atest BluetoothInstrumentationTests --rerun-until-failure
Change-Id: I32712b3dd9dd59286c768d9f4de222f465b8c57d

Bug: 237467631
Test: atest BrowseNodeTest
Change-Id: I97917556ddd532d83870ca62accbe004fa277e05

Bug: 237467631
Test: atest PlayerApplicationSettingsTest
Change-Id: Ibaa8362c4762a1f38f5c5f0fa14d19f603dd2460

If a BT classic that supports both A2DP and HFP is connected
when LE audio is active and music is playing, we should be
careful on deactivating of LE audio otherwise audio could come
from phone speaker.

This CL fixes following audio leak from phone speaker by pending
the activation of HFP and deactivation of LE audio till A2DP is
available.

Before the CL:
1. HFP is activated
2. LE audio is deactivated
3. Music comes from DUT (Audio leak)
4. A2DP is activated
5. Music comes from A2DP

After the CL:
1. HFP is connected, but not activated if HFP is not in use
2. A2DP is connected, activated if A2DP is in use
3. HFP is activated if its activation was pending
4. LE audio is deactivated.

Bug: 247453067
Tag: #refactor
Test: atest BluetoothInstrumentationTests:ActiveDeviceManager
Change-Id: Iee2442b3cc6aeecce4a0ec1e87f1b477c894199b

Mockito does not always support stub override.
It fail when a a method has 2 stub with different argument matcher
Ex:
  doReturn(x).when(y).method(any(Foo.class));
  Foo foo = Foo();
  doReturn(x).when(y).method(eq(foo)); <-- sub-match of above

But the following "should" be ok
  Foo foo = Foo();
  doReturn(x).when(y).method(eq(foo));
  Bar bar = Bar();
  doReturn(x).when(y).method(eq(bar)); <-- same scope match as above

Test: atest BluetoothInstrumentationTests
Bug: 256291144
Change-Id: Ie84e4bbe1ed3c479adc2a194972ba67e34839f50

By disabling leaudio in cuttlefish we are changing the capabilities and
need to update the associated test as it was expecting it

Bug: 250669918
Test: atest pts-bot
Change-Id: I857745b760978c0f2ccf87897904e89241c842a1

Comment is temporary and test need to be fixed.

Bug: 255833353
Test: atest BluetoothInstrumentationTests
Tag: #feature
Change-Id: I4c5c015bc143c3f933a4f2ae7d466c5cb9eb366c

Test: atest "BluetoothInstrumentationTests:com.android.bluetooth.le_audio.LeAudioBroadcastServiceTest"
Bug: 254732845
Tag: #feature
Change-Id: I83dcca1ee15a875ce88d9dd7c666cb1527cf1933

And change LinkedList to ArrayList.

Bug: 255564133
Tag: #refactor
Test: atest BluetoothInstrumentationTests:ActiveDeviceManager
Change-Id: Ie0f9fb03bd867964b5ce028577646703da108b3b

Bug: 243053736
Tag: #refactor
Test: atest BluetoothInstrumentationTests:ActiveDeviceManagerTest
Change-Id: I3baceba1a26867e6dfe464f581432de94d922aa2

Bug: 255598188
Test: atest BluetoothInstrumentationTests
Tag: #feature
Change-Id: I334932a6b4f0c6292a1645ee259a790cd4ac1df5

Added file MediaPlayerBrowserService.kt and MediaPlayer.kt

This is a service which extends "MediaBrowserService" with MediaSession and initial implementation to support browsing folders.
With registration to Media buttons callback which responds to PTT commands(Tried with Play/Pause..).

Moved AudioTrack from A2DP.kt to Utils.kt
All Player operations are performed in MediaPlayer.kt

Reference for older CL: 2253995

Added 21 AVRCP test cases for review

Test: atest pts-bot:AVRCP -v

Change-Id: Icee2a268faa3cbac4cd0001d09baec82eeb01f9d

Test: atest BluetoothOppBatchTest
Bug: 254323235
Tag: #refactor
Change-Id: I633d6fd7c9ccbb8c8a1224f9819f28db94ec3302

Bug: 255598188
Test: atest BluetoothInstrumentationTests
Tag: #feature
Change-Id: Ifab113acfdfa487981c894882f901e537cfd03c9

Bug: 255682218
Bug: 255682560
Tag: #refactor
Test: atest BluetoothInstrumentationTests
Change-Id: I8d7cc2883cec80702e3d7446d9490be723b50cfd

Our goal in a transient loss is to pause the remote player as a courtesy
and then attempt to reestablish playback once we gain focus back.
However, in rare cases, an application (like assistant) can request and
send a pause while we're in a transient loss and already paused. This
should cause playback to stop, however we don't currently react to the
pause and cancel our note to reestablish playback.

This change moves the play/pause decision logic to AVRCP Controller, and
allows local requests to transport controls to cancel our future attempt
to establish playback out of a transient loss if a user sents a pause or
stop.

Tag: #stability
Bug: 254536123
Test: atest BluetoothInstrumentationTests
Change-Id: I277e35f7b432fc7d287cbd4ab19c2ef5f3b870bb

Bug: 255682982
Bug: 255694106
Tag: #refactor
Test: atest BluetoothOppUtilityTest
Change-Id: Ibdfbde86f6b58b5ba3cc2981b7c07735a99e1d24

Sqlite update is now crashing whent getCacheDir is returning null (issue
on mocked context)
Use uiautomation permission for passing tests involving MANAGE_USER
(despite the fact we shouldn't need it)

Test: atest BluetoothInstrumentationTests
Fix: 254325296
Change-Id: I973315eebc347049d8cfe6b62582a1bcb8805b4d

Bug: 243053736
Tag: #refactor
Test: atest BluetoothInstrumentationTests:ActiveDeviceManagerTest
Change-Id: I235e39b23b950e007e8fdb1b9f6c4ea0eb0fda3c

Test: atest pts-bot:AVDTP/SRC
Bug: 239920692

Change-Id: I41039994214e69308ad1801931855a3e0ab155ab

Bug: 237447510
Test: atest pts-bot:HFP
Change-Id: Ibe6d9844c2c15d2f0bec586e633b1cb01c773ca8

Bug: 237447510
Test: none
Change-Id: If25bcd201e8a39f96ef981f5ffb0fd57cb4a6b3f

Android T allows apps to declare a runtime receiver as not exported
by invoking registerReceiver with a new RECEIVER_NOT_EXPORTED flag;
receivers registered with this flag will only receive broadcasts from
the platform and the app itself. However to ensure developers can
properly protect their receivers, all apps targeting a future
platform release and registering a receiver for non-system
broadcasts must specify either the exported or not exported flag
when invoking #registerReceiver; if one of these flags is not
provided, the platform will throw a SecurityException. This commit
updates all the exposed receivers with the RECEIVER_EXPORTED flag
to maintain the existing behavior for these tests.

Bug: 234659204
Tag: #compatibility
Test: atest BassClientServiceTest
Test: atest HapClientTest
Change-Id: Ib832007a0c80755e9a50d029eca45a69b4011ea6

Bug: 237467631
Test: atest BrowseTreeTest
Change-Id: If8538a743fceaf5458f0f7a693482dcf14cfe48a

Some fields were not cleaned up on service shut down.
This was visible in the unit tests.

Bug: 254925603
Tag: #feature
Test: atest BluetoothInstrumentationTests
Change-Id: Ia322147e1d92ddd76f81d95558b3cd4a8e613dbc