Refactor of headless test to allow additional tests

Bug: 147316415

Test: bt_headless --device=40:4e:36:56:f3:9b --loop=1 pairing
Test: bt_headless --device=40:4e:36:56:f3:9b --loop=1 --uuid=0x110e sdp

Change-Id: I202c7af885d86fd77fffeec5a87b56ffe18882da

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

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

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"

Allow link to receive acl manager events

Bug: 146086425
Test: bluetooth_test_gd
Change-Id: I479374d345ec5e9173dc684f5df5b8d98dad8e98

gd: Add security policy check and plumb in shim

Bug: 146086425
Test: bluetooth_test_gd

Change-Id: If765b204850a61330989f2e3f33eab2d5ae7dc3b

DeviceCert: Fix some setup/cleanup

Always set time, disable Bluetooth; always pull logs

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

Cert: Add virtualenv setup for cert tests

* Add cert/set_up_virtualenv.sh for cert tests so that:
  1) We can use common IDEs that work with virtualenv to edit GD cert
     tests and libraries
  2) We no longer have to use PYTHONPATH when running tests
  3) Explicitly list dependencies in virtualenv setup to make the setup
     more Android independent as it really just need that two ZIP files
* Fixed some import directives as gd/ should always be the import root
  event when importing from the same directory

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

gd: Plumb through security policy for service

Security policy gets checked to decide auth level

Bug: 140778599
Test: Compiles
Change-Id: I9fae4c0617f9d9826ed36a89e4d01f0d80f6ffa9

Merge "HAL: Use packets to simplify tests"

Merge "gd: Expand shim security api"

gd: Expand shim security api

Bug: 140778599
Test: Compiles

Change-Id: I4b31e126e938ab9f1904966c097526ed7d37c736

Merge "DO NOT MERGE - Merge QQ1A.200205.002 into master"

Merge "L2CAP: Add link->Authenticate() back to classic trigger"

HAL: Use packets to simplify tests

Bug: 148096951
Test: ./cert/run_device_cert.sh && ./cert/run_cert_facade_only.sh
Change-Id: Ia81fd4fce96368b1317af3a918c2cf3fc36cd233

Controller: Add a sleep for the HostTest

Bug: 149120542
Test: ./cert/run_cert_facade_only.sh
Change-Id: I5d8de7b3599e6e4a0478ba37ee6caa178aa0674a

L2CAP: Add link->Authenticate() back to classic trigger

Bug: 145638034
Test: bluetooth_test_gd
Change-Id: I7df8f5df6524b9ea617a0f1d1051a187777dd9a8

Cert: Sample code to skip a test

Note: It won't show a test as Passed, but Skipped instead

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

Cert: Run all test suites

Test: cert/run_device_cert.sh and cert/run_cert_facade_only.sh
Change-Id: I4e153e18c1824b4a59c13a6c89c2738c7b2fd723

DeviceCert: Set device time on test run

To ensure we get consistent timestamp

Test: cert/run_device_cert.sh
Bug: 148147248
Change-Id: I8bdf0cbe1a858d748787f5e19bff9bc523cb5621

RFCOMM: Refuse connection if max RFCOMM ports are used

Issue:
When max RFCOMM ports are used, the stack returns the current listening
port as new port while accepting the incoming RFCOMM connection.
This leads to an assert.

Steps to reproduce
==================
1. Connect DUT with RFCOMM clients until all 30 RFCOMM ports used.
   (DUT using 17 ports for RFCOMM servers and 13 ports for clients)
3. Initiate SPP connection from remote to DUT

Bluetooth restarted due to assert in RFCOMM

Expected results: Incoming connection fails
Fix: Return null instead of current listening port when new port
     creation fails

Test: Simulated scenario by reducing max rfc ports and tested
Bug: 147075796
Change-Id: I46cd8e6707575e9cb92670136c9d6e4b9988eb53

Cert: Avoid negative wait times

Bug: 148096951
Test: ./cert/run_device_cert.sh && ./cert/run_cert_facade_only.sh
Change-Id: I58c71b9ee0cadf00342db3863dcb565914250583

HCI: Use ExtendedAdvertising in tests

Bug: 148096951
Test: ./cert/run_device_cert.sh && ./cert/run_cert_facade_only.sh
Change-Id: Ie61cae7108eecada751c1b76bfd712a89e5dac1a