Merge "Disable Link layer socket device test in rootcanal"
am: 460d035101

Change-Id: Id72df7c7f5336a337575c4bc29c5f0cee15b4d53

Merge "Disable Link layer socket device test in rootcanal"

[automerger skipped] Merge "Revert "DO NOT MERGE Hearing Aid: Add JNI API for add/remove whitelist""
am: 864fe7ad39 -s ours
am skip reason: subject contains skip directive

Change-Id: I5c1ff1bbe0f3c7fc665e6f89d4eabebecd26a961

Merge "Revert "DO NOT MERGE Hearing Aid: Add JNI API for add/remove whitelist""

Disable Link layer socket device test in rootcanal

Test: presubmit
Bug: 129537494
Change-Id: I852da314ab65052a0e1a794f5faa9f21e1249efe

Merge "Fix assert macro logical inversion"
am: c5b38fae53

Change-Id: I1651a93c09137f8a5d159cf056d3ea3d0c115831

Merge "Fix assert macro logical inversion"

Fix assert macro logical inversion

Rename FATAL_WHEN to ASSERT and LOG_FATAL_WHEN to ASSERT_LOG

Test: atest
Change-Id: I0e6e15ce7cfbb89e76910181f164e8ff32abc033

Revert "DO NOT MERGE Hearing Aid: Add JNI API for add/remove whitelist"

This reverts commit d65f697c011cc6e5eaad767de3ec726cfd25b27f.
This CL is merge into AOSP in error due to the block copy from
pi-dev-plus-aosp

Test: run unit test
Change-Id: I9e206465e832d263084895b7c7b808df4573f986
Merged-In: Iab11567fa326b6837a621c1edc93e0eceb04ec43

Merge "Fix potential null point dereference"
am: 6bb3296e5e

Change-Id: I92ba1186cf0517023faafc1251b34bc513522000

Merge "Fix potential null point dereference"

Fix potential null point dereference

Use find() to check element in map to avoid unknow element been insert
into map causing null point dereference.

Bug: 121310341
Test: Manual connect/disconnect headset
Change-Id: If127086ec4beb9b735ef3c72456e8e2209826b57

Merge changes Ice4cee21,I6e579811,Ieae0cb21
am: 7c12a1727d

Change-Id: Ie2fd9170d55c4ef2cf2a7b7864489e37dac17e6a

Merge changes Ice4cee21,I6e579811,Ieae0cb21

* changes:
  RootCanal: Add more scripts
  RootCanal: Desktop simulation envrionment
  RootCanal: New Directory Structure

Merge "Get gd os tests running on host"
am: d4e4cc2f51

Change-Id: Icc5eb98cf87d1311b35a5730b5497a290cdace2d

Merge "Get gd os tests running on host"

Merge "Run code coverage only on host target"
am: 36277fcc67

Change-Id: I8dabacad4bc7c30001a55827c8f976704a89d549

RootCanal: Add more scripts

Test: Connect to HCI sockets and LinkLayer sockets
Change-Id: Ice4cee21f295bbb3a3fe35cfe44073028e65d811

RootCanal: Desktop simulation envrionment

Test: nativetest64/root-canal/root-canal [TEST_PORT] [HCI_PORT] [LINK_PORT]
      python scripts/test_channel.py

Change-Id: I6e57981182c392366d7d97249d837694b49dfa4e

RootCanal: New Directory Structure

model/controller/ contains the simulated chip
model/devices/ contains simulated devices
model/setup/ contains the test model
packets/link_layer/ contains the fake LMP packets

Test: cts-verifier Insecure RFCOMM test
      LinkLayerPacketBuilderTest
      rootcanal-packets_test_host
Change-Id: Ieae0cb21f7d57c03797f800797cedae59dd70e49

Get gd os tests running on host

* Run os tests on host
* Fix flaky hander clear test
* Move to linux_generic
* Remove dependency on libchrome
* Allow alarms to run on linux host where we can't get CAP_WAKE_ALARM

Test: atest bluetooth_gd_test_os
Change-Id: Ia95ce39169ac8c963052e3252e7b39ef32bd5bb1

Merge "Run code coverage only on host target"

Run code coverage only on host target

Bug: 129421924
Test: run code coverage
Change-Id: I1dc681c72d8af29e43ccb1441ffd218059b8056c

Merge "BQR: Fix for the issue that Bluetooth keeps on crashing if the controller firmware does not support the BQR feature."
am: c19e159920

Change-Id: Ieb9dfdb35d8740114c2a94e0453de6762ef4ecc9

Merge "BQR: Fix for the issue that Bluetooth keeps on crashing if the controller firmware does not support the BQR feature."

Merge "Only use the new format in gd/"
am: 8b3df83108

Change-Id: I4129cc39765902b3aec64180c814e8105ef95693

Merge "Only use the new format in gd/"

Merge "Copy reactor classes to new directory"
am: 10a6432c6a

Change-Id: If973570cbc40c3110cb9a0caa63ad2c461523aa6

Merge "Copy reactor classes to new directory"

Only use the new format in gd/

Test: Upload a change with long lines
Change-Id: Iba4a08f9556219bfada9a13272463acbf7ddbdce

Merge "A2DP: Initialize the UIPC HAL only when those binderized HALs are not enabled"
am: 2ef65b9065

Change-Id: Ic82f8948428d14e4761892ad53334b2bacdd00d3

Merge "A2DP: Initialize the UIPC HAL only when those binderized HALs are not enabled"

Copy reactor classes to new directory

Test: compile and run unit test
Change-Id: Ie704bf6b80f24bcae86d557ca351a893dcf8bdfa

Merge "Hearing aid: use new common/Timer"
am: 39eba7165c

Change-Id: I31592f1e4ea83cb41e94b50ce76c1e578dffa35c

Merge "Hearing aid: use new common/Timer"

A2DP: Initialize the UIPC HAL only when those binderized HALs are not enabled

When using BluetoothA2dp / BluetoothAudio HAL, the UIPC won't be used,
and is no need to do the initialization. If it is up, there will be an
uipc-main thread running, and we have to release by the UIPC_Close API
before the A2DP source restarting. If we are acquiring a new one before
released, some of its resource will be leaked, and causes the stack
abnormal.

Bug: 128256722
Test: Check uipc-main is running when using the legacy HAL only
Change-Id: Icc8ea102d29c92b58c77099979d17e85e5cb9a83

Add test about post/clear task from callback for handler
am: 13369ff8d1

Change-Id: Iafc38c79a208bd37c3c9872d3e0db1f43ca21cd6

Add test about post/clear task from callback for handler

Test: sudo ./bluetooth_test_common
Change-Id: Ibbf8622a705d3523638ef6cc9e6a62130b877637

Hearing aid: use new common/Timer

Previously osi/alarm cannot be cancelled or freed while in a callback.
Replace it with common/Timer. It's safe to be cancelled in a callback
and well tested.

Bug: 119533256
Test: manual
Change-Id: I73c7c98d683c85ec8f0f4256c3e4bccf3e11b56a

Merge "Fallback to the legacy HAL when IBluetoothAudioProvidersFactory is unsupported"
am: 86ef0c76da

Change-Id: Ie257f7cad6ed4180e8ae3ea858b52edcdbd06f72

Merge "Fallback to the legacy HAL when IBluetoothAudioProvidersFactory is unsupported"

BQR: Fix for the issue that Bluetooth keeps on crashing if the
controller firmware does not support the BQR feature.

Considering for the Treble case, it might upgrade the system image
only and the BQR feature will be enabled on a device whose Bluetooth
controller firmware does not support the BQR feature. The change is
not to crash the system if the controller firmware does not handle the
BQR VSC.

Bug: 129037162
Test: Enabled the BQR feature on the device whose Bluetooth controller
firmware does not support the BQR feature. The Bluetooth can be turned
on and works normally.

Change-Id: I6e9310a2b604679ba300e558ae4482887501e022

Merge "Refine the log messages of audio_bluetooth_hal"
am: cb7b7e432f

Change-Id: If20d0574941b1a3d24119d24b862e30d571ede90

Merge "Add a workaround to play A2DP SBC Mono"
am: 532316c0ae

Change-Id: Ic7f2a1442ede61a7720f2e7c25cda02efc551411

Merge "Refine the log messages of audio_bluetooth_hal"

Merge "Add a workaround to play A2DP SBC Mono"

Merge "Add event_value to BluetoothClassicPairingEventReported"
am: a0060ceed6

Change-Id: I9d2823e129b28fc2b59c788a8245018d07226679

Merge "Add event_value to BluetoothClassicPairingEventReported"

Merge "Use PLOG rather than LOG strerror(errno)."
am: fa450772a8

Change-Id: I4fa20e7921ba336ddc866e3596e1f1e602eea5d6

Merge "Use PLOG rather than LOG strerror(errno)."

Refine the log messages of audio_bluetooth_hal

This CL follows the review suggestions at aosp/920718 and aosp/925233 to
refine logs.

Bug: none
Test: check logs manually
Change-Id: I3c57afaf6d416752687a7a697a310fe98c8ee495

Add a workaround to play A2DP SBC Mono

There is a similar WAR of aosp/522661 at A2DP legacy HAL. In order to
suport MONO channel mode, the PCM audio is pulled as STEREO and mixed
into MONO by the Bluetooth Audio HAL.

Test: Playing SBC mono with Headset
Bug: 127593318
Change-Id: I78f3973ba6c8c733dc18122288a915daed97be65

Add event_value to BluetoothClassicPairingEventReported

* Add an event_value field to log status value such as
 - encryption enabled state
 - simple pairing mode
 - secure connection host support
 - delete all flag
* Use the event_value field to log above information during
  classic pairing process instead

Bug: 128966402
Test: make
Change-Id: I2a87c3837754bfc5bcd55f72325400c428c2ca25

Add Rx RSSI logs for Hearing Aids
am: 10c80b5ce2

Change-Id: I1bbded7fff9ddf1c4d1bb637df1d86d565371d07

Fallback to the legacy HAL when IBluetoothAudioProvidersFactory is unsupported

Because the stack may run without the new BluetoothAudio HAL like GSI
under old devices, it will be nullptr to getService from the
IBluetoothAudioProvidersFactory in such condition. We take nullptr as
unsupported, and fallback to the legacy HAL.

Bug: 128419724
Test: Manually running A2DP and hearing aid with / without the HAL
Change-Id: I606abc3e5b63b7857c3307c879fd4cbe46dd05d9

Add Rx RSSI logs for Hearing Aids

When there is a data buffer flush (which indicates data congestion),
the received RSSI will be queried for this connection multiple times and
logged in the dumpsys.

Bug: 124331686
Test: Manual test
Change-Id: I686f4e34bda3f8067b42d6b41ca8bf316a5bf6f1

Use PLOG rather than LOG strerror(errno).

Test: builds
Change-Id: I75c3311ce00fccc79b8efc198ce69ffed96e805a

Merge "Allow to disable the rootcanal test console with a property"
am: ddd6a6f7c0

Change-Id: Id29f2fbec572e3cbab7273a8223caf7394414add

Merge "Allow to disable the rootcanal test console with a property"

Merge "No need to abort the audio HAL / Bluetooth when failed to suspend"
am: 0cc83418b1

Change-Id: Ia71c87eb6fe33267eee4339d759ee96edaf3ee0a

Merge "No need to abort the audio HAL / Bluetooth when failed to suspend"

Merge "Move controller_properties.json to the vendor image"
am: 6f80399e14

Change-Id: I0810d6c2249370828dd442b8c953932902d01dbf

Merge "Move controller_properties.json to the vendor image"

Merge "compiler based array initialization"
am: e9220762b0

Change-Id: I03324d9d61a934dc142153f6e64b0f7d9aca53cd

Merge "compiler based array initialization"

Merge "Recover from HandsFree client connection collision"
am: 3081c2e81a

Change-Id: I9cddf7fc2a827b20f18393ff62d1cb3fd0032320

Merge "Recover from HandsFree client connection collision"

No need to abort the audio HAL / Bluetooth when failed to suspend

When the headset was disconnecting, the audio HAL may not be able to
suspend the stream successfully, and it is no need to abort the process
for such acceptable failure. This change also adds extra log messages
about HIDL status.

Bug: 127654107
Test: Play / pause music via the Bluetooth manually
Change-Id: I335fcf75708343c7971ebd97514fea5db50f17a0

Merge "Release encoder state when cleaning up the hearing aids instance"
am: 714a57639a

Change-Id: Iebef7301770d31cd38df3007b49aa52873aa0e29

[automerger skipped] Merge "DO NOT MERGE - Merge PPRL.190305.001 into master"
am: 01f16ea9f7 -s ours
am skip reason: subject contains skip directive

Change-Id: I23b7ed3c6c0b6f9af14443698fe7918d58269595

[automerger skipped] DO NOT MERGE - Merge PPRL.190305.001 into master
am: 145c825e98 -s ours
am skip reason: subject contains skip directive

Change-Id: I23b7e1711f6351d09973e25d9defd9a6fefd4032

Merge "Release encoder state when cleaning up the hearing aids instance"

Merge "DO NOT MERGE - Merge PPRL.190305.001 into master"

Allow to disable the rootcanal test console with a property

Rootcanal's test console is in a tcp socket which is forbiden for HALs
by the Android security policy.

Bug: 128355308
Test: run in cuttlefish, ensure no 'avc: denied' messages show up
Change-Id: I74bf07c34166d2df399f33194cbb41e57178edc2

Move controller_properties.json to the vendor image

Bug: 128355308
Test: build
Change-Id: I3cb9140b34af18569a983122eef3bd642516440c

compiler based array initialization

Have the caller null out the array rather than rely on the callee
performing a memset.

Bug: 121194976
Test: compiles. No runtime tests performed.
Change-Id: I1ce3f0530080769b97be0af313e822b74fd375ce

Recover from HandsFree client connection collision

If a HF client fails to connect (due to collision), clean up state such
that subsequent connection attempts will pass.

Bug: 118422992
Test: Connect HF client to AG
Change-Id: I093cf3d229956c66d9e8501eb882071600321555

Release encoder state when cleaning up the hearing aids instance

When the Bluetooth state changes from ON to BLE_ON, the hearing aid
instance was cleaned up, but not encoder state. Since the Bluetooth
process is kept at BLE_ON but not exited, the new instance of the
hearing aids would see the encoder as initialized without starting a new
Bluetooth audio session, and caused the audio HAL to be unable to talk
to the stack. We now reset the encoder state when cleaning up the
instance, so it will start a session next time during the first
connection of a new hearing aid instance.

Bug: 127610666
Test: ON / OFF BT with BLE_ON and switch active device manually
Change-Id: I426fed4ea22c0b858bee273727fca6e2e7481e84

Merge "Hearing Aid Dump Audio Tool: Add No Start Cmd feature"
am: 812ea5b2a4

Change-Id: I4ac4ba987795a949cb455f6e7159351045b77dd7

Merge "Hearing Aid Dump Audio Tool: Add No Start Cmd feature"

DO NOT MERGE - Merge PPRL.190305.001 into master

Bug: 127812889
Change-Id: I733a17b7dd1e24a6f4a19aac2d45e312b15e0199

Merge "DO NOT MERGE - Merge PPRL.190305.001 into stage-aosp-master" into stage-aosp-master

Hearing Aid Dump Audio Tool: Add No Start Cmd feature

When the HCI Snoop logs wraparound, the Hearing Aid Audio Control
Command "Start" is lost. For the case, we can enable No Start Cmd
feature to set a fake "Start" to extract audio data.

Bug: 127745964
Test: ./dump_hearingaid_audio.py -c1 1 -c2 3 -ns true btsnoop_hci.log
Change-Id: Id7f67da0d5476faf38352ab57339fed8e358bac6

Merge "Use a weak pointer to deliver updates to AVRCP devices."
am: 80eeba2af3

Change-Id: Ie9d8501153264be4a755ae88956b585df3c1bdad

Merge "Use a weak pointer to deliver updates to AVRCP devices."

DO NOT MERGE - Merge PPRL.190305.001 into stage-aosp-master

Bug: 127812889
Change-Id: I433e1bf1f64c8be7187bda2c22a2554a936e4d43

Merge "osi: explicitly release wakelock during turn off"
am: b5414b30a0

Change-Id: Ib3b886cfca8edc7faf4a3484e3fa80e14d799616

Merge "osi: explicitly release wakelock during turn off"

Snap for 5240760 from b9dd3863033facdd8608904e4dd59ff3cc52871b to pi-platform-release

Change-Id: I8098c6f610e684be5b84a984b163088d255420b8

Use a weak pointer to deliver updates to AVRCP devices.

If a device disconnects right before a update message gets queued, the
device becomes null and there is a crash when the callback for the
update executes on the disconnected device. This patch switches the
device reference from being Unretained to using a weak pointer so that
the callback just doesn't execute if the device is disconnected.

Bug: 120431125
Test: Use the same test as b/120477414 as that bug causes a disconnect
at the same time as a media update.

Change-Id: I1dcc08e5c9866106e7ec0dad52505e34b42da600

osi: explicitly release wakelock during turn off

Ensures we release the wakelock when turning off if any module hasn't stopped timers properly

Bug: 123289721
Test: atest net_test_bluetooth
Change-Id: Ibec5c262d2ec33ce22f2c8bb2a9b838e8d6b90a7

AVRCP Controller Play Position Changed
am: 6f22c6e83c

Change-Id: I0aeb82a4a8fc1619626d1b546adb8803bd2c1525

AVRCP Controller Play Position Changed

Implement code to register for the AVRCP notification
EVENT_PLAYBACK_POS_CHANGED such that playback position gets update on
skip to beginning, and during audio scrubbing from phone.

Bug: 119119967
Test: Play music, scrub audio to another position on phone, observe
playback position is in sync.

Change-Id: Ib8459dece7629622649ba6ed340dcd697c700b09

Fix for connection attempt after connection cancel
am: 27870c9803

Change-Id: I53d5bce8962f5e3789f0ae41c39bc916ffe85ca7

Fix for connection attempt after connection cancel

When connection attempt is canceled, we put the transport control block
(p_tcb) into CLOSING state, but we never close or free it.
For LE devices that have not established the connection, it's safe to
go straight to CLOSED state and free the p_tcb.

This issue was introduced during connection manager refactor.

Bug: 127242749
Test: GattConnectTest
Change-Id: I8e45ead9b071c60e95d2e60dcea8afdbaefb92b1

Merge "Clear the link key when releasing security records"
am: 6c228c5b58

Change-Id: I5989cec3929a07dbbf36d112c329ffb5c2d37433

Merge "Clear the link key when releasing security records"

Merge "Fall back to CLOCK_BOOTTIME if CLOCK_BOOTTIME_ALARM fails"
am: bbec743e47

Change-Id: I831f7c3f113b807e4ba73169d3ad00d2770e02b8

Merge "Fall back to CLOCK_BOOTTIME if CLOCK_BOOTTIME_ALARM fails"

Clear the link key when releasing security records

Test: compilation
Bug: 127479372
Change-Id: Ia571a2048fa5ca85c45b03b39d398e480aadb55a