Differentiate between services disovered by SDP and LE GATT discovery.
Service avaliability differs between transports.

This patch is not making any functional changes, just making code
clearer.

Test: perform LE and Classic pairing to dual mode device
Bug: 248340112
Change-Id: I7be9d2c0549f8c33eff16b70d6f54dd1f08a71a0

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

Separate the Play and Pause events from the PlayPause combo event since
media information is in place. The peripheral device can now correctly
query media status to send out Play or Pause correspondingly.

Bug: 240780471
Tag: #floss
Test: local test with evtest
Change-Id: I0a7bbd8eaef6a57759ffb7b94e9282bd46924c9e

Parse the topshim passed media information into btif standardized
version, which includes: converting time units into milliseconds,
combining playback status, position, and duration into one struct, and
converting metadata information into AVRCP attributes.

Bug: 240780471
Tag: #floss
Test: ./build.py
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Ied76eba423d509a8c6f8cb4c6371bfd502bbbcac

1. AtvRemote apk need to be pre-installed on DUT(device under test)
2. Open Setting panel, inquiry bluetooth devices
3. Connect with OPPO Echo Play earbuds (this device responses invalid ATT handle in SDP)

Bug: 256771934
Tag: #stability
Change-Id: I479902b327f4f331bc9a647904b63c727ad76dc8
Test:  Remote device connects successfully

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

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

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

The wrapper holds the raw pointer and clones the lifetime from the
pointing object, which forces the compiler to check and fail when the
wrapper lives longer than the data.

Bug: 249687132
Tag: #floss
Test: build.py --target test; Deployed to CB and checked basic features
Change-Id: I07a81a0d8b262d0d0008709164d460fa7e86b0e1

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

Test: atest --host bluetooth_test_gd_unit
Change-Id: Ided68cf686c4be968f47da295edbfc5d5aa301cb

This macro is provided by the gd header os/include/osi.h.
This change is made to remove dependencies from Rootcanal on gd's
runtime.

Bug: 256013143
Test: m root-canal
Change-Id: I5a003a6885cad58c7421c58527524c8a3ea36d3b

This patch provide methods for suspend module to disable active
advertising sets before entering suspend mode and re-enable them
after leaving suspend mode.

Bug: 224602924
Tag: #floss
Test: build.py --target test

Change-Id: Id72fdcfdaab81df746c89c8efe1a539130dbe349

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

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

* changes:
  gatt: Remove reduntant `in_use` flag
  gatt: Improve send queued request from tGATT_TCB
  gatt: Improve queueing of GATT messages

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

* changes:
  Set LeAudio active properly for HAP devices
  Make LE hearing Aid active prior than non hearing aid devices

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

Comment is temporary and test need to be fixed.

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