Merge "PDL: Fix Iterator assignment and add Subrange"
am: b8f84dd241

Change-Id: I2bc18c7f5347a4cf382e20f27ba479540f0e7aed

Merge "PDL: Fix Iterator assignment and add Subrange"

PDL: Fix Iterator assignment and add Subrange

Test: bluetooth_test_gd, bluetooth_packet_parser_test
Change-Id: I28c264bbbc25c8980a37fc2cb85875f50ec4a7ff

Merge "Remove direct connection after notifying the connection up"
am: 9c973884f4

Change-Id: I405789b12c80661afd7347e25a3eb433b0375abf

Merge "Remove direct connection after notifying the connection up"

Merge "GD: HCI Security interface"
am: faa3ceeb4c

Change-Id: I9374f57527b6589b29ecdb296aae13c8acf6425c

Merge "GD: HCI Security interface"

GD: HCI Security interface

Fixes: 137138879
Test: run run_cert.sh
Change-Id: I73ee7e6191d454600c693bd92cc1ea5933ee1379

[automerger skipped] Merge "DO NOT MERGE - Merge pie-platform-release (PPRL.190801.002) into master"
am: fa38805581 -s ours
am skip reason: subject contains skip directive

Change-Id: Ib9dab319ab9109075c28aa768c4a1140e68d256c

[automerger skipped] DO NOT MERGE - Merge pie-platform-release (PPRL.190801.002) into master
am: 52dedf5c30 -s ours
am skip reason: subject contains skip directive

Change-Id: I6677e5a3f53c49c7da35a1a62b20e92fc90d3023

Merge "DO NOT MERGE - Merge pie-platform-release (PPRL.190801.002) into master"

DO NOT MERGE - Skip qt-dev-plus-aosp-without-vendor (5713463) in stage-aosp-master

Bug: 134405016
Change-Id: I513b6ae3eae7b6224d1b7a02ce6323642518f8e4

L2CAP: ClassicFixedChannelAllocator
am: 4eb5631e26

Change-Id: Iaeb5cda4b61b25a414ffd5a0c6d8e8a5727712c6

L2CAP classic fixed service manager
am: 7be4ea7b3a

Change-Id: Ia1311b22d24c7589102983aab64f6d0048cb33e6

DO NOT MERGE - Merge pie-platform-release (PPRL.190801.002) into master

Bug: 139369544
Change-Id: Id4051603d6abdd5c5ad9da3e5a4d969c36e0faa0

L2CAP: ClassicFixedChannelAllocator

This is a utility class for L2CAP link to manage channels

Test: bluetooth_test_gd
Bug: 138261142
Change-Id: I1adaaf29817371bb3f0e68d64f6aec3d3eea2c85

L2CAP classic fixed service manager

Implement classic fixed channel service registration

Test: bluetooth_test_gd
Change-Id: Ic5d5b4543e88a0e4b9918f4583c5d115ef021cf0

Snap for 5674421 from a3c00b27470c47bae945fd6d222b52519375160b to pi-platform-release

Change-Id: I67f7bdc0b565214793359d40055831bc6bc5b309

PDL: Save copies in packet builders
am: c5fda6b45d

Change-Id: I71e8a0b35656af9b61a28de52b45882b79757bad

PDL: Save copies in packet builders

Test: bluetooth_packet_parser_test
Change-Id: Ibf673a73109d003819445cb980d0fc85b1b7557c

L2CAP: Add Classic Fixed Channel APIs
am: b4b4a37ae1

Change-Id: Ie40b5210c2951e3350045495dc17eb00fcc8fb1e

L2CAP: Add Classic Fixed Channel APIs

* Add ClassicFixedChannelManager
* Add ClassicFixedChannelService
* Add ClassicFixedChannel
* Modify cid and constants definition

Bug: 138261142
Test: bluetooth_test_gd
Change-Id: Iae4e398e938436c74d16b49834a4bd5146b03873

Remove direct connection after notifying the connection up

After notifying the connection up, remove direct connection related with
this device to make sure no client is directly connecting the device in
connection_manager.

Bug: 138819537
Test: 1. Repair BLE device 2. Reconnect BLE deivce
3. reboot DUT, make sure Hearing Aid can be connected.

Change-Id: I30ff7c47445ec2a3dbd9702ca79873063210e6df

Merge "PDL: Test fixed-size struct fields"
am: b818515d99

Change-Id: I45eb5bd636e6b6cea778f539cdb0e241009566d4

Merge "PDL: Test fixed-size struct fields"

Merge "BluetoothAudioHAL: Fix the latency and the number of frames were inconsistent"
am: 606974ddc8

Change-Id: I2fc6c800687ed564bdaf611d53abc79f9d98b7ac

Merge "Do not config A2DP codec priorities before retrieving all capabilities"
am: 1131fd98ba

Change-Id: I0e7592d855f0f2d7fc97a19e8e23dfbb260169e6

Merge "BluetoothAudioHAL: Fix the latency and the number of frames were inconsistent"

Merge "Do not config A2DP codec priorities before retrieving all capabilities"

Merge "Fix the test case HCT  BV-01-C"
am: e28586ea5b

Change-Id: I1289d8c4802f673bfff967fbd38050ae8fbbd9ed

Merge "Fix the test case HCT  BV-01-C"

PDL: Test fixed-size struct fields

Add an end_index for bounds checking fields, and use that instead
of subviews, to support extraction in struct Parse() functions.

Bug: 138260498
Test: bluetooth_packet_parser_test --gtest_filter=*FixedTypesStruct*
Change-Id: I1e6819aafdd31d53de046299e7d17fe7dd2de197

BluetoothAudioHAL: Fix the latency and the number of frames were inconsistent

The Bluetooth Audio HAL uses 3 APIs to share the audio latency to
audioserver, so they can do the audio / video synchronization. Those
API's results have to be consistent in the presented number of frames,
and should be reset only when the output was re-opened. Because the HAL
queried information from the Bluetooth which might be reset after the
Bluetooth session restarted, and caused a conflict between current and
previous results. The media frameworks needed more works to be A/V
synchronous again, and users would see the video was frozen.

This CL checks the stack data, and there must be no big delta between
the stack and HAL, or one of them was ever reset, and needs to use local
counters instead.

Bug: 137978401
Test: Switch active device and codec manually
Change-Id: I35bffa834c0de2e8b36e99a96b4a70738cc2b639

Do not config A2DP codec priorities before retrieving all capabilities

* In current A2DP design, if Java A2DP service disabled optional
  codec and invoke native SetCodecUserConfig() while our get codec
  capabilities process is not finished, stack would raise SBC
  priority to 1000000 without notifying the remote device. This
  behavior would cause phone and the remote headset using different
  codec while A2DP streaming and lead to no sound problem.
* We should always check the codec capabilities status before changing
  the codec priority.

Bug: 124254557
Test: manual

Change-Id: I19693ecdd581836829c15ae37d579a4cb3ac0200

Fix the test case HCT  BV-01-C

the value of p_dev->get_rpt_snt may become 255 after p_dev->get_rpt_snt--
if it happens to be 0

Bug: 138261469

GD HCI: register events for gd test on device
am: aa235aad54

Change-Id: Ief033e7015a0d0ddcab5446ef1f64f9ad306b4e2

GD HCI: register events for gd test on device

Bug: 138260216
Test: run run_cert.sh
Change-Id: I7838fec1a7cfcc9a6e82830e81b7f9308f3654ce

Fix CoC not being able to establish first connection
am: 3118cdcb90

Change-Id: I3bfcb361c026808282780a179f14b529495cc6f6

Fix CoC not being able to establish first connection

When establishing conenction, we must know address type of the device we
connect to. We learn it during scan. We store this information in BTIF
layer, and later copy it into BTM if someone tries to make connection.

CoC code was not copying address type information between layers. This
caused us to use bad address type, which resulted in no connection being
created.

This patch fixes the issue, by making sure address type is properly
copied from BTIF into BTM

Test: acts BleCocTest
Bug: 133444088
Change-Id: I08609d92df245649882ad0b186a6080bff61a96e

Merge "Hearing Aid: Remove std::promise from suspend / resume calls, and stop audio ticks by no devices"
am: 8df3472243

Change-Id: I37d7dde72aaf66d7ca791c30d66a608eebac7c8a

Merge "Hearing Aid: Remove std::promise from suspend / resume calls, and stop audio ticks by no devices"

Modify log format
am: 02d91afc68

Change-Id: I06af07140653cf7ddb04c48baf70b9607fca7aca

Modify log format

Always log file name and line number.
Instead of __PRETTY_FUNC_ use __func__ and log just the function name.
Logging full name with namespace, return value and parameters produces
unreadable gibberish.

This makes it easier to map log line to line that produced it.

Test: compile, look through the logs.
Change-Id: I3a72e8e498ded604fe61179191f1bc98b44c6a5c

Rootcanal: Allow using TCP port when TIME_WAIT
am: 33a39edd25

Change-Id: If56e88c68ed6bb7bc4a45b52d7fa9027c9972e4a

L2CAP: Add L2CAP Module
am: fcb2bbfa76

Change-Id: I794fb1e289ac44a5badb07a46f0afe2f716fb25b

Rootcanal: Allow using TCP port when TIME_WAIT

Test: cert/run_cert.sh
Change-Id: I473add448fcd167f6e33b34281298c69a831d733

L2CAP: Add L2CAP Module

* Add an empty L2CAP module
* Add facades for L2CAP module
* Add cert facades for L2CAP module
* Add SimpleL2CapTest to run_cert.sh

Test: cert/run_cert.h
Change-Id: I00b3fe1bfb6e72dbbf1ca441ceed7dd4d62afb26

Merge "AVRCP Controller Shuffle/Repeat support"
am: 8074a57ca9

Change-Id: Ia812910183625cc3c9628d55951df291de405cd1

Merge "AVRCP Controller Shuffle/Repeat support"

HCI: Use futures to wait for packets in tests
am: cad5ded518

Change-Id: I23f6b1b1cd6e48a077aaedf2fb697dc497dd5653

HCI: Use futures to wait for packets in tests

Test: bluetooth_gd_test on a device
Change-Id: I3e61b83df47204380398646350b90c3f63734bca

AVRCP Controller Shuffle/Repeat support

Fetch player settings when the addressed player changes.

Bug: 72495707
Test: AvrcpControllerStateMachineTest#testShuffle
      AvrcpControllerStateMachineTest#testRepeat

Change-Id: I4df1f130dafb96708ec7a0a2c13c776affcc6318

Merge "HCI: Print unhandled event type"
am: b9211ab38d

Change-Id: Iffaca77ec0edbe2d645645e69850c1e48892bffd

Merge "HCI: Print unhandled event type"

Cert: Run on device
am: 486b6a72a5

Change-Id: Iadf4731f08166f9bce0e25ff77fe05854bea9acc

HCI: Print unhandled event type

Test: send unhandled event
Change-Id: Iafdf004609cae4e15861664eb8e430e263655250

Cert: Run on device

Add a sample device_config.json to allow cert testing on device.

1. Fill in Android device serial number in device_config.json.
2. Build device target for libbluetooth_gd, libgrpc++_unsecure,
stack_with_facade, and bluetooth_cert_stack. Push them to the device
under /system/lib64 and /system/bin correspondingly.
3. Set up Android tcp forward and reverse
   - For DUT:
     adb -s <DUT> forward tcp:8899 tcp:8899
     adb -s <DUT> forward tcp:8897 tcp:8897
     adb -s <DUT> reverse tcp:8895 tcp:8895 <- host is the server
   - For CERT:
     adb -s <DUT> forward tcp:8898 tcp:8898
     adb -s <DUT> forward tcp:8896 tcp:8896
     adb -s <DUT> reverse tcp:8894 tcp:8884 <- Host is the server
4. Run cert/run_device_cert.sh

Bug: 138260216
Test: cert/run_device_cert.sh
Change-Id: Ic2921a31910e370d86c4aa4143e92b5b15d8a1e8

Merge "HCI: Wait for commands using std::future"
am: 3388ba9bbc

Change-Id: Ie314fc89eed1868b01f494b33f5c3f5898483abc

Merge "HCI: Wait for commands using std::future"

Merge "HAL cert and facade: Unregister HCI on shutdown"
am: 7145059a37

Change-Id: I8034f577f6d12a92fc2c433b54fdf2bd1e53c244

Merge "HAL cert and facade: Unregister HCI on shutdown"

HCI: Wait for commands using std::future

Test: bluetooth_test_gd --gtest_filter=*AclMan* on a device
Change-Id: I915242546efdfaf6553e36cbe19a37290435991b

Merge "OS: Account for enqueue/dequeue race"
am: 0fa25c6e8f

Change-Id: I2c139ed7ab8c49ab5ec3c9b56a83333c9a2ef893

Merge "OS: Account for enqueue/dequeue race"

OS: Account for enqueue/dequeue race

Test: on-device: bluetooth_test_gd --gtest_filter=*Queue*
Change-Id: I3429e35ebd9a98f1054e40e9507995daeae35c65

Merge "PDL: Use GenBounds for Structs"
am: a9a37cf504

Change-Id: Ic3cb61103a863b0b7f64202ee1edd25cd5e11986

Merge "PDL: Use GenBounds for Structs"

PDL: Use GenBounds for Structs

Test: bluetooth_packet_parser_test
Change-Id: Ibb6e7bd441b8fd19f17818dc74ac4aff10b519a4

HAL cert and facade: Unregister HCI on shutdown

Test: cert/run_cert.sh
Change-Id: I730b26ec7a15f4ec539c42e16a774c442be93ac9

Merge "Support vendor AT command and response for Hfp Client in bt stack"
am: 5b2eb835ac

Change-Id: Ifd860114ad0c7d76388a146c21c1c9214b0dc9b7

Merge "Support vendor AT command and response for Hfp Client in bt stack"

Cert signal server: Use TCP instead of UNIX socket
am: defcdca4ee

Change-Id: I64fc300263102e366d1e7808c457ed5aa7cab22e

Cert signal server: Use TCP instead of UNIX socket

Since we want to run the stacks on the phone, we need to use TCP socket
and set up TCP forwarding to run the test.

Test: cert/run_cert.sh
Change-Id: I3fd6c17318d372ab0be4065df4aea2a5bb469ab3

Merge "PDL: Add Structs"
am: df2d6a5ac4

Change-Id: I59078b063e01f04e9b87c2248dc4b93115af2d5b

Merge "PDL: Add Structs"

Hearing Aid: Remove std::promise from suspend / resume calls, and stop audio ticks by no devices

The Hearing Aid uses the stack main thread to be the media task thread.
This thread handles events / feeds audio data sequentially, and won't
have a race condition within the single thread. The promise is a bit
redundant, but wasted time to cause a potential deadlock that the caller
waits for the task thread finished which needed the caller's resource.

This CL also stops the audio ticks when no devices available, and no
need to wait for next timer expired. Besides, move all timer controls
into the media task thread, so timer is started / stopped at the same
thread, too.

Bug: 134996542
Test: add big delay within functions, and check no abnormal manually
Change-Id: If0b279a191d86b30db41a682dc7f0ad9470cb10e

Merge "OS: Get promise before registering in tests"
am: 3edb0f069e

Change-Id: I518e0a718a4b7b54051b03061d2c50fea0330694

Merge "OS: Get promise before registering in tests"

PDL: Add Structs

Test: bluetooth_packet_parser_test
Change-Id: I2d0288aa4e62589dd03d93a1b47ce0585584cab5

OS: Get promise before registering in tests

https://cplusplus.github.io/LWG/issue2412

Test: on-device: bluetooth_test_gd --gtest_filter=*Queue*
Change-Id: I2603763ea6c9197ed56f741b2beb100c1a9b8e19

Merge "OS: Use LOG_ALWAYS_FATAL for Android"
am: ff41bee660

Change-Id: Ida4de285e9a95edda049e5491f8292af7a4a4c15

Merge "OS: Use LOG_ALWAYS_FATAL for Android"

Merge "Split ARRAY into fixed size ARRAY and VECTOR"
am: a413eb0d55

Change-Id: I15a615f553228515bb56ff25eb72dce1f86fcec5

Merge "Split ARRAY into fixed size ARRAY and VECTOR"

OS: Use LOG_ALWAYS_FATAL for Android

Test: bluetooth_test_gd --gtest_filter=*Death*
Change-Id: I0aa4e85a7fd000eb90c65c51d3ac372634583c69

Split ARRAY into fixed size ARRAY and VECTOR

Test: bluetooth_packet_parser_test
Change-Id: I438d7583fdb99054528b99a3a3a12ec58d115cb3

Remove unnecessary include_dirs
am: 2e78b34dc2

Change-Id: I0f84ab8cf140eca9d19f85d53c8697c88e144801

Remove unnecessary include_dirs

The header directory specified in the include_dirs property is already
included via the header_libs property.

Test: m avrcp_headers
Change-Id: Id3af155e64127c5bc14a17780fa77a4c6147e5d8

Merge "Disable absolute volume for AirPods 2 with MAC address from 9C:64:8B"
am: b185bdefe3

Change-Id: Ica5d6128c9bf0a248a257698b9078703ac04cfe9

Merge "Disable absolute volume for AirPods 2 with MAC address from 9C:64:8B"

Merge "L2CAP: Cid, Mtu, SignalId type and constant"
am: 5cc572ebe4

Change-Id: I3a63d7a11ebbe98c1b7e278a8e121c73a7ff263f

PDL: Extract CustomFieldFixedSize
am: cff15c1724

Change-Id: Iece7efa42bcad1408dcf38ee271c2cb639d37d6f

PDL: Add GenExtractor functions to fields
am: cbfe6cf599

Change-Id: I693c40f07ada6843826b4f98bfa85547bfbb57c0

Merge "L2CAP: Cid, Mtu, SignalId type and constant"

PDL: Separate member and parameters for arrays
am: 835bf7b516

Change-Id: I87195571cd4c7ca02d959137cbc5c4037e90cc94

PDL: Extract CustomFieldFixedSize

Test: bluetooth_packet_parser_test
Change-Id: Ia93442ca1cc173d0b6c813a45c722f14d2f34fa0

PDL: Add GenExtractor functions to fields

Test: bluetooth_packet_parser_test
Change-Id: I6ee3ae82e8e7949f0fd30dfae6f322dcc4769190

PDL: Separate member and parameters for arrays

Test: bluetooth_packet_parser_test
Change-Id: I7c23d29548d63f7c1e23281585b10fd9f43bdad4

PDL: Factor out insert functions
am: e7d1e71984

Change-Id: I1512d0d283f5c14d1450dff119019401b263157b

PDL: Add non-static methods to checkers
am: 2569ca78e5

Change-Id: I85954118348f195d3b33a29d7a7b8925f0558d40