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

Update the sink audio location after loaded from the storage

Bug: 252935428
Tag: #refactor
Test: atest BluetoothInstrumentationTests
Test: BT off/on, and check the battery information
Change-Id: I820b43acb552b776b0322112ee3070ed0930b431

Fix obvious mistake.
Error can be visible if multiple BLE devices is in used, some of the
GATT commands are queued and then first disconnected device has no queue
GATT commands.

Bug: 257046947
Test: manual
 - Connect BLE devices X, Y
 - Queue multiple commands on device X
 - Disconnect device Y

Tag: #feature
Change-Id: Ic0877bf7322dfa5bf0103bb6eab18cadac51244b

75cc3cb7

Change-Id: I8a6669ee978c87edf6752a1c7719797df346bfa0

dd9e6591

Change-Id: Id5711aae12b1f9c5922c28fc7c58f26c6aa68266

267a4807

Change-Id: I8e0ee9fa832207de6cd8e8c5c7f7b5a45a6f282a

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

Revert submission 2268950

Bug: 257046947
Bug: 249431973
Test: reverting changes that broke earlier build
Reason for revert: b/257046947 infinite loop
Reverted Changes:
If19de77a8:gatt: Improve queueing of GATT messages
I1bfb4c546:gatt: Remove reduntant `in_use` flag
I1d3351ebc:gatt: Improve send queued request from tGATT_TCB

Change-Id: Ie898cf668476edfc09b8d94218bab16116268810

Revert submission 2268950

Bug: 257046947
Bug: 249431973
Test: reverting changes that broke earlier build
Reason for revert: b/257046947 infinite loop
Reverted Changes:
If19de77a8:gatt: Improve queueing of GATT messages
I1bfb4c546:gatt: Remove reduntant `in_use` flag
I1d3351ebc:gatt: Improve send queued request from tGATT_TCB

Change-Id: I2c925d07981aca37a36986e4214d70167b9e1cf6

Revert submission 2268950

Bug: 257046947
Bug: 249431973
Test: reverting changes that broke earlier build
Reason for revert: b/257046947 infinite loop
Reverted Changes:
If19de77a8:gatt: Improve queueing of GATT messages
I1bfb4c546:gatt: Remove reduntant `in_use` flag
I1d3351ebc:gatt: Improve send queued request from tGATT_TCB

Change-Id: Ief553d755b4002030a4ae99e16d9f5ebba325235

GATT over LE services are already reported in BTA_DM_GATT_OVER_LE_RES_EVT
no need to report them again. Also, calling BTA_DM_DISC_RES_EVT here
might mess up SDP retry counter sdp_attempts in btif_dm.cc. It should be
invoked just for SDP.

Bug: 248340112
Test: Pair with BLE and Dual mode device, ensure services are discovered
and reported

Change-Id: I3261cbd1b80e50689178db2487154df1ad4c3557

Also fix up some other test related infra

Bug: 254473601
Test: gd/cert/run
Tag: #refactor
BYPASS_LONG_LINES_REASON: Bluetooth likes 120 lines

Change-Id: Idfd4e501e453864798f9bb1b00f35dd4fdef6db0

We will immediately send them with rest of SDP results in BTA_DM_DISC_RES_EVT

Test: pair with Dual Mode device
Bug: 248340112
Change-Id: I7b5f0de233c8d3f438e72598450ab20f457d32cb

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

- 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

The filegroup :BluetoothHciTestSources was neither being
compiled nor run.

- Remove duplicate test files "le_advertising_manager_test.cc",
  "le_scanning_manager_test.cc", already compiled in
  :BluetoothHciUnitTestSources
- Remove duplicate tests in hci_layer_test.cc: new tests were
  implemented in hci_layer_unittest.cc, move remaining tests
  to hci_layer_unittest.cc
- Move all remaining files to :bluetoothHciUnitTestSources and
  fix compilation issues

This change leaves 4 tests disabled:
  HciTest.HciTimeOut
  LeAddressManagerTest.rotator_address_for_multiple_client
  StackManagerTest.start_and_shutdown_no_module
  StackManagerTest.get_module_instance

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