Merge "AVRCP Controller manage active device"

Merge changes from topic "gd_sm_ui_wiring"

* changes:
  SM: Add UI Callbacks to CPH
  Security UI Callbacks wiring

Merge "Remove redundant BTA_AR_DEBUG define"

Get rid of Scanning shim

Bug: 149757450
Change-Id: I8f41bb6d79324bab6dbe40ea5223f42431c57638

Merge "L2CAP: Add test scripts for pts test"

Merge "L2CAP: Update remote tx_window during channel configuration"

Remove redundant BTA_AR_DEBUG define

Bug: none
Test: compile
Change-Id: Id63e2f9a8881c16b2b48eeef2b941f7c0cde628b

SM: Add UI Callbacks to CPH

Test: bluetooth_test_gd
Change-Id: I8b7b587f720a897e416ed3abea0fde6ad6788338

Security UI Callbacks wiring

Test: bluetooth_test_gd
Change-Id: I713e4efd923359a36fc943362d59ae619dd515ca

Merge "Fix crash at unexpected code after root inflammation"

Get rid of Storage shim

Bug: 149757450
Change-Id: I90130121c5fdf23ec8cacf19fedd4b3e154351a5

Get rid of Connectability shim

Bug: 149757450
Change-Id: Idef691e97109e2634fc05fc7bddfeca72b967d11

Get rid of Page shim

Bug: 149757450
Change-Id: I9ce812831a0d8ff62772ea5b98ab4a7e646b162a

Get rid of Security shim

Bug: 149757450
Change-Id: Id7137d64059cf5d15cd94419255c4b1f051bee60

smp_generate_stk: reduce log level for key type information

Change-Id: I7499a3f507914db9cb9c062f4ad9cfccdb51f06b

Merge "gd: Unregister events at the hci layer"

Merge "Save bonding and pairing state to security record"

Get rid of Discoverability shim

Bug: 149757450
Change-Id: Ibe6cccc0971d4a723ba5489332ca7933e7609a34

Get rid of the redundant shim interfaces

It was agred to expose gd types into the system/bt/main folder.
All the interfaces already use pimpl pattern to make them testable.
Thus there is no need for the I* shim interfaces.

Test: updated tests
Bug: 149757450
Change-Id: I9662f9ab48c61b764e4df082da1a73483cb0b2b9

Save bonding and pairing state to security record

Bug: 142341141
Test: bluetooth_test_gd

Change-Id: I226d9f2f74e12d911fbb8e1f3d5924db7763bdaf

Canonicalize header include files

Bug: 147316415
Test: Compiles and Builds
Change-Id: I00316dba9cce8d727fc718250237e80ff87cdd00

Do not redefine logging if already defined

This patch lets us include gd code in system/bt/main. Currently if we
were to do that, we would get errors about logs being redefined.

Test: compilation
Change-Id: Ifc2c9eb913c311a50763a26f864dd91872c273cc

Merge "gd: Add shim access to name db"

Modify service_type constants to not collide with constants in GD

Test: compilation
Change-Id: Ieb9540a3c5ca390a1e04d20ae378d1ab06b33707

L2CAP: Add test scripts for pts test

Add following PTS test scripts:
  L2CAP/EXF/BV-01-C
  L2CAP/CMC/BV-01-C
  L2CAP/CMC/BV-02-C
  L2CAP/ERM/BV-01-C
  L2CAP/ERM/BV-02-C
  L2CAP/ERM/BV-03-C
  L2CAP/ERM/BV-08-C
  L2CAP/ERM/BV-09-C
  L2CAP/ERM/BV-10-C
  L2CAP/ERM/BV-11-C
  L2CAP/ERM/BV-18-C
  L2CAP/ERM/BV-19-C
  L2CAP/ERM/BV-20-C

Add cert tests:
  test_initiated_configurtion_ertm [L2CAP/CMC/BV-01-C]
  test_receive_i_frames [L2CAP/ERM/BV-02-C]
  test_acknowledging_received_i_frames [L2CAP/ERM/BV-03-C]
  test_transmit_s_frame_rr_with_poll_bit_set [L2CAP/ERM/BV-08-C]
  test_transmit_s_frame_rr_with_final_bit_set [L2CAP/ERM/BV-09-C]
  test_receive_s_frame_rr_final_bit_set [L2CAP/ERM/BV-18-C]
  test_receive_i_frame_final_bit_set [L2CAP/ERM/BV-19-C]
  test_recieve_rnr [L2CAP/ERM/BV-20-C]

Bug: 145784332
Test: run_cert.sh
Change-Id: If5b25880f731dbfbb0447a6edaccbbf3d8829d11

L2CAP: Update remote tx_window during channel configuration

* Update tx_window as transmit window in configuration request from
  remote.
* Fix logic of checking valid f bit, f bit should be valid only if:
  1. tx_state is in XMIT and f not set
  2. tx_state is in WAIT_F and f set to POLL_RESPONSE

* Add follwoing test cases in simple_l2cap_cert:
  test_resume_transmitting_when_received_rr
  test_resume_transmitting_when_acknowledge_previously_sent
  test_respond_to_rej
  test_handle_duplicate_srej
  test_handle_Receipt_rej_and_rr_with_f_set
  test_handle_rej_and_i_frame_with_f_set

* Add following test case in pts_l2cap_test:
  test_L2CAP_ERM_BV_05_C
  test_L2CAP_ERM_BV_06_C
  test_L2CAP_ERM_BV_13_C
  test_L2CAP_ERM_BI_03_C
  test_L2CAP_ERM_BI_04_C
  test_L2CAP_ERM_BI_05_C

Bug: 149182548
Test: run_cert.sh
Change-Id: I0828e570017b954980f23912937e36524814a0f6

gd: Unregister events at the hci layer

bt_headless with the pairing option fails

Bug:149524116
Test: blueooth_test_gd
Test: bt_headless
Change-Id: I65d9f928c6ff63e5ddde8380cae847d180876fac

Merge "PDL: Serialize nested packets in Python"

Merge changes Iea352d4b,Ic0737465

* changes:
  GRPC: Use server_stream_call.cancelled() instead of hidden API
  RootCanal HAL: Fixed race condition when tearing down rootcanal HAL

PDL: Serialize nested packets in Python

Bug: 148096951
Test: ./cert/run_cert_facade_only.sh
Change-Id: I2c54bb5d35dec9a9b5101165d2d1ccb22b116ef4

Fix crash at unexpected code after root inflammation

* Clean up the hci/startup timers after the receiving of root
  inflammation packet so they will not be scheduled again and
  expire before the abort_timer.
* Abort Bluetooth when HAL is dead with correct reason.

Bug: 144592765
Test: Manual
Change-Id: I1458defeebaac7951aa8c7f52decf5b6ca3ab4fc

gd: Add shim access to name db

Provides replacement for remote name request db
for legacy shim side.

Bug: 149524116
Test: bluetooth_test_gd

Change-Id: I69c8c65c01935ae371debedcef2d921cea7beb4a

Merge "Use mobly.asserts"

Merge "gd: Authenticate only if necessary"

Merge "gd: Add link layer collision hci error code"

Merge "gd: Add a remote name database"

Merge "gd: Ensure all events are handled for new classic pairing manager"

Merge "gd: Expand and log more acl events"

Merge "Set proper scan type before first discovery"

gd: Add a remote name database

Simple module to cache remote names.  Used by shim layer.

Bug: 149524116
Test: bluetooth_test_gd

Change-Id: I63e94e0ff1ea04c3ec35d502ac1de001a11f7b98

gd: Authenticate only if necessary

If a device are pairing but not bonding then
authenticate may get called twice, once when
the channel is set up and again when the
pairing manager is launched.

Bug: 149524116
Test: bluetooth_test_gd

Change-Id: Icddecd549030ade41e5679d46580881b56833d9e

gd: Expand and log more acl events

Bug: 146086425
Test: blueooth_test_gd

Change-Id: I8dc506eea0bbcc3a9310d758311f369a31993bfd

gd: Ensure all events are handled for new classic pairing manager

a new classic pairing manager never processes the initiating event.

Bug: 146086425
Test: bluetooth_test_gd

Change-Id: I3471bc5849315456d7429747e894cf80eda8b8cb

Merge "Update time stamp when new inq_db"

Merge "Remove some unused inquiry code and callback function"

Use mobly.asserts

Test: cert/run_cert_facade_only.sh
Change-Id: I3f19bba39953eca9c013aab8813610c3fbfb5f83

Get rid of UI callbacks from SecurityManagerListener

Test: updated unittests
Bug: 142341141
Change-Id: If6b086f74d92e757d02df4174af73edb36d6de3d

Merge "HCI: Set default event mask"

Merge "Cert: Move DUT assignment to common code"

Update time stamp when new inq_db

Test: Simulate a sitution with many LE devices around and start le scan
Bug: 146474336
Change-Id: I3a0f87786d0c7b13e4d85942ac0387fc334c26d4

Set proper scan type before first discovery

Set scan type when start first discover after bt enabled.
And replace adv evt hex code to define code

Test: Enable Bluetooth and then start discovery.
Bug: 146474336
Change-Id: I63c8cda5343de15381c83da4ea3afde6b75aaf6a

Remove some unused inquiry code and callback function

Test: manually
Bug: 146474336
Change-Id: I6d3a0d9066ef6a4fbc14f938db6f120147340804

Merge "Don't guess transport during pairing"

Security UI interface cleanup

We currently have two separate UI interfaces for LE and Classic, this
patch merges them into single one.

Bug: 142341141
Test: tests updated
Change-Id: I5fa347a1d4d50d85bb051782ca27f38f3d146239

Merge "Pass UI Into Security Manager"

GRPC: Use server_stream_call.cancelled() instead of hidden API

* Use server_stream_call.cancelled() to check if a stream
  is cancelled to avoid calling hidden APIs

Test: run cert tests multiple times
Bug: 148147248
Change-Id: Iea352d4b428b9b57b911576a44f7a91842613fc7

RootCanal HAL: Fixed race condition when tearing down rootcanal HAL

* One race condition exist when incoming_packet_callback_ is being
  reset while an incoming packet is being processed. This is fixed
  by adding a dedicated mutex incoming_packet_callback_mutex_
* Another race condition happen when HciHalHostRootcanal::Stop()
  tries to close the rootcanal FD while incomding_packet_callback
  is trying to read from it

Test: run cert tests multiple times
Bug: 148147248
Change-Id: Ic07374651c5ec38e90e7ddc545ca4fcd4a84cd04

Cert: Move DUT assignment to common code

Test: run_cert_facade_only.sh
Change-Id: I5ca6fd006294f00e06a4d6b52832a1d64aae8c9f

Don't guess transport during pairing

Test: pair to AirPods
Bug: 149278864
Change-Id: I689ba0a0966b3f7a2cc2ac9a29ddf694017bfb0b

HCI: Set default event mask

Bug: 145638034
Test: ./cert/run_cert_facade_only.sh
Change-Id: I6b5df371162222cda4c6219be56cb95611acb3e5

Merge "Initialize device_type before use it"

gd: Add link layer collision hci error code

Seen with pairing with two authentication requests

Bug: 145638034
Test: bluetooth_test_gd

Change-Id: I8af068f9a11a67fae00d4b108082a3208a44d95a

Pass UI Into Security Manager

Currently we have a static UI handler inside SecurityManagerImpl. Have
it passed from upper layers instead, to match what we will need in the
future.

Bug: 142341141
Change-Id: Iabc865a325b53c6455017347ade58ea23dd02a00

gd: Provide more info for unimplemented link responses

Provide the unimplemented event info as they occur

Bug: 146086425
Test: bluetooth_test_gd
Change-Id: I7b06be303cdb36748304775b9b8554d0c605288b

Initialize device_type before use it

We are using the default value of device_type when the transport type
is unkown, initialize it in case we get something weird.

Bug: 149353835
Test: Manual
Change-Id: I56d13fcebdf330356d1214cb095fcd4d5e658ff9

Merge "Cert: Use site-packages instead of root"

LE Pairing Handler wiring

Test: manual : single pairing with LE device should pass
Bug: 142341141
Change-Id: I4ec096e2d728d4b4caea1f4c4b5ba85cadaea0ae

Merge "Increase allowed error in repeating_timer_unittest"

Merge changes I5acd7ba9,If41d86e9,I4cea9ed3,I30a4ed0b

* changes:
  Cert: Always unzip for host tests
  HCI: Add LE AclManager tests
  HCI: Advertise an address with D for DUT
  HCI: Spell COMPLETE_ADVERTISEMENT

Increase allowed error in repeating_timer_unittest

Bug: 147469737
Test: atest bluetooth_test_common
Change-Id: I076192d55835b34300962e450407f8c21109ac1e

Merge "DeviceCert: Add timeout for some test cases"

Merge changes I685873b0,Iac3078fa

* changes:
  HF_Client: Send BTA_HF_CLIENT_RFC_CLOSE_EVT when client_cb == NULL
  HF_Client: Free the RFC if the handle doesn't match

Cert: Always unzip for host tests

Bug: 148096951
Test: ./cert/run_cert_facade_only.sh
Change-Id: I5acd7ba9860b54dc9b5f984dc4c3f301a6bdef85

HCI: Add LE AclManager tests

Bug: 148096951
Test: ./cert/run_cert_facade_only.sh
Change-Id: If41d86e9b34ae5f74a1917a0f5de03d8f7a8892a

HCI: Advertise an address with D for DUT

Bug: 148096951
Test: ./cert/run_cert_facade_only.sh
Change-Id: I4cea9ed386b28a50507563681c3d14ef4df8c74d

HCI: Spell COMPLETE_ADVERTISEMENT

Bug: 148096951
Test: ./run/cert_facade_only.sh
Change-Id: I30a4ed0b6f71a4a546aae95fff39de97da262c6c

Cert: Use site-packages instead of root

* Symbolic link libraries in site-packages instead of root one
* Now we can finally run cert tests in virtualenv without PYTHONPATH

Example command without PYTHONPATH:

    act.py -c $ANDROID_BUILD_TOP/system/bt/gd/cert/host_only_config.json -tf $ANDROID_BUILD_TOP/system/bt/gd/cert/cert_testcases_facade_only -tp $ANDROID_BUILD_TOP/system/bt/gd

Bug: 148147248
Test: set up virtual env and run tests
Change-Id: I42ba29602addab41059916071c5d0b42cf17fd8f

DeviceCert: Add timeout for some test cases

Test: cert/run_device_cert.sh
Change-Id: I494634ed77d15c3c8bbd8bf7082fe004560dc5c5

SecurityRecordDatabase

Store SecurityRecords directly in vector, rather than shared_ptr
Move management of SecurityRecord into separate unit -
SecurityRecordDatabase.

Bug: 142341141
Change-Id: I0cc2dd8a7ddcf5a01117f0ebf7bd68111a93a2c5

Merge "DeviceCert: Fix AclManagerTest"

Merge "RFCOMM: Refuse connection if max RFCOMM ports are used"

Merge "Cert: Add a script to set up the on device cert test"

DeviceCert: Fix AclManagerTest

Test: cert/run_device_cert.sh
Change-Id: I1b749ba95c536b9df55c0a882f21c87a6cf83dd5

SecurityRecord: naming and comments clarification

Change-Id: I1de20bb38d76d1dfd91d6f16e0909d7db7561203

Merge "BQR: Add Bluetooth Quality Report v2 feature"

DeviceCert: Fix LeAdvertisingManagerTest, LeScanningManagerTest

Initialize extended advertising config correctly
Use extended scan commands

Test: cert/run_device_cert.sh
Change-Id: Icac6e753cf0ec47b0ec8d4504769a060fd031c48

Cert: Add a script to set up the on device cert test

This script follows the documentation and other information
derived from working with the testing framework.

Bug: 148147584
Test: ./cert/set_up_and_run_device_cert.sh
Change-Id: Ie89e2c67be36142ad5f445c43945b19c74992062

Cert: Use ResetBuilder in DirectHciTest

Bug: 148096951
Test: ./cert/run_device_cert.sh
Change-Id: If3e8e9af526662d59ade8778ae5949a5e5785cb3

AclManager: Add an incoming connection test

Add FetchIncomingConnection to get the next events.

Bug: 148096951
Test: ./cert/run_cert_facade_only.sh
Change-Id: I68576f027fb4a703e3dc360ac09c2206539feb8a

HCI: Connect without valid clock offsets

Bug: 148096951
Test: ./cert/run_cert_facade_only.sh
Change-Id: I3036f63704028f1deab038bebdfacb84dc5be6d9

Merge "gd: Pair 2 phones"

Merge "Cert: Always unzip the latest files quietly"

gd: Pair 2 phones

Allow GD Pixel3 phones to pair with legacy Pixel2.
Dependent on a few other minor patches too large to be
written in the margin.

Bug: 141555841
Test: Paired GD Pixel 3 to legacy Pixel2
Test: bluetooth_test_gd

Change-Id: I76be337ac2b4041dd679b0caa169156af2f43eed

Cert: Always unzip the latest files quietly

Bug: 148096951
Test: ./cert/run_device_cert.sh after modifying a .proto file
Change-Id: I99c860a9d29bb48b44d99696b0e67a01d333cdd6

Merge "Controller: Add a sleep for the HostTest"

Merge "DeviceCert: Fix some setup/cleanup"

BQR: Add Bluetooth Quality Report v2 feature

The BQR is designed to be separated into two parts, one is for the
stack/framework to be aware/handle (Quality_Event_Mask bit 0 ~ 15, event
triggering) and another one is the stack/framework does not need to
handle further (Quality_Event_Mask bit 16 ~ 31).

The BQR v2 adds four new events:
Bit 4 - Root Inflammation:
When the controller encounters an error it shall report Root
Inflammation event indicating the error code to the host.

Bit 16 - LMP/LL message trace:
The controller sends the LMP/LL message handshaking with the remote
device to the host.

Bit 17 - Bluetooth Multi-profile/Coex scheduling trace:
The controller sends its scheduling information on handling the
Bluetooth multiple profiles and wireless coexistence in the 2.4 Ghz band
to the host.

Bit 18 - Enable the Controller Debug Information mechanism:
After enabling the Controller Debug Information mechanism, the
controller just can autonomously report debug logging information via
the controller Debug Info sub-event to the host.

Bug: 136079361
Test: Verified four new added events.
Change-Id: Ie393606143f0158fbcc607e1f41ceceb00abd461

gd: Let acl manager handle encryption changes

Test: Compiles
Bug: 145638034
Change-Id: I5d1ba13b648d74dcb80b4ef94173b3ee6ea2942e

Merge "Allow link to receive acl manager events"

Merge "Cert: Add virtualenv setup for cert tests"